Outsource scan actions

[lennartdohmann]

Outsource the consequences of a file scan from tag function, as this is not just tagging and caused a bug in the event handler where the quarantine was created empty.

[Copilot]

Pull Request Overview

This PR refactors the handling of malicious file scan results by moving the quarantine and prefix setting logic from the tagFile method in VerdictService to the actual scan callers. This separation of concerns ensures that file actions are properly executed in all scan contexts.

Key changes:

• Removed quarantine logic from the tagFile method in VerdictService
• Added malicious file handling logic to scan callers (ScanService, ScanController, FileEventsListener)
• Enhanced error handling and logging for failed quarantine operations

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
lib/Service/VerdictService.php	Removed quarantine and prefix setting logic from tagFile method
lib/Service/ScanService.php	Added malicious file handling with quarantine and prefix setting
lib/Controller/ScanController.php	Added malicious file handling with quarantine and prefix setting
lib/EventListener/FileEventsListener.php	Enhanced error handling with fallback quarantine logic

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}