bug fixes

packages/fps-importer/index.ts

@@ -127,7 +127,6 @@ class FpsProblemImportHandler extends Handler {
     }
 }
 
-export const sideEffect = true;
 export async function apply(ctx: Context) {
     ctx.Route('problem_import_fps', '/problem/import/fps', FpsProblemImportHandler, PERM.PERM_CREATE_PROBLEM);
     ctx.inject('ProblemAdd', 'problem_import_fps', { icon: 'copy', text: 'From FPS File' });

packages/hydrojudge/src/hosts/builtin.ts

@@ -4,17 +4,16 @@ import path from 'path';
 import fs from 'fs-extra';
 import type { JudgeResultBody } from 'hydrooj';
 import {
-    Logger, SettingModel, StorageModel, SystemModel, TaskModel,
+    SettingModel, StorageModel, SystemModel, TaskModel,
 } from 'hydrooj';
 import { end, next } from 'hydrooj/src/handler/judge';
 import { processTestdata } from '../cases';
 import { getConfig } from '../config';
 import { FormatError, SystemError } from '../error';
 import { Context } from '../judge/interface';
+import logger from '../log';
 import { JudgeTask } from '../task';
 
-const logger = new Logger('judge');
-
 const session = {
     config: { detail: SystemModel.get('hydrojudge.detail') },
     async fetchFile(name: string) {

packages/hydrojudge/src/service.ts

@@ -16,7 +16,6 @@ declare module 'hydrooj' {
 }
 
 export function apply(ctx: Context) {
-    if (process.env.NODE_APP_INSTANCE === '0') {
-        ctx.once('app/started', () => require('./hosts/builtin').postInit());
-    }
+    if (process.env.NODE_APP_INSTANCE !== '0') return;
+    ctx.once('app/started', () => require('./hosts/builtin').postInit());
 }

packages/migrate/scripts/syzoj.ts

@@ -370,10 +370,11 @@ export async function run({
     const ddocs = await query('SELECT * FROM `article`');
     const didMap = {};
     for (const ddoc of ddocs) {
+        const _id = Time.getObjectID(new Date(ddoc.public_time * 1000), false);
         const data: Partial<DiscussionDoc> = {
-            _id: Time.getObjectID(new Date(ddoc.public_time * 1000), false),
+            _id,
             docType: DocumentModel.TYPE_DISCUSSION,
-            docId: Time.getObjectID(new Date(ddoc.public_time * 1000), false),
+            docId: _id,
             owner: uidMap[ddoc.user_id] || 0,
             title: ddoc.title,
             content: ddoc.content,
@@ -401,10 +402,11 @@ export async function run({
     */
     const drdocs = await query('SELECT * FROM `article_comment`');
     for (const drdoc of drdocs) {
+        const _id = Time.getObjectID(new Date(drdoc.public_time * 1000), false);
         const data: Partial<DiscussionReplyDoc> = {
-            _id: Time.getObjectID(new Date(drdoc.public_time * 1000), false),
+            _id,
             domainId,
-            docId: Time.getObjectID(new Date(drdoc.public_time * 1000), false),
+            docId: _id,
             docType: DocumentModel.TYPE_DISCUSSION_REPLY,
             content: drdoc.content,
             owner: uidMap[drdoc.user_id],