Added FIXME about registering jobs in apps.php

diff --git a/apps/files_sharing/appinfo/app.php b/apps/files_sharing/appinfo/app.php
index a0448f7a6a74c6dedd1c7a0306ec0eedc18bbdb1..d009fbca3b9ba893be47e4329401dcf65d43cff4 100644 (file)
--- a/apps/files_sharing/appinfo/app.php
+++ b/apps/files_sharing/appinfo/app.php
@@ -51,6 +51,8 @@ OCP\Share::registerBackend('folder', 'OC_Share_Backend_Folder', 'file');
 OCP\Util::addScript('files_sharing', 'share');
 OCP\Util::addScript('files_sharing', 'external');
 
+// FIXME: registering a job here will cause additional useless SQL queries
+// when the route is not cron.php, needs a better way
 \OC::$server->getJobList()->add('OCA\Files_sharing\Lib\DeleteOrphanedSharesJob');
 
 \OC::$server->getActivityManager()->registerExtension(function() {