diff --git a/lib/Controller/FilesController.php b/lib/Controller/FilesController.php index 2dd532ae3..6fccf7308 100644 --- a/lib/Controller/FilesController.php +++ b/lib/Controller/FilesController.php @@ -12,7 +12,6 @@ namespace OCA\OpenProject\Controller; use Exception; -use OC_Util; use OCA\Activity\Data; use OCA\Activity\GroupHelperDisabled; use OCA\Activity\UserSettings; @@ -81,11 +80,6 @@ class FilesController extends OCSController { */ private $davUtils; - /** - * @var LoggerInterface - */ - private $log; - /** * @param string $appName * @param IRequest $request @@ -95,13 +89,9 @@ class FilesController extends OCSController { * @param IManager $activityManager * @param IAppManager $appManager * @param IDBConnection $connection - * @param IValidator $richObjectValidator * @param ILogger $logger - * @param IL10N $l - * @param IConfig $config * @param IUserManager $userManager * @param DavUtil $davUtils - * @param LoggerInterface $log */ public function __construct(string $appName, @@ -114,8 +104,7 @@ public function __construct(string $appName, IDBConnection $connection, ILogger $logger, IUserManager $userManager, - DavUtil $davUtils, - LoggerInterface $log + DavUtil $davUtils ) { parent::__construct($appName, $request); $this->user = $userSession->getUser(); @@ -127,7 +116,6 @@ public function __construct(string $appName, $this->userManager = $userManager; $this->appManager = $appManager; $this->davUtils = $davUtils; - $this->log = $log; } /** diff --git a/tests/lib/Controller/FilesControllerTest.php b/tests/lib/Controller/FilesControllerTest.php index 5cd473121..b5f1e7358 100644 --- a/tests/lib/Controller/FilesControllerTest.php +++ b/tests/lib/Controller/FilesControllerTest.php @@ -13,7 +13,6 @@ use OCP\IUserManager; use PHPUnit\Framework\MockObject\MockObject; use PHPUnit\Framework\TestCase; -use Psr\Log\LoggerInterface; use function PHPUnit\Framework\assertSame; /** @@ -1014,7 +1013,6 @@ private function createFilesController( $this->createMock(ILogger::class), $this->createMock(IUserManager::class), $this->createMock(DavUtil::class), - $this->createMock(LoggerInterface::class) ); if ($trashManagerMock === null) { $trashManagerMock = $this->getMockBuilder('\OCA\Files_Trashbin\Trash\ITrashManager')->getMock(); @@ -1077,8 +1075,7 @@ public function getFilesControllerMock( $this->createMock(IDBConnection::class), $this->createMock(ILogger::class), $this->createMock(IUserManager::class), - $davUtilsMock, - $this->createMock(LoggerInterface::class) + $davUtilsMock ]) ->onlyMethods($onlyMethods) ->getMock();