Use Drupal's date_format function to theme dates in mails
towlie - February 17, 2009 - 20:13
| Project: | Webform |
| Version: | 6.x-2.7 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
Hi everyone,
I'm using a german translation of Drupal. Webform is fully translated, the months are showing up right, but in the E-Mail confirmation the months aren't translated. I tried "Locale" and "String Overrides" - no success. Anyone any suggestion?
Greetings from Germany,
towlie
Don't forget to bring a towel!
#1
I believe that's the same issue I stumbled upon. However, it can be resolved with a simple patch. In date.inc (theme_webform_mail_date) instead of using PHP's date() function Drupal's date_format() function should be used. Patch file is attached, please review. Thanks.
#2
Works great! Thanks a lot!
Should be added to the next release. (Maybe someone should check it in an english version as well.)
#3
Ah, great. Thanks mglanznig! I'm moving this to patch, RTBC, so I remember to add it.
#4
Same problem now in the new version. I fixed the date.inc again but the patch doesn't work...
#5
The code is good, but the patch file seems to be broken.
It is trying to remove the correct line and adding the old one.
Try this patch, should work fine.
#6
It works fine :)
Thanks!
#7
The patch still needs to be committed, so we'll leave the status as RTBC for now.
#8
Committed to all versions.
#9
Automatically closed -- issue fixed for 2 weeks with no activity.