@@ -102,8 +102,7 @@ |
||
| 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 | } |
@@ -151,7 +151,7 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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])) |
@@ -82,8 +82,7 @@ |
||
| 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( |
@@ -196,7 +196,7 @@ |
||
| 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 | { |
@@ -64,8 +64,7 @@ discard block |
||
| 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 |
||
| 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) |
@@ -121,11 +121,11 @@ |
||
| 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; |
@@ -132,8 +132,7 @@ |
||
| 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 | } |
@@ -64,7 +64,7 @@ discard block |
||
| 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 |
||
| 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 | |
@@ -100,7 +100,7 @@ discard block |
||
| 100 | 100 | $return[$column] = $this->db->setQuery( |
| 101 | 101 | $this->db->getQuery(true) |
| 102 | 102 | ->select($column) |
| 103 | - ->select('COUNT(' . $column . ') AS count') |
|
| 103 | + ->select('COUNT('.$column.') AS count') |
|
| 104 | 104 | ->from($this->db->quoteName('#__jstats')) |
| 105 | 105 | ->where('modified BETWEEN DATE_SUB(NOW(), INTERVAL 90 DAY) AND NOW()') |
| 106 | 106 | ->group($column) |
@@ -112,11 +112,11 @@ discard block |
||
| 112 | 112 | { |
| 113 | 113 | $return['cms_php_version'] = $this->db->setQuery( |
| 114 | 114 | $this->db->getQuery(true) |
| 115 | - ->select('CONCAT(' . $this->db->quoteName('cms_version') . ', ' . $this->db->quote(' - ') . ', ' . $this->db->quoteName('php_version') . ') AS cms_php_version') |
|
| 115 | + ->select('CONCAT('.$this->db->quoteName('cms_version').', '.$this->db->quote(' - ').', '.$this->db->quoteName('php_version').') AS cms_php_version') |
|
| 116 | 116 | ->select('COUNT(*) AS count') |
| 117 | 117 | ->from($this->db->quoteName('#__jstats')) |
| 118 | 118 | ->where('modified BETWEEN DATE_SUB(NOW(), INTERVAL 90 DAY) AND NOW()') |
| 119 | - ->group('CONCAT(' . $this->db->quoteName('cms_version') . ', ' . $this->db->quote(' - ') . ', ' . $this->db->quoteName('php_version') . ')') |
|
| 119 | + ->group('CONCAT('.$this->db->quoteName('cms_version').', '.$this->db->quote(' - ').', '.$this->db->quoteName('php_version').')') |
|
| 120 | 120 | )->loadAssocList(); |
| 121 | 121 | continue; |
| 122 | 122 | } |
@@ -125,11 +125,11 @@ discard block |
||
| 125 | 125 | { |
| 126 | 126 | $return['db_type_version'] = $this->db->setQuery( |
| 127 | 127 | $this->db->getQuery(true) |
| 128 | - ->select('CONCAT(' . $this->db->quoteName('db_type') . ', ' . $this->db->quote(' - ') . ', ' . $this->db->quoteName('db_version') . ') AS db_type_version') |
|
| 128 | + ->select('CONCAT('.$this->db->quoteName('db_type').', '.$this->db->quote(' - ').', '.$this->db->quoteName('db_version').') AS db_type_version') |
|
| 129 | 129 | ->select('COUNT(*) AS count') |
| 130 | 130 | ->from($this->db->quoteName('#__jstats')) |
| 131 | 131 | ->where('modified BETWEEN DATE_SUB(NOW(), INTERVAL 90 DAY) AND NOW()') |
| 132 | - ->group('CONCAT(' . $this->db->quoteName('db_type') . ', ' . $this->db->quote(' - ') . ', ' . $this->db->quoteName('db_version') . ')') |
|
| 132 | + ->group('CONCAT('.$this->db->quoteName('db_type').', '.$this->db->quote(' - ').', '.$this->db->quoteName('db_version').')') |
|
| 133 | 133 | )->loadAssocList(); |
| 134 | 134 | continue; |
| 135 | 135 | } |
@@ -89,7 +89,7 @@ discard block |
||
| 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 |
||
| 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 | /** |
@@ -135,7 +135,7 @@ |
||
| 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 | |
@@ -41,7 +41,7 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 | } |
@@ -188,8 +188,7 @@ discard block |
||
| 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 |
||
| 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 | } |