Migrate activity app from ILogger

Migrated files in:

- `lib\AppInfo`
- `lib\Controller`
- `lib\Command`

Resolves : https://github.com/nextcloud/server/issues/32127

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>

lib/AppInfo/Application.php

@@ -47,7 +47,6 @@ use OCP\AppFramework\Bootstrap\IRegistrationContext;
 use OCP\IConfig;
 use OCP\IDateTimeFormatter;
 use OCP\IDBConnection;
-use OCP\ILogger;
 use OCP\IURLGenerator;
 use OCP\IUserManager;
 use OCP\L10N\IFactory;
@@ -127,7 +126,7 @@ class Application extends App implements IBootstrap {
 				$c->get(IManager::class),
 				$c->get(IValidator::class),
 				$c->get(IConfig::class),
-				$c->get(ILogger::class)
+				$c->get(LoggerInterface::class),
 			);
 		});
 

lib/Command/SendEmails.php

@@ -27,7 +27,7 @@ use OC\Core\Command\Base;
 use OCA\Activity\MailQueueHandler;
 use OCA\Activity\UserSettings;
 use OCP\IConfig;
-use OCP\ILogger;
+use Psr\Log\LoggerInterface;
 use Stecman\Component\Symfony\Console\BashCompletion\CompletionContext;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -35,23 +35,14 @@ use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class SendEmails extends Base {
-	/** @var MailQueueHandler */
-	protected $queueHandler;
-
-	/** @var IConfig */
-	protected $config;
-
-	/** @var ILogger */
-	protected $logger;
-
 	/**
 	 * @param MailQueueHandler $queueHandler
 	 * @param IConfig $config
-	 * @param ILogger $logger
+	 * @param LoggerInterface $logger
 	 */
-	public function __construct(MailQueueHandler $queueHandler,
-		IConfig $config,
-		ILogger $logger) {
+	public function __construct(protected MailQueueHandler $queueHandler,
+		protected IConfig $config,
+		protected LoggerInterface $logger) {
 		parent::__construct();
 
 		$this->queueHandler = $queueHandler;

lib/Controller/APIv1Controller.php

@@ -30,18 +30,6 @@ use OCP\AppFramework\OCSController;
 use OCP\IRequest;
 
 class APIv1Controller extends OCSController {
-	/** @var Data */
-	protected $data;
-
-	/** @var GroupHelper */
-	protected $groupHelper;
-
-	/** @var UserSettings */
-	protected $userSettings;
-
-	/** @var CurrentUser */
-	protected $currentUser;
-
 	/**
 	 * @param string $appName
 	 * @param IRequest $request
@@ -52,16 +40,11 @@ class APIv1Controller extends OCSController {
 	 */
 	public function __construct($appName,
 		IRequest $request,
-		Data $data,
-		GroupHelper $groupHelper,
-		UserSettings $userSettings,
-		CurrentUser $currentUser) {
+		protected Data $data,
+		protected GroupHelper $groupHelper,
+		protected UserSettings $userSettings,
+		protected CurrentUser $currentUser) {
 		parent::__construct($appName, $request);
-
-		$this->data = $data;
-		$this->userSettings = $userSettings;
-		$this->groupHelper = $groupHelper;
-		$this->currentUser = $currentUser;
 	}
 
 	/**

lib/Controller/APIv2Controller.php

@@ -67,54 +67,20 @@ class APIv2Controller extends OCSController {
 	/** @var bool */
 	protected $loadPreviews;
 
-	/** @var IManager */
-	protected $activityManager;
-
-	/** @var Data */
-	protected $data;
-
-	/** @var GroupHelper */
-	protected $helper;
-
-	/** @var UserSettings */
-	protected $settings;
-
-	/** @var IURLGenerator */
-	protected $urlGenerator;
-
-	/** @var IUserSession */
-	protected $userSession;
-
-	/** @var IPreview */
-	protected $preview;
-
-	/** @var IMimeTypeDetector */
-	protected $mimeTypeDetector;
-
-	/** @var ViewInfoCache */
-	protected $infoCache;
-
 	public function __construct($appName,
 		IRequest $request,
-		IManager $activityManager,
-		Data $data,
-		GroupHelper $helper,
-		UserSettings $settings,
-		IURLGenerator $urlGenerator,
-		IUserSession $userSession,
-		IPreview $preview,
-		IMimeTypeDetector $mimeTypeDetector,
-		ViewInfoCache $infoCache) {
+		protected IManager $activityManager,
+		protected Data $data,
+		protected GroupHelper $helper,
+		protected UserSettings $settings,
+		protected IURLGenerator $urlGenerator,
+		protected IUserSession $userSession,
+		protected IPreview $preview,
+		protected IMimeTypeDetector $mimeTypeDetector,
+		protected ViewInfoCache $infoCache,
+		) {
 		parent::__construct($appName, $request);
 		$this->activityManager = $activityManager;
-		$this->data = $data;
-		$this->helper = $helper;
-		$this->settings = $settings;
-		$this->urlGenerator = $urlGenerator;
-		$this->userSession = $userSession;
-		$this->preview = $preview;
-		$this->mimeTypeDetector = $mimeTypeDetector;
-		$this->infoCache = $infoCache;
 	}
 
 	/**

lib/Controller/ActivitiesController.php

@@ -33,6 +33,7 @@ use OCP\IL10N;
 use OCP\IRequest;
 
 class ActivitiesController extends Controller {
+
 	public function __construct(
 		string $appName,
 		IRequest $request,
@@ -40,8 +41,7 @@ class ActivitiesController extends Controller {
 		protected Data $data,
 		protected Navigation $navigation,
 		protected IEventDispatcher $eventDispatcher,
-		private IL10N $l10n
-	) {
+		private IL10N $l10n) {
 		parent::__construct($appName, $request);
 	}
 

lib/Controller/RemoteActivityController.php

@@ -36,34 +36,14 @@ use OCP\IUser;
 use OCP\IUserManager;
 
 class RemoteActivityController extends OCSController {
-	/** @var IDBConnection */
-	protected $db;
-
-	/** @var IUserManager */
-	protected $userManager;
-
-	/** @var IAppManager */
-	protected $appManager;
-
-	/** @var IRootFolder */
-	protected $rootFolder;
-
-	/** @var IActivityManager */
-	protected $activityManager;
-
 	public function __construct($appName,
 		IRequest $request,
-		IDBConnection $db,
-		IUserManager $userManager,
-		IAppManager $appManager,
-		IRootFolder $rootFolder,
-		IActivityManager $activityManager) {
+		protected IDBConnection $db,
+		protected IUserManager $userManager,
+		protected IAppManager $appManager,
+		protected IRootFolder $rootFolder,
+		protected IActivityManager $activityManager) {
 		parent::__construct($appName, $request);
-		$this->db = $db;
-		$this->userManager = $userManager;
-		$this->appManager = $appManager;
-		$this->rootFolder = $rootFolder;
-		$this->activityManager = $activityManager;
 	}
 
 	/**