Daimona / BotRiconferme

includes/Task/CreatePage.php

Spacing +46 added lines, -46 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare( strict_types=1 );
+<?php declare(strict_types=1);
 
 namespace BotRiconferme\Task;
 
@@ -11,42 +11,42 @@ 
 	 * @inheritDoc
 	 */
 	public function run() : TaskResult {
-		$this->getLogger()->info( 'Starting task CreatePage' );
+		$this->getLogger()->info('Starting task CreatePage');
 		$users = $this->getDataProvider()->getUsersToProcess();
 
-		foreach ( $users as $user => $groups ) {
-			$this->processUser( $user, $groups );
+		foreach ($users as $user => $groups) {
+			$this->processUser($user, $groups);
 		}
 
-		$this->getLogger()->info( 'Task CreatePage completed successfully' );
-		return new TaskResult( self::STATUS_OK );
+		$this->getLogger()->info('Task CreatePage completed successfully');
+		return new TaskResult(self::STATUS_OK);
 	}
 
 	/**
 	 * @param string $user
 	 * @param array $groups
 	 */
-	protected function processUser( string $user, array $groups ) {
+	protected function processUser(string $user, array $groups) {
 		try {
-			$num = $this->getLastPageNum( $user ) + 1;
-		} catch ( TaskException $e ) {
+			$num = $this->getLastPageNum($user) + 1;
+		} catch (TaskException $e) {
 			// The page was already created.
-			$this->getDataProvider()->removeUser( $user );
-			$this->getLogger()->warning( $e->getMessage() . "\nRemoving $user." );
+			$this->getDataProvider()->removeUser($user);
+			$this->getLogger()->warning($e->getMessage() . "\nRemoving $user.");
 			return;
 		}
 
-		$baseTitle = $this->getConfig()->get( 'ric-main-page' ) . "/$user";
+		$baseTitle = $this->getConfig()->get('ric-main-page') . "/$user";
 		$pageTitle = "$baseTitle/$num";
-		$this->doCreatePage( $pageTitle, $user, $groups );
+		$this->doCreatePage($pageTitle, $user, $groups);
 
-		$newText = str_replace( '$title', $pageTitle, $this->getConfig()->get( 'ric-base-page-text' ) );
-		if ( $num === 1 ) {
-			$this->createBasePage( $baseTitle, $newText );
+		$newText = str_replace('$title', $pageTitle, $this->getConfig()->get('ric-base-page-text'));
+		if ($num === 1) {
+			$this->createBasePage($baseTitle, $newText);
 		} else {
-			$this->updateBasePage( $baseTitle, $newText );
+			$this->updateBasePage($baseTitle, $newText);
 		}
-		$this->getDataProvider()->addCreatedPages( $pageTitle );
+		$this->getDataProvider()->addCreatedPages($pageTitle);
 	}
 
 	/**
@@ -54,9 +54,9 @@ 
 	 * @return int
 	 * @throws TaskException
 	 */
-	protected function getLastPageNum( string $user ) : int {
-		$this->getLogger()->debug( "Retrieving previous pages for $user" );
-		$unprefixedTitle = explode( ':', $this->getConfig()->get( 'ric-main-page' ), 2 )[1];
+	protected function getLastPageNum(string $user) : int {
+		$this->getLogger()->debug("Retrieving previous pages for $user");
+		$unprefixedTitle = explode(':', $this->getConfig()->get('ric-main-page'), 2)[1];
 		$params = [
 			'action' => 'query',
 			'list' => 'allpages',
@@ -65,16 +65,16 @@ 
 			'aplimit' => 'max'
 		];
 
-		$res = ( RequestBase::newFromParams( $params ) )->execute();
+		$res = (RequestBase::newFromParams($params))->execute();
 
 		$last = 0;
-		foreach ( $res['query']['allpages'] as $page ) {
-			if ( $this->pageWasCreatedToday( $page['title'] ) ) {
-				throw new TaskException( 'Page ' . $page['title'] . ' was already created.' );
+		foreach ($res['query']['allpages'] as $page) {
+			if ($this->pageWasCreatedToday($page['title'])) {
+				throw new TaskException('Page ' . $page['title'] . ' was already created.');
 			}
-			$bits = explode( '/', $page['title'] );
-			$cur = intval( end( $bits ) );
-			if ( is_numeric( $cur ) && $cur > $last ) {
+			$bits = explode('/', $page['title']);
+			$cur = intval(end($bits));
+			if (is_numeric($cur) && $cur > $last) {
 				$last = $cur;
 			}
 		}
@@ -85,7 +85,7 @@ 
 	 * @param string $title
 	 * @return bool
 	 */
-	private function pageWasCreatedToday( string $title ) : bool {
+	private function pageWasCreatedToday(string $title) : bool {
 		$params = [
 			'action' => 'query',
 			'prop' => 'revisions',
@@ -96,10 +96,10 @@ 
 			'rvdir' => 'newer'
 		];
 
-		$res = ( RequestBase::newFromParams( $params ) )->execute();
+		$res = (RequestBase::newFromParams($params))->execute();
 		$data = $res['query']['pages'];
-		$time = strtotime( reset( $data )['revisions'][0]['timestamp'] );
-		return date( 'z/Y' ) === date( 'z/Y', $time );
+		$time = strtotime(reset($data)['revisions'][0]['timestamp']);
+		return date('z/Y') === date('z/Y', $time);
 	}
 
 	/**
@@ -107,55 +107,55 @@ 
 	 * @param string $user
 	 * @param array $groups
 	 */
-	protected function doCreatePage( string $title, string $user, array $groups ) {
-		$this->getLogger()->info( "Creating page $title" );
-		$text = $this->getConfig()->get( 'ric-page-text' );
+	protected function doCreatePage(string $title, string $user, array $groups) {
+		$this->getLogger()->info("Creating page $title");
+		$text = $this->getConfig()->get('ric-page-text');
 		$textParams = [
 			'$user' => $user,
 			'$date' => $groups['sysop'],
 			'$burocrate' => $groups['bureaucrat'] ?? '',
 			'$checkuser' => $groups['checkuser'] ?? ''
 		];
-		$text = strtr( $text, $textParams );
+		$text = strtr($text, $textParams);
 
 		$params = [
 			'title' => $title,
 			'text' => $text,
-			'summary' => $this->getConfig()->get( 'ric-page-summary' )
+			'summary' => $this->getConfig()->get('ric-page-summary')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 
 	/**
 	 * @param string $title
 	 * @param string $newText
 	 */
-	protected function createBasePage( string $title, string $newText ) {
-		$this->getLogger()->info( "Creating base page $title" );
+	protected function createBasePage(string $title, string $newText) {
+		$this->getLogger()->info("Creating base page $title");
 
 		$params = [
 			'title' => $title,
 			'text' => $newText,
-			'summary' => $this->getConfig()->get( 'ric-base-page-summary' )
+			'summary' => $this->getConfig()->get('ric-base-page-summary')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 
 	/**
 	 * @param string $title
 	 * @param string $newText
 	 */
-	protected function updateBasePage( string $title, string $newText ) {
-		$this->getLogger()->info( "Updating base page $title" );
+	protected function updateBasePage(string $title, string $newText) {
+		$this->getLogger()->info("Updating base page $title");
 
 		$params = [
 			'title' => $title,
 			'appendtext' => $newText,
-			'summary' => $this->getConfig()->get( 'ric-base-page-summary-update' )
+			'summary' => $this->getConfig()->get('ric-base-page-summary-update')
 		];
 
-		$this->getController()->editPage( $params );
+		$this->getController()->editPage($params);
 	}
 }