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

Issues on 3locales page when locales 2 and 3 are identical #758

Open
TheoChevalier opened this issue Jul 2, 2016 · 1 comment
Open

Issues on 3locales page when locales 2 and 3 are identical #758

TheoChevalier opened this issue Jul 2, 2016 · 1 comment

Comments

@TheoChevalier
Copy link
Member

For example on this page: https://transvision.mozfr.org/3locales/?recherche=test&repo=aurora&sourcelocale=en-US&locale=sr&locale2=sr&search_type=strings

The 3rd header is missing, and links to source files are broken for the 3rd locale. This is caused by this check https://github.com/mozfr/transvision/blob/master/app/classes/Transvision/Search.php#L395-396

Before considering removing or working around it, I’d like to understand why it’s here :) I assume it’s because it’s conceptually false to state we have 3 locales while 2 are the same one. But that might be something else since currently “Source locale” and “extra locale” can be the same, we’re only checking locales 2 and 3.

Maybe on the 3locales page, we should redirect to the main search if we detect the 3rd locale is identical to the second one? Or just remove this check?

@ghost
Copy link

ghost commented Sep 6, 2016

I think we should redirect to the main search if the 3rd locale is the same as the 2nd one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant