Passed
Push — master ( 390609...8584e7 )
by Daimona
02:06
created
includes/TaskManager.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme;
4 4
 
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 	 * @param Logger $logger
55 55
 	 * @param Wiki $wiki
56 56
 	 */
57
-	public function __construct( Logger $logger, Wiki $wiki ) {
57
+	public function __construct ( Logger $logger, Wiki $wiki ) {
58 58
 		$this->logger = $logger;
59 59
 		$this->wiki = $wiki;
60 60
 		$this->provider = new TaskDataProvider( $this->logger, $this->wiki );
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 	 * @param string|null $name Only used in MODE_TASK and MODE_SUBTASK
68 68
 	 * @return TaskResult
69 69
 	 */
70
-	public function run( string $mode, string $name = null ) : TaskResult {
70
+	public function run ( string $mode, string $name = null ) : TaskResult {
71 71
 		if ( $mode === self::MODE_COMPLETE ) {
72 72
 			return $this->runAllTasks();
73 73
 		} elseif ( $name === null ) {
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 	 *
83 83
 	 * @return TaskResult
84 84
 	 */
85
-	protected function runAllTasks() : TaskResult {
85
+	protected function runAllTasks () : TaskResult {
86 86
 		$orderedList = [
87 87
 			'update-list',
88 88
 			'start-new',
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 	 * @param string $name
105 105
 	 * @return TaskResult
106 106
 	 */
107
-	protected function runTask( string $name ) : TaskResult {
107
+	protected function runTask ( string $name ) : TaskResult {
108 108
 		if ( !isset( self::TASKS_MAP[ $name ] ) ) {
109 109
 			throw new \InvalidArgumentException( "'$name' is not a valid task." );
110 110
 		}
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
 	 * @param string $name
120 120
 	 * @return TaskResult
121 121
 	 */
122
-	protected function runSubtask( string $name ) : TaskResult {
122
+	protected function runSubtask ( string $name ) : TaskResult {
123 123
 		if ( !isset( self::SUBTASKS_MAP[ $name ] ) ) {
124 124
 			throw new \InvalidArgumentException( "'$name' is not a valid subtask." );
125 125
 		}
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
 	 * @param string $class
135 135
 	 * @return Task
136 136
 	 */
137
-	private function getTaskInstance( string $class ) : Task {
137
+	private function getTaskInstance ( string $class ) : Task {
138 138
 		return new $class( $this->logger, $this->wiki, $this->provider );
139 139
 	}
140 140
 
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
 	 * @param string $class
145 145
 	 * @return Subtask
146 146
 	 */
147
-	private function getSubtaskInstance( string $class ) : Subtask {
147
+	private function getSubtaskInstance ( string $class ) : Subtask {
148 148
 		return new $class( $this->logger, $this->wiki, $this->provider );
149 149
 	}
150 150
 }
Please login to merge, or discard this patch.
includes/Config.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme;
4 4
 
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 	/** @var self */
14 14
 	private static $instance;
15 15
 	/** @var array */
16
-	private $opts = [];
16
+	private $opts = [ ];
17 17
 	/** @var Wiki */
18 18
 	private $wiki;
19 19
 
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 	 *
23 23
 	 * @param Wiki $wiki
24 24
 	 */
25
-	private function __construct( Wiki $wiki ) {
25
+	private function __construct ( Wiki $wiki ) {
26 26
 		$this->wiki = $wiki;
27 27
 	}
28 28
 
@@ -33,16 +33,16 @@  discard block
 block discarded – undo
33 33
 	 * @param Wiki $wiki
34 34
 	 * @throws ConfigException
35 35
 	 */
36
-	public static function init( array $defaults, Wiki $wiki ) {
36
+	public static function init ( array $defaults, Wiki $wiki ) {
37 37
 		if ( self::$instance ) {
38 38
 			throw new ConfigException( 'Config was already initialized' );
39 39
 		}
40 40
 
41 41
 		$inst = new self( $wiki );
42
-		$inst->set( 'list-title', $defaults['list-title'] );
43
-		$inst->set( 'msg-title', $defaults['msg-title'] );
44
-		$inst->set( 'username', $defaults['username'] );
45
-		$inst->set( 'password', $defaults['password'] );
42
+		$inst->set( 'list-title', $defaults[ 'list-title' ] );
43
+		$inst->set( 'msg-title', $defaults[ 'msg-title' ] );
44
+		$inst->set( 'username', $defaults[ 'username' ] );
45
+		$inst->set( 'password', $defaults[ 'password' ] );
46 46
 		self::$instance = $inst;
47 47
 
48 48
 		// On-wiki values
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 	 * @return string
63 63
 	 * @throws ConfigException
64 64
 	 */
65
-	public function getWikiMessage( string $key ) : string {
65
+	public function getWikiMessage ( string $key ) : string {
66 66
 		static $messages = null;
67 67
 		if ( $messages === null ) {
68 68
 			try {
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 		if ( !isset( $messages[ $key ] ) ) {
76 76
 			throw new ConfigException( "Message '$key' does not exist." );
77 77
 		}
78
-		return $messages[$key];
78
+		return $messages[ $key ];
79 79
 	}
80 80
 
81 81
 	/**
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 	 * @param string $key
85 85
 	 * @param mixed $value
86 86
 	 */
87
-	protected function set( string $key, $value ) {
87
+	protected function set ( string $key, $value ) {
88 88
 		$this->opts[ $key ] = $value;
89 89
 	}
90 90
 
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
 	 * @return self
95 95
 	 * @throws ConfigException
96 96
 	 */
97
-	public static function getInstance() : self {
97
+	public static function getInstance () : self {
98 98
 		if ( !self::$instance ) {
99 99
 			throw new ConfigException( 'Config not yet initialized' );
100 100
 		}
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
 	 * @return mixed
109 109
 	 * @throws ConfigException
110 110
 	 */
111
-	public function get( string $opt ) {
111
+	public function get ( string $opt ) {
112 112
 		if ( !isset( $this->opts[ $opt ] ) ) {
113 113
 			throw new ConfigException( "Config option '$opt' not set." );
114 114
 		}
Please login to merge, or discard this patch.
includes/Bot.php 1 patch
Spacing   +7 added lines, -8 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme;
4 4
 
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
 	 * @param Logger $logger
20 20
 	 * @param Wiki $wiki
21 21
 	 */
22
-	public function __construct( Logger $logger, Wiki $wiki ) {
22
+	public function __construct ( Logger $logger, Wiki $wiki ) {
23 23
 		$this->logger = $logger;
24 24
 		$this->wiki = $wiki;
25 25
 	}
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 	 * @param string $mode
31 31
 	 * @param string|null $name
32 32
 	 */
33
-	private function run( string $mode = TaskManager::MODE_COMPLETE, string $name = null ) {
33
+	private function run ( string $mode = TaskManager::MODE_COMPLETE, string $name = null ) {
34 34
 		$activity = $mode === TaskManager::MODE_COMPLETE ? TaskManager::MODE_COMPLETE : "$mode $name";
35 35
 		$this->logger->info( "Running $activity" );
36 36
 		$manager = new TaskManager( $this->logger, $this->wiki );
@@ -39,8 +39,7 @@  discard block
 block discarded – undo
39 39
 		$base = "Execution of $activity";
40 40
 		if ( $res->isOK() ) {
41 41
 			$msg = $res->getStatus() === TaskResult::STATUS_NOTHING ?
42
-				': nothing to do' :
43
-				' completed successfully';
42
+				': nothing to do' : ' completed successfully';
44 43
 			$this->logger->info( $base . $msg . ".\n$line\n\n" );
45 44
 		} else {
46 45
 			$this->logger->error( "$base failed.\n$res\n$line\n\n" );
@@ -50,7 +49,7 @@  discard block
 block discarded – undo
50 49
 	/**
51 50
 	 * Entry point for the whole process
52 51
 	 */
53
-	public function runAll() {
52
+	public function runAll () {
54 53
 		$this->run();
55 54
 	}
56 55
 
@@ -59,7 +58,7 @@  discard block
 block discarded – undo
59 58
 	 *
60 59
 	 * @param string $task
61 60
 	 */
62
-	public function runTask( string $task ) {
61
+	public function runTask ( string $task ) {
63 62
 		$this->run( TaskManager::MODE_TASK, $task );
64 63
 	}
65 64
 
@@ -68,7 +67,7 @@  discard block
 block discarded – undo
68 67
 	 *
69 68
 	 * @param string $subtask
70 69
 	 */
71
-	public function runSubtask( string $subtask ) {
70
+	public function runSubtask ( string $subtask ) {
72 71
 		$this->run( TaskManager::MODE_SUBTASK, $subtask );
73 72
 	}
74 73
 }
Please login to merge, or discard this patch.
includes/ContextSource.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme;
4 4
 
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 	 * @param Logger $logger
25 25
 	 * @param Wiki $wiki
26 26
 	 */
27
-	public function __construct( Logger $logger, Wiki $wiki ) {
27
+	public function __construct ( Logger $logger, Wiki $wiki ) {
28 28
 		$this->setLogger( $logger );
29 29
 		$this->setConfig( Config::getInstance() );
30 30
 		$this->setWiki( $wiki );
@@ -33,14 +33,14 @@  discard block
 block discarded – undo
33 33
 	/**
34 34
 	 * @return LoggerInterface
35 35
 	 */
36
-	protected function getLogger() : LoggerInterface {
36
+	protected function getLogger () : LoggerInterface {
37 37
 		return $this->logger;
38 38
 	}
39 39
 
40 40
 	/**
41 41
 	 * @inheritDoc
42 42
 	 */
43
-	public function setLogger( LoggerInterface $logger ) {
43
+	public function setLogger ( LoggerInterface $logger ) {
44 44
 		$this->logger = $logger;
45 45
 	}
46 46
 
@@ -50,35 +50,35 @@  discard block
 block discarded – undo
50 50
 	 * @param string $optname
51 51
 	 * @return mixed
52 52
 	 */
53
-	protected function getOpt( string $optname ) {
53
+	protected function getOpt ( string $optname ) {
54 54
 		return $this->getConfig()->get( $optname );
55 55
 	}
56 56
 
57 57
 	/**
58 58
 	 * @return Config
59 59
 	 */
60
-	protected function getConfig() : Config {
60
+	protected function getConfig () : Config {
61 61
 		return $this->config;
62 62
 	}
63 63
 
64 64
 	/**
65 65
 	 * @param Config $cfg
66 66
 	 */
67
-	protected function setConfig( Config $cfg ) {
67
+	protected function setConfig ( Config $cfg ) {
68 68
 		$this->config = $cfg;
69 69
 	}
70 70
 
71 71
 	/**
72 72
 	 * @return Wiki
73 73
 	 */
74
-	protected function getWiki() : Wiki {
74
+	protected function getWiki () : Wiki {
75 75
 		return $this->wiki;
76 76
 	}
77 77
 
78 78
 	/**
79 79
 	 * @param Wiki $wiki
80 80
 	 */
81
-	protected function setWiki( Wiki $wiki ) {
81
+	protected function setWiki ( Wiki $wiki ) {
82 82
 		$this->wiki = $wiki;
83 83
 	}
84 84
 
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
 	 * @param string $key
89 89
 	 * @return Message
90 90
 	 */
91
-	protected function msg( string $key ) : Message {
91
+	protected function msg ( string $key ) : Message {
92 92
 		return new Message( $key );
93 93
 	}
94 94
 
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 	 * @param string $title
99 99
 	 * @return Page
100 100
 	 */
101
-	protected function getPage( string $title ) : Page {
101
+	protected function getPage ( string $title ) : Page {
102 102
 		return new Page( $title, $this->getWiki() );
103 103
 	}
104 104
 }
Please login to merge, or discard this patch.
includes/TaskDataProvider.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme;
4 4
 
@@ -13,16 +13,16 @@  discard block
 block discarded – undo
13 13
 	/** @var array[] */
14 14
 	private $processUsers;
15 15
 	/** @var PageRiconferma[] */
16
-	private $createdPages = [];
16
+	private $createdPages = [ ];
17 17
 
18 18
 	/**
19 19
 	 * Get a list of users to execute tasks on.
20 20
 	 *
21 21
 	 * @return array[]
22 22
 	 */
23
-	public function getUsersToProcess() : array {
23
+	public function getUsersToProcess () : array {
24 24
 		if ( $this->processUsers === null ) {
25
-			$this->processUsers = [];
25
+			$this->processUsers = [ ];
26 26
 			foreach ( PageBotList::get( $this->getWiki() )->getAdminsList() as $user => $groups ) {
27 27
 				if ( $this->shouldAddUser( $groups ) ) {
28 28
 					$this->processUsers[ $user ] = $groups;
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 	 * @param string[] $groups
40 40
 	 * @return bool
41 41
 	 */
42
-	private function shouldAddUser( array $groups ) : bool {
42
+	private function shouldAddUser ( array $groups ) : bool {
43 43
 		$override = true;
44 44
 		$timestamp = PageBotList::getOverrideTimestamp( $groups );
45 45
 
@@ -58,10 +58,10 @@  discard block
 block discarded – undo
58 58
 	 *
59 59
 	 * @return PageRiconferma[]
60 60
 	 */
61
-	public function getOpenPages() : array {
61
+	public function getOpenPages () : array {
62 62
 		static $list = null;
63 63
 		if ( $list === null ) {
64
-			$list = [];
64
+			$list = [ ];
65 65
 			$mainTitle = $this->getOpt( 'main-page-title' );
66 66
 			$params = [
67 67
 				'action' => 'query',
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 			$pages = RequestBase::newFromParams( $params )->execute()->query->pages;
76 76
 			foreach ( reset( $pages )->templates as $page ) {
77 77
 				if ( preg_match( "!$titleReg\/[^\/]+\/\d!", $page->title ) ) {
78
-					$list[] = new PageRiconferma( $page->title, $this->getWiki() );
78
+					$list[ ] = new PageRiconferma( $page->title, $this->getWiki() );
79 79
 				}
80 80
 			}
81 81
 		}
@@ -88,13 +88,13 @@  discard block
 block discarded – undo
88 88
 	 *
89 89
 	 * @return PageRiconferma[]
90 90
 	 */
91
-	public function getPagesToClose() : array {
91
+	public function getPagesToClose () : array {
92 92
 		static $list = null;
93 93
 		if ( $list === null ) {
94
-			$list = [];
94
+			$list = [ ];
95 95
 			foreach ( $this->getOpenPages() as $page ) {
96 96
 				if ( time() > $page->getEndTimestamp() ) {
97
-					$list[] = $page;
97
+					$list[ ] = $page;
98 98
 				}
99 99
 			}
100 100
 		}
@@ -106,21 +106,21 @@  discard block
 block discarded – undo
106 106
 	 *
107 107
 	 * @param string $name
108 108
 	 */
109
-	public function removeUser( string $name ) {
109
+	public function removeUser ( string $name ) {
110 110
 		unset( $this->processUsers[ $name ] );
111 111
 	}
112 112
 
113 113
 	/**
114 114
 	 * @return PageRiconferma[]
115 115
 	 */
116
-	public function getCreatedPages() : array {
116
+	public function getCreatedPages () : array {
117 117
 		return $this->createdPages;
118 118
 	}
119 119
 
120 120
 	/**
121 121
 	 * @param PageRiconferma $page
122 122
 	 */
123
-	public function addCreatedPages( PageRiconferma $page ) {
124
-		$this->createdPages[] = $page;
123
+	public function addCreatedPages ( PageRiconferma $page ) {
124
+		$this->createdPages[ ] = $page;
125 125
 	}
126 126
 }
Please login to merge, or discard this patch.
includes/Wiki/User.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme\Wiki;
4 4
 
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 	 * @param string $name
18 18
 	 * @param Wiki $wiki
19 19
 	 */
20
-	public function __construct( string $name, Wiki $wiki ) {
20
+	public function __construct ( string $name, Wiki $wiki ) {
21 21
 		parent::__construct( $wiki );
22 22
 		$this->name = $name;
23 23
 	}
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 	/**
26 26
 	 * @return string
27 27
 	 */
28
-	public function getName() : string {
28
+	public function getName () : string {
29 29
 		return $this->name;
30 30
 	}
31 31
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 	 *
35 35
 	 * @return string[]
36 36
 	 */
37
-	public function getGroups() : array {
37
+	public function getGroups () : array {
38 38
 		return array_keys( $this->getGroupsWithDates() );
39 39
 	}
40 40
 
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 	 *
45 45
 	 * @return string[]
46 46
 	 */
47
-	public function getGroupsWithDates() : array {
47
+	public function getGroupsWithDates () : array {
48 48
 		if ( $this->groups === null ) {
49 49
 			$usersList = PageBotList::get( $this->wiki )->getAdminsList();
50 50
 			$this->groups = $usersList[ $this->name ];
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
 	 * @param string $groupName
59 59
 	 * @return bool
60 60
 	 */
61
-	public function inGroup( string $groupName ) : bool {
61
+	public function inGroup ( string $groupName ) : bool {
62 62
 		return in_array( $groupName, $this->getGroups() );
63 63
 	}
64 64
 
@@ -67,14 +67,14 @@  discard block
 block discarded – undo
67 67
 	 *
68 68
 	 * @inheritDoc
69 69
 	 */
70
-	public function getRegex() : string {
70
+	public function getRegex () : string {
71 71
 		return str_replace( ' ', '[ _]', preg_quote( $this->name ) );
72 72
 	}
73 73
 
74 74
 	/**
75 75
 	 * @return string
76 76
 	 */
77
-	public function __toString() {
77
+	public function __toString () {
78 78
 		return $this->name;
79 79
 	}
80 80
 }
Please login to merge, or discard this patch.
includes/Wiki/Page/PageBotList.php 1 patch
Spacing   +13 added lines, -15 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme\Wiki\Page;
4 4
 
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 	 * @private Use self::get()
14 14
 	 * @param Wiki $wiki
15 15
 	 */
16
-	public function __construct( Wiki $wiki ) {
16
+	public function __construct ( Wiki $wiki ) {
17 17
 		parent::__construct( Config::getInstance()->get( 'list-title' ), $wiki );
18 18
 	}
19 19
 
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 	 * @param Wiki $wiki
24 24
 	 * @return self
25 25
 	 */
26
-	public static function get( Wiki $wiki ) : self {
26
+	public static function get ( Wiki $wiki ) : self {
27 27
 		static $instance = null;
28 28
 		if ( $instance === null ) {
29 29
 			$instance = new self( $wiki );
@@ -35,16 +35,16 @@  discard block
 block discarded – undo
35 35
 	 * @param string[] $groups
36 36
 	 * @return int|null
37 37
 	 */
38
-	public static function getOverrideTimestamp( array $groups ) : ?int {
38
+	public static function getOverrideTimestamp ( array $groups ) : ?int {
39 39
 		if ( !array_intersect_key( $groups, [ 'override-perm' => true, 'override' => true ] ) ) {
40 40
 			return null;
41 41
 		}
42 42
 
43 43
 		// A one-time override takes precedence
44 44
 		if ( array_key_exists( 'override', $groups ) ) {
45
-			$date = $groups['override'];
45
+			$date = $groups[ 'override' ];
46 46
 		} else {
47
-			$date = $groups['override-prem'] . '/' . date( 'Y' );
47
+			$date = $groups[ 'override-prem' ] . '/' . date( 'Y' );
48 48
 		}
49 49
 		return \DateTime::createFromFormat( 'd/m/Y', $date )->getTimestamp();
50 50
 	}
@@ -55,17 +55,15 @@  discard block
 block discarded – undo
55 55
 	 * @param array $groups
56 56
 	 * @return int
57 57
 	 */
58
-	public static function getValidFlagTimestamp( array $groups ): int {
59
-		$checkuser = isset( $groups['checkuser'] ) ?
60
-			\DateTime::createFromFormat( 'd/m/Y', $groups['checkuser'] )->getTimestamp() :
61
-			0;
62
-		$bureaucrat = isset( $groups['bureaucrat'] ) ?
63
-			\DateTime::createFromFormat( 'd/m/Y', $groups['bureaucrat'] )->getTimestamp() :
64
-			0;
58
+	public static function getValidFlagTimestamp ( array $groups ): int {
59
+		$checkuser = isset( $groups[ 'checkuser' ] ) ?
60
+			\DateTime::createFromFormat( 'd/m/Y', $groups[ 'checkuser' ] )->getTimestamp() : 0;
61
+		$bureaucrat = isset( $groups[ 'bureaucrat' ] ) ?
62
+			\DateTime::createFromFormat( 'd/m/Y', $groups[ 'bureaucrat' ] )->getTimestamp() : 0;
65 63
 
66 64
 		$timestamp = max( $bureaucrat, $checkuser );
67 65
 		if ( $timestamp === 0 ) {
68
-			$timestamp = \DateTime::createFromFormat( 'd/m/Y', $groups['sysop'] )->getTimestamp();
66
+			$timestamp = \DateTime::createFromFormat( 'd/m/Y', $groups[ 'sysop' ] )->getTimestamp();
69 67
 		}
70 68
 		return $timestamp;
71 69
 	}
@@ -75,7 +73,7 @@  discard block
 block discarded – undo
75 73
 	 *
76 74
 	 * @return array[]
77 75
 	 */
78
-	public function getAdminsList() : array {
76
+	public function getAdminsList () : array {
79 77
 		return json_decode( $this->getContent(), true );
80 78
 	}
81 79
 }
Please login to merge, or discard this patch.
includes/Wiki/Page/Page.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme\Wiki\Page;
4 4
 
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
 	 * @param string $title
20 20
 	 * @param Wiki $wiki For the site where the page lives
21 21
 	 */
22
-	public function __construct( string $title, Wiki $wiki ) {
22
+	public function __construct ( string $title, Wiki $wiki ) {
23 23
 		parent::__construct( $wiki );
24 24
 		$this->title = $title;
25 25
 	}
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
 	/**
28 28
 	 * @return string
29 29
 	 */
30
-	public function getTitle() : string {
30
+	public function getTitle () : string {
31 31
 		return $this->title;
32 32
 	}
33 33
 
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 	 * @param int|null $section A section number to retrieve the content of that section
38 38
 	 * @return string
39 39
 	 */
40
-	public function getContent( int $section = null ) : string {
40
+	public function getContent ( int $section = null ) : string {
41 41
 		if ( $this->content === null ) {
42 42
 			$this->content = $this->wiki->getPageContent( $this->title, $section );
43 43
 		}
@@ -50,18 +50,18 @@  discard block
 block discarded – undo
50 50
 	 * @param array $params
51 51
 	 * @throws \LogicException
52 52
 	 */
53
-	public function edit( array $params ) {
53
+	public function edit ( array $params ) {
54 54
 		$params = [
55 55
 			'title' => $this->getTitle()
56 56
 		] + $params;
57 57
 
58 58
 		$this->wiki->editPage( $params );
59
-		if ( isset( $params['text'] ) ) {
60
-			$this->content = $params['text'];
61
-		} elseif ( isset( $params['appendtext'] ) ) {
62
-			$this->content .= $params['appendtext'];
63
-		} elseif ( isset( $params['prependtext'] ) ) {
64
-			$this->content = $params['prependtext'] . $this->content;
59
+		if ( isset( $params[ 'text' ] ) ) {
60
+			$this->content = $params[ 'text' ];
61
+		} elseif ( isset( $params[ 'appendtext' ] ) ) {
62
+			$this->content .= $params[ 'appendtext' ];
63
+		} elseif ( isset( $params[ 'prependtext' ] ) ) {
64
+			$this->content = $params[ 'prependtext' ] . $this->content;
65 65
 		} else {
66 66
 			throw new \LogicException(
67 67
 				'Unrecognized text param for edit. Params: ' . var_export( $params, true )
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 	 *
75 75
 	 * @return bool
76 76
 	 */
77
-	public function exists() : bool {
77
+	public function exists () : bool {
78 78
 		$res = RequestBase::newFromParams( [
79 79
 			'action' => 'query',
80 80
 			'titles' => $this->getTitle()
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
 	 * @param string $regex
90 90
 	 * @return bool
91 91
 	 */
92
-	public function matches( string $regex ) : bool {
92
+	public function matches ( string $regex ) : bool {
93 93
 		return (bool)preg_match( $regex, $this->getContent() );
94 94
 	}
95 95
 
@@ -101,8 +101,8 @@  discard block
 block discarded – undo
101 101
 	 * @return string[]
102 102
 	 * @throws \Exception
103 103
 	 */
104
-	public function getMatch( string $regex ) : array {
105
-		$ret = [];
104
+	public function getMatch ( string $regex ) : array {
105
+		$ret = [ ];
106 106
 		if ( preg_match( $regex, $this->getContent(), $ret ) === 0 ) {
107 107
 			throw new \Exception( "The content of $this does not match the given regex $regex" );
108 108
 		}
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 	 *
115 115
 	 * @inheritDoc
116 116
 	 */
117
-	public function getRegex() : string {
117
+	public function getRegex () : string {
118 118
 		return str_replace( ' ', '[ _]', preg_quote( $this->title ) );
119 119
 	}
120 120
 
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
 	 *
124 124
 	 * @return string
125 125
 	 */
126
-	public function __toString() {
126
+	public function __toString () {
127 127
 		return $this->getTitle();
128 128
 	}
129 129
 }
Please login to merge, or discard this patch.
includes/Wiki/Page/PageRiconferma.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -1,4 +1,4 @@  discard block
 block discarded – undo
1
-<?php declare( strict_types=1 );
1
+<?php declare(strict_types=1);
2 2
 
3 3
 namespace BotRiconferme\Wiki\Page;
4 4
 
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 	private $supportSection;
14 14
 	private $opposeSection;
15 15
 	/** @var array Counts of votes for each section */
16
-	private $sectionCounts = [];
16
+	private $sectionCounts = [ ];
17 17
 
18 18
 	// Possible outcomes of a vote
19 19
 	public const OUTCOME_OK = 0;
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
 	 * because they can vary depending on whether the page is a vote, which is relatively
33 33
 	 * expensive to know since it requires parsing the content of the page.
34 34
 	 */
35
-	private function defineSections() {
35
+	private function defineSections () {
36 36
 		$this->supportSection = $this->isVote() ? 3 : 0;
37 37
 		$this->opposeSection = $this->isVote() ? 4 : 3;
38 38
 	}
@@ -42,8 +42,8 @@  discard block
 block discarded – undo
42 42
 	 *
43 43
 	 * @return User
44 44
 	 */
45
-	public function getUser() : User {
46
-		$name = explode( '/', $this->title )[2];
45
+	public function getUser () : User {
46
+		$name = explode( '/', $this->title )[ 2 ];
47 47
 		return new User( $name, $this->wiki );
48 48
 	}
49 49
 
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 	 *
53 53
 	 * @return int
54 54
 	 */
55
-	public function getNum() : int {
55
+	public function getNum () : int {
56 56
 		$bits = explode( '/', $this->getTitle() );
57 57
 		return intval( end( $bits ) );
58 58
 	}
@@ -62,8 +62,8 @@  discard block
 block discarded – undo
62 62
 	 *
63 63
 	 * @return string
64 64
 	 */
65
-	public function getUserNum() : string {
66
-		return explode( '/', $this->getTitle(), 3 )[2];
65
+	public function getUserNum () : string {
66
+		return explode( '/', $this->getTitle(), 3 )[ 2 ];
67 67
 	}
68 68
 
69 69
 	/**
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 	 *
72 72
 	 * @return string
73 73
 	 */
74
-	public function getBaseTitle() : string {
74
+	public function getBaseTitle () : string {
75 75
 		return substr( $this->getTitle(), 0, strrpos( $this->getTitle(), '/' ) );
76 76
 	}
77 77
 
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
 	 *
81 81
 	 * @return int
82 82
 	 */
83
-	public function getOpposingCount() : int {
83
+	public function getOpposingCount () : int {
84 84
 		$this->defineSections();
85 85
 		return $this->getCountForSection( $this->opposeSection );
86 86
 	}
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 	 * @return int
92 92
 	 * @throws \BadMethodCallException
93 93
 	 */
94
-	public function getSupportCount() : int {
94
+	public function getSupportCount () : int {
95 95
 		if ( !$this->isVote() ) {
96 96
 			throw new \BadMethodCallException( 'Cannot get support for a non-vote page.' );
97 97
 		}
@@ -105,13 +105,13 @@  discard block
 block discarded – undo
105 105
 	 * @param int $secNum
106 106
 	 * @return int
107 107
 	 */
108
-	protected function getCountForSection( int $secNum ) : int {
108
+	protected function getCountForSection ( int $secNum ) : int {
109 109
 		if ( !isset( $this->sectionCounts[ $secNum ] ) ) {
110 110
 			$content = $this->wiki->getPageContent( $this->title, $secNum );
111 111
 			// Let's hope that this is good enough...
112
-			$this->sectionCounts[$secNum] = preg_match_all( "/^\# *(?![# *:]|\.\.\.$)/m", $content );
112
+			$this->sectionCounts[ $secNum ] = preg_match_all( "/^\# *(?![# *:]|\.\.\.$)/m", $content );
113 113
 		}
114
-		return $this->sectionCounts[$secNum];
114
+		return $this->sectionCounts[ $secNum ];
115 115
 	}
116 116
 
117 117
 	/**
@@ -119,9 +119,9 @@  discard block
 block discarded – undo
119 119
 	 *
120 120
 	 * @return int
121 121
 	 */
122
-	protected function getQuorum() : int {
122
+	protected function getQuorum () : int {
123 123
 		$reg = "!soddisfare il \[\[[^|\]]+\|quorum]] di '''(\d+) voti'''!";
124
-		return intval( $this->getMatch( $reg )[1] );
124
+		return intval( $this->getMatch( $reg )[ 1 ] );
125 125
 	}
126 126
 
127 127
 	/**
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
 	 *
130 130
 	 * @return bool
131 131
 	 */
132
-	public function hasOpposition() : bool {
132
+	public function hasOpposition () : bool {
133 133
 		return $this->getOpposingCount() >= self::REQUIRED_OPPOSE;
134 134
 	}
135 135
 
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 	 *
139 139
 	 * @return int One of the OUTCOME_* constants
140 140
 	 */
141
-	public function getOutcome() : int {
141
+	public function getOutcome () : int {
142 142
 		if ( !$this->isVote() ) {
143 143
 			return self::OUTCOME_OK;
144 144
 		}
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 	 * @throws \BadMethodCallException
162 162
 	 * @throws \LogicException
163 163
 	 */
164
-	public function getOutcomeText() : string {
164
+	public function getOutcomeText () : string {
165 165
 		if ( !$this->isVote() ) {
166 166
 			throw new \BadMethodCallException( 'No need for an outcome text.' );
167 167
 		}
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
 	 *
196 196
 	 * @return bool
197 197
 	 */
198
-	public function isVote() : bool {
198
+	public function isVote () : bool {
199 199
 		$sectionReg = '/<!-- SEZIONE DA UTILIZZARE PER/';
200 200
 		return !$this->matches( $sectionReg );
201 201
 	}
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
 	 *
206 206
 	 * @return int
207 207
 	 */
208
-	public function getCreationTimestamp() : int {
208
+	public function getCreationTimestamp () : int {
209 209
 		return $this->wiki->getPageCreationTS( $this->title );
210 210
 	}
211 211
 
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
 	 *
215 215
 	 * @return int
216 216
 	 */
217
-	public function getEndTimestamp() : int {
217
+	public function getEndTimestamp () : int {
218 218
 		if ( $this->isVote() ) {
219 219
 			$reg = "!La votazione ha inizio il.+ alle ore ([\d:]+) e ha termine il (.+) alla stessa ora!";
220 220
 			list( , $hours, $day ) = $this->getMatch( $reg );
Please login to merge, or discard this patch.