Remove dead arguments

Signed-off-by: Joas Schilling <coding@schilljs.com>
This commit is contained in:
Joas Schilling 2017-04-24 10:18:14 +02:00
parent e19126425b
commit 7a87fedac3
No known key found for this signature in database
GPG Key ID: E166FD8976B3BAC8
2 changed files with 1 additions and 45 deletions

View File

@ -31,23 +31,20 @@ namespace OCA\Provisioning_API\Controller;
use OC\Accounts\AccountManager; use OC\Accounts\AccountManager;
use OC\Settings\Mailer\NewUserMailHelper; use OC\Settings\Mailer\NewUserMailHelper;
use \OC_Helper; use OC_Helper;
use OCP\AppFramework\Http\DataResponse; use OCP\AppFramework\Http\DataResponse;
use OCP\AppFramework\OCS\OCSException; use OCP\AppFramework\OCS\OCSException;
use OCP\AppFramework\OCS\OCSForbiddenException; use OCP\AppFramework\OCS\OCSForbiddenException;
use OCP\AppFramework\OCSController; use OCP\AppFramework\OCSController;
use OCP\Defaults;
use OCP\Files\NotFoundException; use OCP\Files\NotFoundException;
use OCP\IConfig; use OCP\IConfig;
use OCP\IGroup; use OCP\IGroup;
use OCP\IGroupManager; use OCP\IGroupManager;
use OCP\ILogger; use OCP\ILogger;
use OCP\IRequest; use OCP\IRequest;
use OCP\IURLGenerator;
use OCP\IUserManager; use OCP\IUserManager;
use OCP\IUserSession; use OCP\IUserSession;
use OCP\L10N\IFactory; use OCP\L10N\IFactory;
use OCP\Mail\IMailer;
class UsersController extends OCSController { class UsersController extends OCSController {
@ -63,14 +60,6 @@ class UsersController extends OCSController {
private $accountManager; private $accountManager;
/** @var ILogger */ /** @var ILogger */
private $logger; private $logger;
/** @var string */
private $fromMailAddress;
/** @var IURLGenerator */
private $urlGenerator;
/** @var IMailer */
private $mailer;
/** @var Defaults */
private $defaults;
/** @var IFactory */ /** @var IFactory */
private $l10nFactory; private $l10nFactory;
/** @var NewUserMailHelper */ /** @var NewUserMailHelper */
@ -85,10 +74,6 @@ class UsersController extends OCSController {
* @param IUserSession $userSession * @param IUserSession $userSession
* @param AccountManager $accountManager * @param AccountManager $accountManager
* @param ILogger $logger * @param ILogger $logger
* @param string $fromMailAddress
* @param IURLGenerator $urlGenerator
* @param IMailer $mailer
* @param Defaults $defaults
* @param IFactory $l10nFactory * @param IFactory $l10nFactory
* @param NewUserMailHelper $newUserMailHelper * @param NewUserMailHelper $newUserMailHelper
*/ */
@ -100,10 +85,6 @@ class UsersController extends OCSController {
IUserSession $userSession, IUserSession $userSession,
AccountManager $accountManager, AccountManager $accountManager,
ILogger $logger, ILogger $logger,
$fromMailAddress,
IURLGenerator $urlGenerator,
IMailer $mailer,
Defaults $defaults,
IFactory $l10nFactory, IFactory $l10nFactory,
NewUserMailHelper $newUserMailHelper) { NewUserMailHelper $newUserMailHelper) {
parent::__construct($appName, $request); parent::__construct($appName, $request);
@ -114,10 +95,6 @@ class UsersController extends OCSController {
$this->userSession = $userSession; $this->userSession = $userSession;
$this->accountManager = $accountManager; $this->accountManager = $accountManager;
$this->logger = $logger; $this->logger = $logger;
$this->fromMailAddress = $fromMailAddress;
$this->urlGenerator = $urlGenerator;
$this->mailer = $mailer;
$this->defaults = $defaults;
$this->l10nFactory = $l10nFactory; $this->l10nFactory = $l10nFactory;
$this->newUserMailHelper = $newUserMailHelper; $this->newUserMailHelper = $newUserMailHelper;
} }

View File

@ -70,12 +70,6 @@ class UsersControllerTest extends TestCase {
protected $accountManager; protected $accountManager;
/** @var IRequest|PHPUnit_Framework_MockObject_MockObject */ /** @var IRequest|PHPUnit_Framework_MockObject_MockObject */
protected $request; protected $request;
/** @var IURLGenerator|PHPUnit_Framework_MockObject_MockObject */
private $urlGenerator;
/** @var IMailer|PHPUnit_Framework_MockObject_MockObject */
private $mailer;
/** @var Defaults|PHPUnit_Framework_MockObject_MockObject */
private $defaults;
/** @var IFactory|PHPUnit_Framework_MockObject_MockObject */ /** @var IFactory|PHPUnit_Framework_MockObject_MockObject */
private $l10nFactory; private $l10nFactory;
/** @var NewUserMailHelper|PHPUnit_Framework_MockObject_MockObject */ /** @var NewUserMailHelper|PHPUnit_Framework_MockObject_MockObject */
@ -91,9 +85,6 @@ class UsersControllerTest extends TestCase {
$this->logger = $this->createMock(ILogger::class); $this->logger = $this->createMock(ILogger::class);
$this->request = $this->createMock(IRequest::class); $this->request = $this->createMock(IRequest::class);
$this->accountManager = $this->createMock(AccountManager::class); $this->accountManager = $this->createMock(AccountManager::class);
$this->urlGenerator = $this->createMock(IURLGenerator::class);
$this->mailer = $this->createMock(IMailer::class);
$this->defaults = $this->createMock(Defaults::class);
$this->l10nFactory = $this->createMock(IFactory::class); $this->l10nFactory = $this->createMock(IFactory::class);
$this->newUserMailHelper = $this->createMock(NewUserMailHelper::class); $this->newUserMailHelper = $this->createMock(NewUserMailHelper::class);
@ -107,10 +98,6 @@ class UsersControllerTest extends TestCase {
$this->userSession, $this->userSession,
$this->accountManager, $this->accountManager,
$this->logger, $this->logger,
'test@example.org',
$this->urlGenerator,
$this->mailer,
$this->defaults,
$this->l10nFactory, $this->l10nFactory,
$this->newUserMailHelper $this->newUserMailHelper
]) ])
@ -2664,10 +2651,6 @@ class UsersControllerTest extends TestCase {
$this->userSession, $this->userSession,
$this->accountManager, $this->accountManager,
$this->logger, $this->logger,
'',
$this->urlGenerator,
$this->mailer,
$this->defaults,
$this->l10nFactory, $this->l10nFactory,
$this->newUserMailHelper $this->newUserMailHelper
]) ])
@ -2728,10 +2711,6 @@ class UsersControllerTest extends TestCase {
$this->userSession, $this->userSession,
$this->accountManager, $this->accountManager,
$this->logger, $this->logger,
'',
$this->urlGenerator,
$this->mailer,
$this->defaults,
$this->l10nFactory, $this->l10nFactory,
$this->newUserMailHelper $this->newUserMailHelper
]) ])