Completed
Pull Request — master (#43)
by Michael
12:40 queued 02:34
created
src/Views/Stats/StatsJsonView.php 2 patches
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -102,8 +102,7 @@
 block discarded – undo
102 102
 		if ($this->recent)
103 103
 		{
104 104
 			$items = $this->repository->getRecentlyUpdatedItems();
105
-		}
106
-		else
105
+		} else
107 106
 		{
108 107
 			$items = $this->repository->getItems($this->source);
109 108
 		}
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
 						case 'cms_php_version':
152 152
 							if ((isset($item['cms_version']) && $item['cms_version'] !== null) && (isset($item['php_version']) && $item['php_version'] !== null))
153 153
 							{
154
-								$index = $item['cms_version'] . ' - ' . $item['php_version'];
154
+								$index = $item['cms_version'].' - '.$item['php_version'];
155 155
 								$cms_php_version[$index] = $item['count'];
156 156
 
157 157
 								$this->totalItems += $item['count'];
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 						case 'db_type_version':
163 163
 							if ((isset($item['db_type']) && $item['db_type'] !== null) && (isset($item['db_version']) && $item['db_version'] !== null))
164 164
 							{
165
-								$index = $item['db_type'] . ' - ' . $item['db_version'];
165
+								$index = $item['db_type'].' - '.$item['db_version'];
166 166
 								$db_type_version[$index] = $item['count'];
167 167
 
168 168
 								$this->totalItems += $item['count'];
@@ -278,12 +278,12 @@  discard block
 block discarded – undo
278 278
 					break;
279 279
 
280 280
 				case 'cms_php_version':
281
-					$index = $item['cms_version'] . ' - ' . $item['php_version'];
281
+					$index = $item['cms_version'].' - '.$item['php_version'];
282 282
 					$data[$this->source][$index] = $item['count'];
283 283
 					break;
284 284
 
285 285
 				case 'db_type_version':
286
-					$index = $item['db_type'] . ' - ' . $item['db_version'];
286
+					$index = $item['db_type'].' - '.$item['db_version'];
287 287
 					$data[$this->source][$index] = $item['count'];
288 288
 					break;
289 289
 
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
326 326
 					foreach ($dataGroup as $row)
327 327
 					{
328 328
 						$exploded = explode('.', $row['name']);
329
-						$version  = $exploded[0] . '.' . ($exploded[1] ?? '0');
329
+						$version  = $exploded[0].'.'.($exploded[1] ?? '0');
330 330
 
331 331
 						// If the container does not exist, add it
332 332
 						if (!isset($counts[$version]))
Please login to merge, or discard this patch.
src/EventListener/AnalyticsSubscriber.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -82,8 +82,7 @@
 block discarded – undo
82 82
 		try
83 83
 		{
84 84
 			$this->analytics->sendPageview();
85
-		}
86
-		catch (\Exception $e)
85
+		} catch (\Exception $e)
87 86
 		{
88 87
 			// Log the error for reference
89 88
 			$this->logger->error(
Please login to merge, or discard this patch.
src/Database/Migrations.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -196,7 +196,7 @@
 block discarded – undo
196 196
 	 */
197 197
 	private function doMigration(string $version): void
198 198
 	{
199
-		$sqlFile = $version . '.sql';
199
+		$sqlFile = $version.'.sql';
200 200
 
201 201
 		if (!$this->filesystem->has($sqlFile))
202 202
 		{
Please login to merge, or discard this patch.
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -64,8 +64,7 @@  discard block
 block discarded – undo
64 64
 					->select('version')
65 65
 					->from('#__migrations')
66 66
 			)->loadColumn();
67
-		}
68
-		catch (ExecutionFailureException | PrepareStatementFailureException $exception)
67
+		} catch (ExecutionFailureException | PrepareStatementFailureException $exception)
69 68
 		{
70 69
 			// On PDO we're checking "42S02, 1146, Table 'XXX.#__migrations' doesn't exist"
71 70
 			if (strpos($exception->getMessage(), "migrations' doesn't exist") === false)
@@ -132,8 +131,7 @@  discard block
 block discarded – undo
132 131
 					->select('version')
133 132
 					->from('#__migrations')
134 133
 			)->loadColumn();
135
-		}
136
-		catch (ExecutionFailureException | PrepareStatementFailureException $exception)
134
+		} catch (ExecutionFailureException | PrepareStatementFailureException $exception)
137 135
 		{
138 136
 			// If the table does not exist, we can still try to run migrations
139 137
 			if (strpos($exception->getMessage(), "migrations' doesn't exist") === false)
Please login to merge, or discard this patch.
src/Kernel.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -121,11 +121,11 @@
 block discarded – undo
121 121
 	private function loadConfiguration(): Registry
122 122
 	{
123 123
 		$registry = new Registry;
124
-		$registry->loadFile(APPROOT . '/etc/config.dist.json');
124
+		$registry->loadFile(APPROOT.'/etc/config.dist.json');
125 125
 
126
-		if (file_exists(APPROOT . '/etc/config.json'))
126
+		if (file_exists(APPROOT.'/etc/config.json'))
127 127
 		{
128
-			$registry->loadFile(APPROOT . '/etc/config.json');
128
+			$registry->loadFile(APPROOT.'/etc/config.json');
129 129
 		}
130 130
 
131 131
 		return $registry;
Please login to merge, or discard this patch.
src/Repositories/StatisticsRepository.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 			return $this->db->setQuery(
65 65
 				$this->db->getQuery(true)
66 66
 					->select('*')
67
-					->from($this->db->quoteName('#__jstats_counter_' . $column))
67
+					->from($this->db->quoteName('#__jstats_counter_'.$column))
68 68
 			)->loadAssocList();
69 69
 		}
70 70
 
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 			$return[$column] = $this->db->setQuery(
76 76
 				$this->db->getQuery(true)
77 77
 					->select('*')
78
-					->from($this->db->quoteName('#__jstats_counter_' . $column))
78
+					->from($this->db->quoteName('#__jstats_counter_'.$column))
79 79
 			)->loadAssocList();
80 80
 		}
81 81
 
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 			$return[$column] = $this->db->setQuery(
99 99
 				$this->db->getQuery(true)
100 100
 					->select($column)
101
-					->select('COUNT(' . $column . ') AS count')
101
+					->select('COUNT('.$column.') AS count')
102 102
 					->from($this->db->quoteName('#__jstats'))
103 103
 					->where('modified BETWEEN DATE_SUB(NOW(), INTERVAL 90 DAY) AND NOW()')
104 104
 					->group($column)
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -132,8 +132,7 @@
 block discarded – undo
132 132
 		if ($recordExists)
133 133
 		{
134 134
 			$this->db->updateObject('#__jstats', $data, ['unique_id']);
135
-		}
136
-		else
135
+		} else
137 136
 		{
138 137
 			$this->db->insertObject('#__jstats', $data, ['unique_id']);
139 138
 		}
Please login to merge, or discard this patch.
src/Providers/MonologServiceProvider.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
 
90 90
 		$level = strtoupper($config->get('log.application', $config->get('log.level', 'error')));
91 91
 
92
-		return new StreamHandler(APPROOT . '/logs/stats.log', \constant('\\Monolog\\Logger::' . $level));
92
+		return new StreamHandler(APPROOT.'/logs/stats.log', \constant('\\Monolog\\Logger::'.$level));
93 93
 	}
94 94
 
95 95
 	/**
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 
107 107
 		$level = strtoupper($config->get('log.database', $config->get('log.level', 'error')));
108 108
 
109
-		return new StreamHandler(APPROOT . '/logs/stats.log', \constant('\\Monolog\\Logger::' . $level));
109
+		return new StreamHandler(APPROOT.'/logs/stats.log', \constant('\\Monolog\\Logger::'.$level));
110 110
 	}
111 111
 
112 112
 	/**
Please login to merge, or discard this patch.
src/Providers/WebApplicationServiceProvider.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -135,7 +135,7 @@
 block discarded – undo
135 135
 			'/:source',
136 136
 			DisplayStatisticsController::class,
137 137
 			[
138
-				'source' => '(' . implode('|', StatisticsRepository::ALLOWED_SOURCES) . ')',
138
+				'source' => '('.implode('|', StatisticsRepository::ALLOWED_SOURCES).')',
139 139
 			]
140 140
 		);
141 141
 
Please login to merge, or discard this patch.
src/Providers/FlysystemServiceProvider.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
 	 */
42 42
 	public function getMigrationsFilesystemService(Container $container): Filesystem
43 43
 	{
44
-		return new Filesystem(new Local(APPROOT . '/etc/migrations'));
44
+		return new Filesystem(new Local(APPROOT.'/etc/migrations'));
45 45
 	}
46 46
 
47 47
 	/**
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 	 */
54 54
 	public function getSnapshotFilesystemService(Container $container): Filesystem
55 55
 	{
56
-		return new Filesystem(new Local(APPROOT . '/snapshots'));
56
+		return new Filesystem(new Local(APPROOT.'/snapshots'));
57 57
 	}
58 58
 
59 59
 	/**
@@ -65,6 +65,6 @@  discard block
 block discarded – undo
65 65
 	 */
66 66
 	public function getVersionsFilesystemService(Container $container): Filesystem
67 67
 	{
68
-		return new Filesystem(new Local(APPROOT . '/versions'));
68
+		return new Filesystem(new Local(APPROOT.'/versions'));
69 69
 	}
70 70
 }
Please login to merge, or discard this patch.
src/Controllers/SubmitDataController.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -188,8 +188,7 @@  discard block
 block discarded – undo
188 188
 		try
189 189
 		{
190 190
 			$validVersions = json_decode($this->filesystem->read('joomla.json'), true);
191
-		}
192
-		catch (FileNotFoundException $exception)
191
+		} catch (FileNotFoundException $exception)
193 192
 		{
194 193
 			throw new \RuntimeException('Missing Joomla! release listing', 500, $exception);
195 194
 		}
@@ -248,8 +247,7 @@  discard block
 block discarded – undo
248 247
 		try
249 248
 		{
250 249
 			$validVersions = json_decode($this->filesystem->read('php.json'), true);
251
-		}
252
-		catch (FileNotFoundException $exception)
250
+		} catch (FileNotFoundException $exception)
253 251
 		{
254 252
 			throw new \RuntimeException('Missing PHP release listing', 500, $exception);
255 253
 		}
Please login to merge, or discard this patch.