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

Skip to content

[FrameworkBundle] Fix notice in Translator #206

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
wants to merge 1 commit into from
Closed

[FrameworkBundle] Fix notice in Translator #206

wants to merge 1 commit into from

Conversation

ornicar
Copy link
Contributor

@ornicar ornicar commented Mar 9, 2011

Make Translator.locale property protected, because the translator class in FrameworkBundle uses it.

I feel somewhat anxious with all these private things. So many doors closed.

Because the translator class in FrameworkBundle uses it.

I feel somewhat anxious with all these private things.
@docteurklein
Copy link
Contributor

indeed, 75b29ff introduces a notice:

Notice: Undefined property: Symfony\Bundle\FrameworkBundle\Translation\Translator::$locale in 
...vendor/symfony/src/Symfony/Bundle/FrameworkBundle/Translation/Translator.php on line 70

@fabpot
Copy link
Member

fabpot commented Mar 9, 2011

see PR 207

SofHad pushed a commit to SofHad/symfony that referenced this pull request Oct 12, 2015
…elease (bocharsky-bw)

This PR was merged into the master branch.

Discussion
----------

Update dependencies according to Symfony v2.7.5 release

Commits
-------

f1aa3cd Update dependencies according to Symfony v2.7.5 release
jderusse pushed a commit to jderusse/symfony that referenced this pull request Mar 30, 2020
Add PHP CodeSniffer advisory 2017-03-01
This pull request was closed.
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

Successfully merging this pull request may close these issues.

3 participants