Skip to content
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

#81 add translation on list and on filters #83

Closed
wants to merge 6 commits into from

Conversation

mcrinquand
Copy link
Contributor

In the configureQuery, with the translatable listener the locale is changed for Doctrine and data are translated on the list.

translationFieldFilter is a callback method to use on doctrine_orm_callback filter to filter on field and on its translation.

public function configureQuery(AdminInterface $admin, ProxyQueryInterface $query, $context = 'list')
{
$this->getTranslatableListener()->setTranslatableLocale($this->getTranslatableLocale($admin));
$this->getTranslatableListener()->setTranslationFallback('');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@liquorvicar i guess this would add more to the problem you report in #61 - then again i think the fix likely will look the same as for the other occurance of this code.

@dbu
Copy link
Contributor

dbu commented Mar 29, 2016

looks good to me. i am not familiar with the sonataci quick check - that one seems to fail.

@greg0ire
Copy link
Contributor

Run make cs to fix the build.

@greg0ire
Copy link
Contributor

i am not familiar with the sonataci quick check

@dbu : you have to click on "Console Output" to see something useful

@dbu
Copy link
Contributor

dbu commented Mar 30, 2016

continued in #84

@dbu dbu closed this Mar 30, 2016
@dbu
Copy link
Contributor

dbu commented Mar 30, 2016

oh, and @mcrinquand: please comment something after updating a pull request. i only get email notifications on comments, not on new commits, so there is quite a risk i would not see new commits.

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.

None yet

4 participants