Locales are not displayed on Facebook Badges page



8 years ago
8 years ago


(Reporter: stas, Assigned: allia.z)





(1 attachment)



8 years ago
+++ This bug was initially created as a clone of Bug #641119 +++

https://www.facebook.com/pages/Firefox-4/130379370362344?sk=app_122300121174636 is always in English, independently of the user's prefs on fb.
Assignee: server-ops → jeremy.orem+bugs

Comment 1

8 years ago
In case anyone gets to this before I wake up:

Looking at the code at <https://svn.mozilla.org/projects/mozilla.com/branches/firefox4/en-US/firefox/fb-badges/index.html>, please change

    /* Set the Locale */
    $locale = $user["user"]["locale"];
    putenv("LC_ALL=" . $locale);
    setlocale(LC_ALL, $locale);


    /* Set the Locale */
    $locale = $user["user"]["locale"];
    echo $locale;
    putenv("LC_ALL=" . $locale);
    echo setlocale(LC_ALL, $locale);
What does it echo when you access <https://www.facebook.com/pages/Firefox-4/130379370362344?sk=app_122300121174636>?

Can you repeat this after temporarily changing your lang prefs on Facebook to a different locale than en-US?

Assignee: jeremy.orem+bugs → shyam

Comment 2

8 years ago
I debugged this with Shyam and it turned out to be a bug in the code.  I'll attach a patch in a sec.
Assignee: shyam → nobody
Component: Server Operations → Facebook
Product: mozilla.org → Websites
QA Contact: mrz → facebook

Comment 3

8 years ago
Created attachment 519628 [details] [diff] [review]
Look for locales in the right dir
Assigning to Zach so he imports it into the upstream SVN.
Assignee: nobody → allia.z

Comment 5

8 years ago
Any chance that this can be fixed today, please?
(In reply to comment #5)
> Any chance that this can be fixed today, please?

Zach has been emailed ... I think we're waiting on him or has this already been committed to svn?
Update: just talked with Zach and he's going to have this patched in the next couple hours. He's also tweaking the CSS styles for locales. Stand by.

Comment 8

8 years ago
(In reply to comment #7)
> He's also tweaking the CSS styles for locales.

In fact, I don't think he should do this.  If he applies the patch Gandalf sent by email to enable custom-css, localizers and the l10n team will be able to tweak the CSS instead.

Comment 9

8 years ago
commited gandalf's fix and this fix
Last Resolved: 8 years ago
Resolution: --- → FIXED
I pulled the L10n-related commit into our SVN in r85307.

Comment 11

8 years ago
w00t!  The stage works now!
Nice! Both William and I see this awesomeness working... marking QA Verified so we can kick off the formal test cycle.


8 years ago
Blocks: 643110
You need to log in before you can comment on or make changes to this bug.