fix(flow_notifications): Improve app loading by checking if notifications app is not installed and enabled when registering the app

Signed-off-by: Faisal Alghamdi <[email protected]>

Author: falghamdi125

lib/Listener/RegisterOperationsListener.php

@@ -27,25 +27,33 @@
 
 use OCA\FlowNotifications\AppInfo\Application;
 use OCA\FlowNotifications\Flow\Operation;
+use OCP\App\IAppManager;
 use OCP\EventDispatcher\Event;
 use OCP\EventDispatcher\IEventListener;
 use OCP\Util;
 use OCP\WorkflowEngine\Events\RegisterOperationsEvent;
+use Psr\Log\LoggerInterface;
 
 /**
  * @template-implements IEventListener<Event|RegisterOperationsEvent>
  */
 class RegisterOperationsListener implements IEventListener {
 	public function __construct(
 		protected readonly Operation $operation,
+		protected IAppManager $appManager,
+		private readonly LoggerInterface $logger,
 	) {
 	}
 
 	public function handle(Event $event): void {
 		if (!$event instanceof RegisterOperationsEvent) {
 			return;
+		} elseif (!$this->appManager->isInstalled('notifications') ||
+		!$this->appManager->isEnabledForUser('notifications')) {
+			$this->logger->error('Failed to register `flow_notifications` app. This could happen due to the `notifications` app isn\'t installed or enabled.', [ 'app' => 'flow_notifications']);
+			return;
 		}
-
+		
 		$event->registerOperation($this->operation);
 		Util::addScript(Application::APP_ID, 'flow_notifications-main');
 	}