Skip to content

Commit

Permalink
fix codesniffer formatting errors
Browse files Browse the repository at this point in the history
  • Loading branch information
Schlaefer committed Jun 15, 2015
1 parent 2676242 commit 8b49355
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 22 deletions.
2 changes: 1 addition & 1 deletion src/I18n/I18n.php
Expand Up @@ -113,7 +113,7 @@ public static function translators()
* @param string|null $locale The locale for the translator.
* @param callable|null $loader A callback function or callable class responsible for
* constructing a translations package instance.
* @return \Aura\Intl\Translator The configured translator.
* @return \Aura\Intl\Translator|void The configured translator.
*/
public static function translator($name = 'default', $locale = null, callable $loader = null)
{
Expand Down
7 changes: 5 additions & 2 deletions src/I18n/TranslatorRegistry.php
Expand Up @@ -174,10 +174,12 @@ public function defaultFormatter($name = null)
* @param string|null $name fallback domain
* @return string
*/
public function defaultFallbackDomain($name = null) {
public function defaultFallbackDomain($name = null)
{
if ($name === null) {
return $this->_defaultFallbackDomain;
}

return $this->_defaultFallbackDomain = $name;
}

Expand Down Expand Up @@ -252,7 +254,8 @@ protected function _getFromLoader($name, $locale)
* @param callable $loader invokable loader
* @return callable loader
*/
public function setLoaderFallback($name, callable $loader) {
public function setLoaderFallback($name, callable $loader)
{
if ($name !== $this->_defaultFallbackDomain) {
$loader = function () use ($loader) {
$package = $loader();
Expand Down
29 changes: 10 additions & 19 deletions tests/TestCase/I18n/I18nTest.php
Expand Up @@ -524,17 +524,13 @@ public function testFallbackTranslatorFallbackDisabled()
I18n::translator('default', 'en_US', function () {
$package = new Package('default');
$package->setMessages(['Dog' => 'bark']);

return $package;
});
I18n::translator('custom', 'en_US',
function () {
$package = new Package('default');
$package->setMessages([]);

return $package;
}
);
I18n::translator('custom', 'en_US', function () {
$package = new Package('default');
$package->setMessages([]);
return $package;
});
$result = __d('custom', 'Dog');
$this->assertEquals('Dog', $result);
}
Expand All @@ -551,22 +547,17 @@ public function testFallbackTranslatorCustomDomain()
I18n::translator($customDomain, 'en_US', function () {
$package = new Package('default');
$package->setMessages(['Dog' => 'bark']);

return $package;
});
I18n::translator('custom', 'en_US',
function () {
$package = new Package('default');
$package->setMessages([]);

return $package;
}
);
I18n::translator('custom', 'en_US', function () {
$package = new Package('default');
$package->setMessages([]);
return $package;
});
$result = __d('custom', 'Dog');
$this->assertEquals('bark', $result);
}


/**
* Tests that it is possible to register a generic translators factory for a domain
* instead of having to create them manually
Expand Down

0 comments on commit 8b49355

Please sign in to comment.