Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Hello
I am testing drupal 7 with french language, and there's a little charset problem with Administration menu when using apache with a default charset not set to UTF-8 : menu items are not displaying properly when loaded via ajax, in this case charset is set by apache.
Since it is common to set a default charset with apache, admin_menu_js_cache function should send Content-type header with correct charset.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | issue-1092970-1.patch | 329 bytes | hansfn |
Comments
Comment #1
hansfn CreditAttribution: hansfn commentedI can report the same problem using Norwegian Bokmål. Since the menu items get almost unreadable, I have bumped the priority to major. I have attached the trivial patch too. (You could consider using drupal_add_http_header in stead of calling header directly, but I'm not sure you gain anything in this case.)
Comment #2
sunComment #3
sunComment #4
madar CreditAttribution: madar commentedI have the same problem. Apply patch #1 is solve the problem for me.
Thanks!
Comment #5
hansfn CreditAttribution: hansfn commentedThx for the feedback. Setting status to tested.
PS! I wonder why this trivial patch hasn't been commited? It fixes a real issue and has no side-effects. Are we waiting for a fixing in core making the change unnecessary? (I checked the 7.x dev release - sorry, if this has been commited in git.)
Comment #7
hansfn CreditAttribution: hansfn commented#1: issue-1092970-1.patch queued for re-testing.
Comment #9
gardy CreditAttribution: gardy commentedJust insert this line to admin_menu.module at line 304, just below the other header settings:
drupal_add_http_header('Content-Type', 'text/csv; utf-8');
This solved the problem for me.
Comment #10
co_mete CreditAttribution: co_mete commentedThanks it worked fine for me.
Comment #11
checker CreditAttribution: checker commentedI have a default_charset = iso-8859-1 and need this patch to make admin_menu useable for me.
Comment #12
sun#1245014: Umlauts in the administration menu are not displayed correctly was marked as duplicate of this issue.
Comment #13
AnybodyWould be great if this can finally be fixed. It's still not working since month.
Regarding the comment above. Why "text/csv" and not "text/html"? Can somebody explain? Aren't we handling HTML content here?
Comment #14
sunThanks for reporting, reviewing, and testing! Committed to all branches.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.