| Conditions | 10 |
| Paths | 1 |
| Total Lines | 166 |
| Code Lines | 61 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 178 | private function registerTranslator(Container $container) |
||
| 179 | { |
||
| 180 | /** |
||
| 181 | * Instance of the Translator Configset. |
||
| 182 | * |
||
| 183 | * @param Container $container Pimple DI container. |
||
| 184 | * @return TranslatorConfig |
||
| 185 | */ |
||
| 186 | $container['translator/config'] = function (Container $container) { |
||
| 187 | $appConfig = isset($container['config']) ? $container['config'] : []; |
||
| 188 | $translatorConfig = isset($appConfig['translator']) ? $appConfig['translator'] : null; |
||
| 189 | return new TranslatorConfig($translatorConfig); |
||
| 190 | }; |
||
| 191 | |||
| 192 | /** |
||
| 193 | * Dictionary of translations grouped by domain and locale. |
||
| 194 | * |
||
| 195 | * @param Container $container Pimple DI container. |
||
| 196 | * @return array |
||
| 197 | */ |
||
| 198 | $container['translator/translations'] = function (Container $container) { |
||
| 199 | $translatorConfig = $container['translator/config']; |
||
| 200 | return $translatorConfig['translations']; |
||
| 201 | }; |
||
| 202 | |||
| 203 | /** |
||
| 204 | * Instance of the Message Selector, that is used to resolve a translation. |
||
| 205 | * |
||
| 206 | * @return MessageSelector |
||
| 207 | */ |
||
| 208 | $container['translator/message-selector'] = function () { |
||
| 209 | return new MessageSelector(); |
||
| 210 | }; |
||
| 211 | |||
| 212 | /** |
||
| 213 | * Instance of the Translator, that is used for translation. |
||
| 214 | * |
||
| 215 | * @todo Improve file loader with a map of file formats. |
||
| 216 | * @param Container $container Pimple DI container. |
||
| 217 | * @return Translator |
||
| 218 | */ |
||
| 219 | $container['translator'] = function (Container $container) { |
||
| 220 | $translatorConfig = $container['translator/config']; |
||
| 221 | $translator = new Translator([ |
||
| 222 | 'manager' => $container['locales/manager'], |
||
| 223 | 'message_selector' => $container['translator/message-selector'], |
||
| 224 | 'cache_dir' => $translatorConfig['cache_dir'], |
||
| 225 | 'debug' => $translatorConfig['debug'] |
||
| 226 | ]); |
||
| 227 | |||
| 228 | $translator->setFallbackLocales($container['locales/fallback-languages']); |
||
| 229 | |||
| 230 | $translator->addLoader('array', $container['translator/loader/array']); |
||
| 231 | |||
| 232 | foreach ($translatorConfig['loaders'] as $loader) { |
||
| 233 | $translator->addLoader($loader, $container['translator/loader/file/'.$loader]); |
||
| 234 | $paths = array_reverse($translatorConfig['paths']); |
||
|
|
|||
| 235 | foreach ($translatorConfig['paths'] as $path) { |
||
| 236 | $path = realpath($container['config']['base_path'].$path); |
||
| 237 | if ($path === false) { |
||
| 238 | continue; |
||
| 239 | } |
||
| 240 | $files = glob($path.'/*.'.$loader); |
||
| 241 | foreach ($files as $f) { |
||
| 242 | $names = explode('.', basename($f)); |
||
| 243 | if (count($names) < 3) { |
||
| 244 | continue; |
||
| 245 | } |
||
| 246 | $lang = $names[1]; |
||
| 247 | $domain = $names[0]; |
||
| 248 | $translator->addResource($loader, $f, $lang, $domain); |
||
| 249 | } |
||
| 250 | } |
||
| 251 | } |
||
| 252 | |||
| 253 | foreach ($container['translator/translations'] as $domain => $data) { |
||
| 254 | foreach ($data as $locale => $messages) { |
||
| 255 | $translator->addResource('array', $messages, $locale, $domain); |
||
| 256 | } |
||
| 257 | } |
||
| 258 | |||
| 259 | return $translator; |
||
| 260 | }; |
||
| 261 | |||
| 262 | /** |
||
| 263 | * @return ArrayLoader |
||
| 264 | */ |
||
| 265 | $container['translator/loader/array'] = function () { |
||
| 266 | return new ArrayLoader(); |
||
| 267 | }; |
||
| 268 | |||
| 269 | /** |
||
| 270 | * @return CsvFileLoader |
||
| 271 | */ |
||
| 272 | $container['translator/loader/file/csv'] = function () { |
||
| 273 | return new CsvFileLoader(); |
||
| 274 | }; |
||
| 275 | |||
| 276 | /** |
||
| 277 | * @return IcuDatFileLoader |
||
| 278 | */ |
||
| 279 | $container['translator/loader/file/dat'] = function () { |
||
| 280 | return new IcuDatFileLoader(); |
||
| 281 | }; |
||
| 282 | |||
| 283 | /** |
||
| 284 | * @return IcuResFileLoader |
||
| 285 | */ |
||
| 286 | $container['translator/loader/file/res'] = function () { |
||
| 287 | return new IcuResFileLoader(); |
||
| 288 | }; |
||
| 289 | |||
| 290 | /** |
||
| 291 | * @return IniFileLoader |
||
| 292 | */ |
||
| 293 | $container['translator/loader/file/ini'] = function () { |
||
| 294 | return new IniFileLoader(); |
||
| 295 | }; |
||
| 296 | |||
| 297 | /** |
||
| 298 | * @return JsonFileLoader |
||
| 299 | */ |
||
| 300 | $container['translator/loader/file/json'] = function () { |
||
| 301 | return new JsonFileLoader(); |
||
| 302 | }; |
||
| 303 | |||
| 304 | /** |
||
| 305 | * @return MoFileLoader |
||
| 306 | */ |
||
| 307 | $container['translator/loader/file/mo'] = function () { |
||
| 308 | return new MoFileLoader(); |
||
| 309 | }; |
||
| 310 | |||
| 311 | /** |
||
| 312 | * @return PhpFileLoader |
||
| 313 | */ |
||
| 314 | $container['translator/loader/file/php'] = function () { |
||
| 315 | return new PhpFileLoader(); |
||
| 316 | }; |
||
| 317 | |||
| 318 | /** |
||
| 319 | * @return PoFileLoader |
||
| 320 | */ |
||
| 321 | $container['translator/loader/file/po'] = function () { |
||
| 322 | return new PoFileLoader(); |
||
| 323 | }; |
||
| 324 | |||
| 325 | /** |
||
| 326 | * @return QtFileLoader |
||
| 327 | */ |
||
| 328 | $container['translator/loader/file/qt'] = function () { |
||
| 329 | return new QtFileLoader(); |
||
| 330 | }; |
||
| 331 | |||
| 332 | /** |
||
| 333 | * @return XliffFileLoader |
||
| 334 | */ |
||
| 335 | $container['translator/loader/file/xliff'] = function () { |
||
| 336 | return new XliffFileLoader(); |
||
| 337 | }; |
||
| 338 | |||
| 339 | /** |
||
| 340 | * @return YamlFileLoader |
||
| 341 | */ |
||
| 342 | $container['translator/loader/file/yaml'] = function () { |
||
| 343 | return new YamlFileLoader(); |
||
| 344 | }; |
||
| 373 |