Skip to content

Commit

Permalink
fix lint
Browse files Browse the repository at this point in the history
Signed-off-by: Swikriti Tripathi <swikriti808@gmail.com>
  • Loading branch information
SwikritiT committed Oct 17, 2023
1 parent ae683d3 commit 4296367
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 17 deletions.
14 changes: 1 addition & 13 deletions lib/Controller/FilesController.php
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -81,11 +80,6 @@ class FilesController extends OCSController {
*/
private $davUtils;

/**
* @var LoggerInterface
*/
private $log;

/**
* @param string $appName
* @param IRequest $request
Expand All @@ -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,
Expand All @@ -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();
Expand All @@ -127,7 +116,6 @@ public function __construct(string $appName,
$this->userManager = $userManager;
$this->appManager = $appManager;
$this->davUtils = $davUtils;
$this->log = $log;
}

/**
Expand Down
5 changes: 1 addition & 4 deletions tests/lib/Controller/FilesControllerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -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;

/**
Expand Down Expand Up @@ -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();
Expand Down Expand Up @@ -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();
Expand Down

0 comments on commit 4296367

Please sign in to comment.