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.
I was reviewing the changes in Drupal 6.8 and noticed what I think is a missing return statement.
I don't know what potential bugs it couses, but I decided to create a patch for it anyway. This is my first ever Drupal patch so please be nice...
Comment | File | Size | Author |
---|---|---|---|
#5 | locale.inc_.patch | 372 bytes | kajetan |
#3 | locale.inc_.patch | 374 bytes | kajetan |
locale.inc_.patch | 374 bytes | kajetan | |
Comments
Comment #1
Damien Tournoud CreditAttribution: Damien Tournoud commentedBumping to D7, which has exactly the same code. The only consequence of that bug is that the user is redirected to admin/build/translate instead of admin/build/translate/import.
The patch itself applies to D7 and looks good. I'm not convinced this requires testing, so RTBC.
Comment #3
kajetan CreditAttribution: kajetan commentedOk, thanx. I changed the status and upload another patch, that was made against 7.x-dev. Hope it will work.
Comment #5
kajetan CreditAttribution: kajetan commentedThat didn't work... So how about this? (there seems to be no good tools to create patches on windows)
Comment #6
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Moving to D6. Thanks!
Comment #7
Gábor HojtsyCommitted to Drupal 6 as well.