Skip to content

Commit

Permalink
tweak 335c6a8
Browse files Browse the repository at this point in the history
  • Loading branch information
noridev committed Nov 24, 2023
1 parent 6a860a4 commit 617dbfa
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 9 deletions.
16 changes: 9 additions & 7 deletions packages/backend/src/server/api/endpoints/admin/emoji/steal.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,16 +4,16 @@
*/

import { Inject, Injectable } from '@nestjs/common';
import { IsNull } from 'typeorm';
import { Endpoint } from '@/server/api/endpoint-base.js';
import type { EmojisRepository } from '@/models/index.js';
import type { EmojisRepository } from '@/models/_.js';
import { IdService } from '@/core/IdService.js';
import type { DriveFile } from '@/models/entities/DriveFile.js';
import type { MiDriveFile } from '@/models/DriveFile.js';
import { DI } from '@/di-symbols.js';
import { DriveService } from '@/core/DriveService.js';
import { GlobalEventService } from '@/core/GlobalEventService.js';
import { EmojiEntityService } from '@/core/entities/EmojiEntityService.js';
import { ApiError } from '../../../error.js';
import { IsNull } from 'typeorm';

export const meta = {
tags: ['admin'],
Expand Down Expand Up @@ -56,9 +56,10 @@ export const paramDef = {
required: ['name', 'host'],
} as const;

// TODO: ロジックをサービスに切り出す

@Injectable()
// eslint-disable-next-line import/no-default-export
export default class extends Endpoint<typeof meta, typeof paramDef> {
export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-disable-next-line import/no-default-export
constructor(
@Inject(DI.emojisRepository)
private emojisRepository: EmojisRepository,
Expand All @@ -80,16 +81,17 @@ export default class extends Endpoint<typeof meta, typeof paramDef> {
throw new ApiError(meta.errors.localEmojiAlreadyExists);
}

let driveFile: DriveFile;
let driveFile: MiDriveFile;

try {
// Create file
driveFile = await this.driveService.uploadFromUrl({ url: emoji.originalUrl, user: null, force: true });
} catch (e) {
throw new ApiError();
}

const copied = await this.emojisRepository.insert({
id: this.idService.genId(),
id: this.idService.gen(),
updatedAt: new Date(),
name: emoji.name,
host: null,
Expand Down
4 changes: 2 additions & 2 deletions packages/frontend/src/components/global/MkCustomEmoji.vue
Original file line number Diff line number Diff line change
Expand Up @@ -96,8 +96,8 @@ function onClick(ev: MouseEvent) {
}, ...(props.host && $i && ($i.isAdmin || $i.policies.canManageCustomEmojis) ? [{
text: i18n.ts.import,
icon: 'ti ti-plus',
action: async () => {
await os.apiWithDialog('admin/emoji/steal', {
action: () => {
os.apiWithDialog('admin/emoji/steal', {
name: customEmojiName.value,
host: props.host,
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -383,6 +383,8 @@ export default function(props: MfmProps) {
normal: props.plain,
host: props.author.host,
useOriginalSize: scale >= 2.5,
menu: props.enableEmojiMenu,
menuReaction: props.enableEmojiMenuReaction,
})];
}
}
Expand Down

0 comments on commit 617dbfa

Please sign in to comment.