refactor(ApPersonService.ts): 返り値をvoidに統一

この返り値を参照しているコードは見当たらなかった。
また、普通に意味がない値であるように見受けられた。
This commit is contained in:
okayurisotto 2023-07-03 20:41:09 +09:00
parent c2c2efb7fb
commit bfa0fcd6f0

View File

@ -397,7 +397,7 @@ export class ApPersonService implements OnModuleInit {
* @param movePreventUris URIがPersonのmovedToに指定されていたり10回より多く回っている場合これ以上アカウント移行を行わない * @param movePreventUris URIがPersonのmovedToに指定されていたり10回より多く回っている場合これ以上アカウント移行を行わない
*/ */
@bindThis @bindThis
public async updatePerson(uri: string, resolver?: Resolver | null, hint?: IObject, movePreventUris: string[] = []): Promise<string | void> { public async updatePerson(uri: string, resolver?: Resolver | null, hint?: IObject, movePreventUris: string[] = []): Promise<void> {
if (typeof uri !== 'string') throw new Error('uri is not string'); if (typeof uri !== 'string') throw new Error('uri is not string');
// URIがこのサーバーを指しているならスキップ // URIがこのサーバーを指しているならスキップ
@ -536,17 +536,14 @@ export class ApPersonService implements OnModuleInit {
exist.movedAt.getTime() + 1000 * 60 * 60 * 24 * 14 < updated.movedAt.getTime() exist.movedAt.getTime() + 1000 * 60 * 60 * 24 * 14 < updated.movedAt.getTime()
)) { )) {
this.logger.info(`Start to process Move of @${updated.username}@${updated.host} (${uri})`); this.logger.info(`Start to process Move of @${updated.username}@${updated.host} (${uri})`);
return this.processRemoteMove(updated, movePreventUris) await this.processRemoteMove(updated, movePreventUris)
.then(result => { .then(result => {
this.logger.info(`Processing Move Finished [${result}] @${updated.username}@${updated.host} (${uri})`); this.logger.info(`Processing Move Finished [${result}] @${updated.username}@${updated.host} (${uri})`);
return result;
}) })
.catch(e => { .catch(e => {
this.logger.info(`Processing Move Failed @${updated.username}@${updated.host} (${uri})`, { stack: e }); this.logger.info(`Processing Move Failed @${updated.username}@${updated.host} (${uri})`, { stack: e });
}); });
} }
return 'skip';
} }
/** /**