Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Line 131 of Translator.php missing semi-colon #7290

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
brownie1 opened this issue Mar 6, 2013 · 1 comment
Closed

Line 131 of Translator.php missing semi-colon #7290

brownie1 opened this issue Mar 6, 2013 · 1 comment

Comments

@brownie1
Copy link

brownie1 commented Mar 6, 2013

When I used composer to get Translator for my Behat installation the I found line 131 missing a semi-colon. The original function looks like this:

public function setFallbackLocale($locales)
{
$this->setFallbackLocales(is_array($locales) ? $locales : array($locales))
}

I added a semi-colon to the line and everything works well. The line should look like the following:
$this->setFallbackLocales(is_array($locales) ? $locales : array($locales));

@stof
Copy link
Member

stof commented Mar 6, 2013

can you send a pull request ?

@fabpot fabpot closed this as completed in 4cf0848 Mar 6, 2013
igorw added a commit to igorw/symfony that referenced this issue Mar 8, 2013
* master: (449 commits)
  [2.2] add http_method_override option to ease setup
  fixed typo (closes symfony#7290)
  fixed ticket 7210
  [2.3] [DI] Improve the generated PHPDoc of the dumped PHP container
  [FrameworkBundle] Fixes invalid serialized objects in cache
  [WebProfilerBundle] removed the display of some information when their value is n/a
  remove dead code in yaml component
  Fixed typo in UPGRADE-2.2
  fixed typo
  RedisProfilerStorage wrong db-number/index-number selected
  [2.3] [DomCrawler] adds schema relative URL support to link
  [DependencyInjection] added a test for the previous merge (refs symfony#7261)
  Unset loading[$id] in ContainerBuilder on exception
  Default validation message translation fix.
  remove() should not use deprecated getParent() so it does not trigger deprecation internally
  adjust routing tests to not use prefix in addCollection
  add test for uniqueness of resources
  added tests for addDefaults, addRequirements, addOptions
  adjust RouteCollectionTest for the addCollection change and refactor the tests to only skip the part that really needs the config component
  added tests for remove() that wasnt covered yet and special route name
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants