Completed
Pull Request — master (#158)
by Aleš
09:05
created
src/Diagnostics/Panel.php 2 patches
Doc Comments   +2 added lines patch added patch discarded remove patch
@@ -338,6 +338,8 @@
 block discarded – undo
338 338
 	/**
339 339
 	 * Returns link to editor.
340 340
 	 *
341
+	 * @param integer $line
342
+	 * @param string $text
341 343
 	 * @return \Nette\Utils\Html|string
342 344
 	 */
343 345
 	private static function editorLink($file, $line, $text = NULL)
Please login to merge, or discard this patch.
Spacing   +26 added lines, -27 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 	public function getTab(): ?string
84 84
 	{
85 85
 		return '<span title="Translation"><img width="16px" height="16px" src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAACAAAAAgCAYAAABzenr0AAAAAXNSR0IArs4c6QAAAAZiS0dEAP8A/wD/oL2nkwAAAAlwSFlzAAALEwAACxMBAJqcGAAAAAd0SU1FB90DExAuL9uIsPUAAAQ2SURBVFjD7Ze9bxxVFMV/583skjgOIXEaKCNFAgmQ+ExS8qEUCEHH30BBScG/QM1fEmgoAAkoAigSKKFASElHESEFbGcdez7uoXhvZteW7ewmQjRMsTs7s/Pufeece+4d+P/4jw8ddvHq1avv2b5mm02Lr9Y7/gpQBCFDD5YhgsAoRLhHMtHnRYNAAb3yt92//9y3v3x+MFY6LIGIuGYbKedngzAGsPK5Kb8BGRAu9xiuL+zQ6NphsQ5NYB7YhA1jQCObyOHG/3sMlpM0hmGNcmXh7w9PYL5wfsphbCMPaOR7BipEJahV0FHZvZ2D+1i2qY+gYERCZTGZzLdFkLNIht/v72AHGM5Pa85MqowUc4p81PaPo2CgIa9RxOZ8jowDZm0WXo2oBdttN8ihIABC5dPLI7BIQQA5prEGIeYdbnVtSSpD3WCaCCqpMB8jXSshEBFEROZ9TCTvepC/be63HWA2ppNRoJttP1flvkp5BArmNBQl5zSIMJtNRxJEwHqVOF1XGLHVdSQp+0CpAi1UytJVYHteBQ7cm7AgIKnAH7CWhBDrVRqfud91UERLFNPyihQMwaOUIAI5I9D0PU2Xz9frCjs4WVVUMhJstR2SwaUWgvx7VQqyG2b4IoIoO9xsWipllJ6sK1JB7Exd4whmbUcYlCVcRKrVq0DSqCIVRlPZ4SCqO7MHC0Ey1Epiu+k5XafMvbRaFXCgdBxBOIgw202/TyNBtmuHibJRYba6LjuAsgjjCA3UxwWPCEwanRCJ7bYd/eHspFootdwhdvqeJsyegyaCSVK271US2E9BNp8otbTd9STlB89NJvRFmJREJ4K7e0El2Op6NiY1x7GQlpoawihgq+lIZF2s1xWdu9GAcO6Yp6rcE4OcwGDGR7nhUglYRjL32pbOQdMH61VCzgiFc8eUs+2cTIkuoI2MmH20D9TLzEzuTafgwtp0dNXO5FlhsOvSh23zzBPTBfDm4ny0BBBWAInWpSmVmSDzqrEko0TpBwct9W+xGgKL41hWoMeJB/LQsa/JlOYkVDrmAHlpwnF0M6of2gdIC7vUvDUNk1HMbwVRbmo+jA3tOlaYByTYbRJdiG1E3UFFg2IvLxqm67qR1yg71kLflDXvwhHhsxtbSycw2xWffXiPP+/NmFTiGyX21t9m99RbWDCZ1Fy5dAlJeXyz6aOfz48Mzli6aPTp8rMXz/585UVeun7z4WW418Kll0/w9Pk1/t5JTHdh2otpElMl1PX8eP06fdus9ObhQ2g4VBrVxpvvdl/Ovmgacev2Dnf+2GXnxi6zGw/KMJrNJUXPS6fXmCaNw+d8GNV8KHXgp87xyg83tVQCALPvLlvCVUK/3tnh9t3mkw8+/u3Tx3kNu/H6C7z6063lnLBU4dd98NHzF9Z47eKpE4/7Hngw+LHHzveX3zmAyBv/xsvpP+li/lm3bxkuAAAAAElFTkSuQmCC" />'
86
-			. $this->translator->getLocale() . ($this->untranslated ? ' <b>(' . count(array_unique($this->untranslated, SORT_REGULAR)) . ' errors)</b>' : '')
86
+			. $this->translator->getLocale().($this->untranslated ? ' <b>('.count(array_unique($this->untranslated, SORT_REGULAR)).' errors)</b>' : '')
87 87
 			. '</span>';
88 88
 	}
89 89
 
@@ -114,17 +114,17 @@  discard block
 block discarded – undo
114 114
 
115 115
 				/** @var \Nette\Application\Request[] $snapshot */
116 116
 				$params = $snapshot['request']->getParameters();
117
-				$s .= '<tr><th width="10px">&nbsp;</th>' .
118
-					'<th>' . $h($snapshot['request']->getPresenterName() . (isset($params['action']) ? ':' . $params['action'] : '')) . '</th>' .
119
-					'<th>' . $h($snapshot['locale']) . '</th></tr>';
117
+				$s .= '<tr><th width="10px">&nbsp;</th>'.
118
+					'<th>'.$h($snapshot['request']->getPresenterName().(isset($params['action']) ? ':'.$params['action'] : '')).'</th>'.
119
+					'<th>'.$h($snapshot['locale']).'</th></tr>';
120 120
 
121 121
 				$l = 1;
122 122
 				/** @var mixed[][] $snapshot */
123 123
 				foreach ($snapshot['resolvers'] as $name => $resolvedLocale) {
124
-					$s .= '<tr><td>' . ($l++) . '.</td><td>' . $h($name) . '</td><td>' . $h($resolvedLocale) . '</td></tr>';
124
+					$s .= '<tr><td>'.($l++).'.</td><td>'.$h($name).'</td><td>'.$h($resolvedLocale).'</td></tr>';
125 125
 				}
126 126
 
127
-				$panel[] = '<table style="width:100%">' . $s . '</table>';
127
+				$panel[] = '<table style="width:100%">'.$s.'</table>';
128 128
 			}
129 129
 		}
130 130
 
@@ -142,14 +142,13 @@  discard block
 block discarded – undo
142 142
 			}
143 143
 
144 144
 			$panel[] = '<h2>Ignored resources</h2>';
145
-			$panel[] = '<p>Whitelist config: ' . implode(', ', array_map($h, $this->localeWhitelist)) . '</p>';
145
+			$panel[] = '<p>Whitelist config: '.implode(', ', array_map($h, $this->localeWhitelist)).'</p>';
146 146
 			$panel[] = $this->renderResources($this->ignoredResources);
147 147
 		}
148 148
 
149
-		return empty($panel) ? '' :
150
-			'<h1>Missing translations: ' . count(array_unique($this->untranslated, SORT_REGULAR)) .
151
-			', Resources: ' . count(Arrays::flatten($this->resources)) . '</h1>' .
152
-			'<div class="nette-inner tracy-inner kdyby-TranslationPanel" style="min-width:500px">' . implode($panel) . '</div>' .
149
+		return empty($panel) ? '' : '<h1>Missing translations: '.count(array_unique($this->untranslated, SORT_REGULAR)).
150
+			', Resources: '.count(Arrays::flatten($this->resources)).'</h1>'.
151
+			'<div class="nette-inner tracy-inner kdyby-TranslationPanel" style="min-width:500px">'.implode($panel).'</div>'.
153 152
 			'<style>
154 153
 				#nette-debug .kdyby-TranslationPanel h2,
155 154
 				#tracy-debug .kdyby-TranslationPanel h2 {font-size: 23px;}
@@ -167,19 +166,19 @@  discard block
 block discarded – undo
167 166
 			$s .= '<tr><td>';
168 167
 
169 168
 			if ($message instanceof \Exception || $message instanceof \Throwable) {
170
-				$s .= '<span style="color:red">' . $h($message->getMessage()) . '</span>';
169
+				$s .= '<span style="color:red">'.$h($message->getMessage()).'</span>';
171 170
 
172 171
 			} elseif ($message instanceof Html) {
173
-				$s .= '<span style="color:red">Nette\Utils\Html(' . $h((string) $message) . ')</span>';
172
+				$s .= '<span style="color:red">Nette\Utils\Html('.$h((string) $message).')</span>';
174 173
 
175 174
 			} else {
176 175
 				$s .= $h($message);
177 176
 			}
178 177
 
179
-			$s .= '</td><td>' . $h($untranslated['domain']) . '</td></tr>';
178
+			$s .= '</td><td>'.$h($untranslated['domain']).'</td></tr>';
180 179
 		}
181 180
 
182
-		return '<table style="width:100%"><tr><th>Untranslated message</th><th>Translation domain</th></tr>' . $s . '</table>';
181
+		return '<table style="width:100%"><tr><th>Untranslated message</th><th>Translation domain</th></tr>'.$s.'</table>';
183 182
 	}
184 183
 
185 184
 	private function renderResources($resourcesMap)
@@ -191,25 +190,25 @@  discard block
 block discarded – undo
191 190
 		foreach ($resourcesMap as $locale => $resources) {
192 191
 			foreach ($resources as $resourcePath => $domain) {
193 192
 				$s .= '<tr>';
194
-				$s .= '<td>' . $h($locale) . '</td>';
195
-				$s .= '<td>' . $h($domain) . '</td>';
193
+				$s .= '<td>'.$h($locale).'</td>';
194
+				$s .= '<td>'.$h($domain).'</td>';
196 195
 
197
-				$relativePath = str_replace(rtrim($this->rootDir, '/') . '/', '', $resourcePath);
196
+				$relativePath = str_replace(rtrim($this->rootDir, '/').'/', '', $resourcePath);
198 197
 				if (Strings::startsWith($relativePath, 'vendor/')) {
199 198
 					$parts = explode('/', $relativePath, 4);
200 199
 					$left = array_pop($parts);
201
-					$relativePath = $h(implode('/', $parts) . '/.../') . '<b>' . $h(basename($left)) . '</b>';
200
+					$relativePath = $h(implode('/', $parts).'/.../').'<b>'.$h(basename($left)).'</b>';
202 201
 
203 202
 				} else {
204
-					$relativePath = $h(dirname($relativePath)) . '/<b>' . $h(basename($relativePath)) . '</b>';
203
+					$relativePath = $h(dirname($relativePath)).'/<b>'.$h(basename($relativePath)).'</b>';
205 204
 				}
206 205
 
207
-				$s .= '<td>' . self::editorLink($resourcePath, 1, $relativePath) . '</td>';
206
+				$s .= '<td>'.self::editorLink($resourcePath, 1, $relativePath).'</td>';
208 207
 				$s .= '</tr>';
209 208
 			}
210 209
 		}
211 210
 
212
-		return '<table style="width:100%"><tr><th>Locale</th><th>Domain</th><th>Resource filename</th></tr>' . $s . '</table>';
211
+		return '<table style="width:100%"><tr><th>Locale</th><th>Domain</th><th>Resource filename</th></tr>'.$s.'</table>';
213 212
 	}
214 213
 
215 214
 	/**
@@ -245,7 +244,7 @@  discard block
 block discarded – undo
245 244
 	public function addResource($format, $resource, $locale, $domain)
246 245
 	{
247 246
 		if (is_array($resource)) {
248
-			$resource = 'array ' . md5(serialize($resource));
247
+			$resource = 'array '.md5(serialize($resource));
249 248
 		}
250 249
 
251 250
 		$this->resources[$locale][$resource] = $domain;
@@ -265,7 +264,7 @@  discard block
 block discarded – undo
265 264
 	public function addIgnoredResource($format, $resource, $locale, $domain)
266 265
 	{
267 266
 		if (is_array($resource)) {
268
-			$resource = 'array ' . md5(serialize($resource));
267
+			$resource = 'array '.md5(serialize($resource));
269 268
 		}
270 269
 
271 270
 		$this->ignoredResources[$locale][$resource] = $domain;
@@ -323,14 +322,14 @@  discard block
 block discarded – undo
323 322
 			return NULL;
324 323
 		}
325 324
 
326
-		$method = YamlFileLoader::class . '::load';
325
+		$method = YamlFileLoader::class.'::load';
327 326
 		$call = Helpers::findTrace($e->getPrevious()->getTrace(), $method);
328 327
 		if ($call !== NULL) {
329 328
 			return [
330 329
 				'tab' => 'YAML dictionary',
331
-				'panel' => '<p><b>File:</b> ' . self::editorLink($call['args'][0], $previous->getParsedLine()) . '</p>'
330
+				'panel' => '<p><b>File:</b> '.self::editorLink($call['args'][0], $previous->getParsedLine()).'</p>'
332 331
 					. ($previous->getParsedLine() ? BlueScreen::highlightFile($call['args'][0], $previous->getParsedLine()) : '')
333
-					. '<p>' . $previous->getMessage() . ' </p>',
332
+					. '<p>'.$previous->getMessage().' </p>',
334 333
 			];
335 334
 		}
336 335
 	}
Please login to merge, or discard this patch.
src/Translator.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -121,7 +121,7 @@
 block discarded – undo
121 121
 	 * Translates the given string.
122 122
 	 *
123 123
 	 * @param string|\Kdyby\Translation\Phrase|mixed $message The message id
124
-	 * @param string|array|NULL $parameters An array of parameters for the message
124
+	 * @param integer|null $parameters An array of parameters for the message
125 125
 	 * @throws \InvalidArgumentException
126 126
 	 * @return string|\Nette\Utils\IHtmlString|\Latte\Runtime\IHtmlString
127 127
 	 */
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
 
172 172
 		$tmp = [];
173 173
 		foreach ($params as $key => $val) {
174
-			$tmp['%' . trim($key, '%') . '%'] = $val;
174
+			$tmp['%'.trim($key, '%').'%'] = $val;
175 175
 		}
176 176
 		$params = $tmp;
177 177
 
@@ -477,7 +477,7 @@  discard block
 block discarded – undo
477 477
 	 */
478 478
 	public static function buildWhitelistRegexp($whitelist)
479 479
 	{
480
-		return ($whitelist !== NULL) ? '~^(' . implode('|', $whitelist) . ')~i' : NULL;
480
+		return ($whitelist !== NULL) ? '~^('.implode('|', $whitelist).')~i' : NULL;
481 481
 	}
482 482
 
483 483
 }
Please login to merge, or discard this patch.
src/DI/TranslationExtension.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 
139 139
 		$builder->addDefinition($this->prefix('helpers'))
140 140
 			->setClass(TemplateHelpers::class)
141
-			->setFactory($this->prefix('@default') . '::createTemplateHelpers');
141
+			->setFactory($this->prefix('@default').'::createTemplateHelpers');
142 142
 
143 143
 		$builder->addDefinition($this->prefix('fallbackResolver'))
144 144
 			->setClass(FallbackResolver::class);
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
 		$builder->addDefinition($this->prefix('loader'))
173 173
 			->setClass(TranslationLoader::class);
174 174
 
175
-		$loaders = $this->loadFromFile(__DIR__ . '/config/loaders.neon');
175
+		$loaders = $this->loadFromFile(__DIR__.'/config/loaders.neon');
176 176
 		$this->loadLoaders($loaders, $config['loaders'] ?: array_keys($loaders));
177 177
 
178 178
 		if ($this->isRegisteredConsoleExtension()) {
@@ -235,8 +235,8 @@  discard block
 block discarded – undo
235 235
 	{
236 236
 		$builder = $this->getContainerBuilder();
237 237
 
238
-		foreach ($this->loadFromFile(__DIR__ . '/config/dumpers.neon') as $format => $class) {
239
-			$builder->addDefinition($this->prefix('dumper.' . $format))
238
+		foreach ($this->loadFromFile(__DIR__.'/config/dumpers.neon') as $format => $class) {
239
+			$builder->addDefinition($this->prefix('dumper.'.$format))
240 240
 				->setClass($class)
241 241
 				->addTag(self::TAG_DUMPER, $format);
242 242
 		}
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
 			if (array_search($format, $allowed) === FALSE) {
251 251
 				continue;
252 252
 			}
253
-			$builder->addDefinition($this->prefix('loader.' . $format))
253
+			$builder->addDefinition($this->prefix('loader.'.$format))
254 254
 				->setClass($class)
255 255
 				->addTag(self::TAG_LOADER, $format);
256 256
 		}
@@ -260,8 +260,8 @@  discard block
 block discarded – undo
260 260
 	{
261 261
 		$builder = $this->getContainerBuilder();
262 262
 
263
-		foreach ($this->loadFromFile(__DIR__ . '/config/extractors.neon') as $format => $class) {
264
-			$builder->addDefinition($this->prefix('extractor.' . $format))
263
+		foreach ($this->loadFromFile(__DIR__.'/config/extractors.neon') as $format => $class) {
264
+			$builder->addDefinition($this->prefix('extractor.'.$format))
265 265
 				->setClass($class)
266 266
 				->addTag(self::TAG_EXTRACTOR, $format);
267 267
 		}
@@ -275,7 +275,7 @@  discard block
 block discarded – undo
275 275
 		$this->beforeCompileLogging($config);
276 276
 
277 277
 		/** @var Definition|FactoryDefinition $def */
278
-		$registerToLatte = function (Definition $def) {
278
+		$registerToLatte = function(Definition $def) {
279 279
 			$def->getResultDefinition()
280 280
 				->addSetup('?->onCompile[] = function($engine) { ?::install($engine->getCompiler()); }', ['@self', new PhpLiteral(TranslateMacros::class)]);
281 281
 
@@ -317,7 +317,7 @@  discard block
 block discarded – undo
317 317
 		foreach ($builder->findByTag(self::TAG_EXTRACTOR) as $extractorId => $meta) {
318 318
 			Validators::assert($meta, 'string:2..');
319 319
 
320
-			$extractor->addSetup('addExtractor', [$meta, '@' . $extractorId]);
320
+			$extractor->addSetup('addExtractor', [$meta, '@'.$extractorId]);
321 321
 
322 322
 			$builder->getDefinition($extractorId)->setAutowired(FALSE);
323 323
 		}
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
326 326
 		foreach ($builder->findByTag(self::TAG_DUMPER) as $dumperId => $meta) {
327 327
 			Validators::assert($meta, 'string:2..');
328 328
 
329
-			$writer->addSetup('addDumper', [$meta, '@' . $dumperId]);
329
+			$writer->addSetup('addDumper', [$meta, '@'.$dumperId]);
330 330
 
331 331
 			$builder->getDefinition($dumperId)->setAutowired(FALSE);
332 332
 		}
@@ -349,7 +349,7 @@  discard block
 block discarded – undo
349 349
 			$config['dirs'] = array_merge($config['dirs'], array_values($extension->getTranslationResources()));
350 350
 		}
351 351
 
352
-		$config['dirs'] = array_map(function ($dir) use ($builder) {
352
+		$config['dirs'] = array_map(function($dir) use ($builder) {
353 353
 			$path = Helpers::expand($dir, $builder->parameters);
354 354
 			return str_replace((DIRECTORY_SEPARATOR === '/') ? '\\' : '/', DIRECTORY_SEPARATOR, $path);
355 355
 		}, $config['dirs']);
@@ -393,8 +393,8 @@  discard block
 block discarded – undo
393 393
 		$whitelistRegexp = KdybyTranslator::buildWhitelistRegexp($config['whitelist']);
394 394
 		$translator = $builder->getDefinition($this->prefix('default'));
395 395
 
396
-		$mask = array_map(function ($value) {
397
-			return '*.*.' . $value;
396
+		$mask = array_map(function($value) {
397
+			return '*.*.'.$value;
398 398
 		}, array_keys($this->loaders));
399 399
 
400 400
 		foreach (Finder::findFiles($mask)->from($dirs) as $file) {
@@ -484,7 +484,7 @@  discard block
 block discarded – undo
484 484
 	 */
485 485
 	public static function register(Configurator $configurator)
486 486
 	{
487
-		$configurator->onCompile[] = function ($config, Compiler $compiler) {
487
+		$configurator->onCompile[] = function($config, Compiler $compiler) {
488 488
 			$compiler->addExtension('translation', new TranslationExtension());
489 489
 		};
490 490
 	}
Please login to merge, or discard this patch.