diff options
author | jenkins-bot <jenkins-bot@gerrit.wikimedia.org> | 2021-12-20 22:40:05 +0000 |
---|---|---|
committer | Gerrit Code Review <gerrit@wikimedia.org> | 2021-12-20 22:40:05 +0000 |
commit | 55a76edf33f2077b71695b1cf06fcfb6ac42099a (patch) | |
tree | 4ebcce6003d4de334f070f2a6f5ebf3971acec40 | |
parent | 8fda40e95f5ce3c1107291073446d78c3ae74eed (diff) | |
parent | e85a8f3032233882df0db50d2acb6b5bbe300c85 (diff) |
Merge "Reorder constructor params to UndeletePage"
-rw-r--r-- | includes/page/PageCommandFactory.php | 6 | ||||
-rw-r--r-- | includes/page/UndeletePage.php | 22 |
2 files changed, 14 insertions, 14 deletions
diff --git a/includes/page/PageCommandFactory.php b/includes/page/PageCommandFactory.php index 1cd68de41c8f..6cbb42802d37 100644 --- a/includes/page/PageCommandFactory.php +++ b/includes/page/PageCommandFactory.php @@ -343,10 +343,10 @@ class PageCommandFactory implements $this->undeletePageLogger, $this->revisionStore, $this->wikiPageFactory, - $page, - $authority, $this->pageUpdaterFactory, - $this->contentHandlerFactory + $this->contentHandlerFactory, + $page, + $authority ); } } diff --git a/includes/page/UndeletePage.php b/includes/page/UndeletePage.php index 0a522dee4b9a..071e8a1cb133 100644 --- a/includes/page/UndeletePage.php +++ b/includes/page/UndeletePage.php @@ -71,6 +71,10 @@ class UndeletePage { private $revisionStore; /** @var WikiPageFactory */ private $wikiPageFactory; + /** @var PageUpdaterFactory */ + private $pageUpdaterFactory; + /** @var IContentHandlerFactory */ + private $contentHandlerFactory; /** @var ProperPageIdentity */ private $page; @@ -88,10 +92,6 @@ class UndeletePage { private $unsuppress = false; /** @var string[] */ private $tags = []; - /** @var PageUpdaterFactory */ - private $pageUpdaterFactory; - /** @var IContentHandlerFactory */ - private $contentHandlerFactory; /** * @param HookContainer $hookContainer @@ -102,10 +102,10 @@ class UndeletePage { * @param LoggerInterface $logger * @param RevisionStore $revisionStore * @param WikiPageFactory $wikiPageFactory - * @param ProperPageIdentity $page - * @param Authority $performer * @param PageUpdaterFactory $pageUpdaterFactory * @param IContentHandlerFactory $contentHandlerFactory + * @param ProperPageIdentity $page + * @param Authority $performer */ public function __construct( HookContainer $hookContainer, @@ -116,10 +116,10 @@ class UndeletePage { LoggerInterface $logger, RevisionStore $revisionStore, WikiPageFactory $wikiPageFactory, - ProperPageIdentity $page, - Authority $performer, PageUpdaterFactory $pageUpdaterFactory, - IContentHandlerFactory $contentHandlerFactory + IContentHandlerFactory $contentHandlerFactory, + ProperPageIdentity $page, + Authority $performer ) { $this->hookRunner = new HookRunner( $hookContainer ); $this->jobQueueGroup = $jobQueueGroup; @@ -129,11 +129,11 @@ class UndeletePage { $this->logger = $logger; $this->revisionStore = $revisionStore; $this->wikiPageFactory = $wikiPageFactory; + $this->pageUpdaterFactory = $pageUpdaterFactory; + $this->contentHandlerFactory = $contentHandlerFactory; $this->page = $page; $this->performer = $performer; - $this->pageUpdaterFactory = $pageUpdaterFactory; - $this->contentHandlerFactory = $contentHandlerFactory; } /** |