Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
database system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
23 Apr 2010 at 23:42 UTC
Updated:
16 Nov 2014 at 00:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Crell commentedhm. That's because we used % instead of @ as the placeholder type there. Oopsies.
Comment #2
Crell commentedA simple but tedious patch.
Comment #4
Crell commentedOh for the love of pete... It applies just fine for me, you stupid bot.
Comment #5
Crell commented#2: at-no-percent.patch queued for re-testing.
Comment #7
naxoc commentedCame across this today. Still here. Reroll for D8.
Comment #8
Crell commentedWe probably want to remove those t() calls anyway, but at least for now this is a bugfix.
Marking for backport, too. Thanks, naxoc!
Comment #9
webchickI confess I did not review all of these in-depth. :P But scanned it and it seems to be just the same % to @ change over and over and over. Well. Except for that dirty limerick that naxoc added to settings.php (I kid :)).
Committed and pushed to 8.x. Thanks!
Moving back to 7.x.
Comment #10
dcam commentedBackported #7 to D7.
Comment #11
mgiffordSo the changeover "% to @ change over" is just getting the format_string right.
It's in D8. It applies nicely at D7 (and it installs easily). I can't see any problem with marking this RTBC.
Comment #14
dcam commentedYet another random failure.
Comment #17
dcam commentedComment #20
dcam commentedComment #23
dcam commentedComment #26
dcam commentedComment #27
David_Rothstein commentedCommitted to 7.x - thanks!
So technically this does break lots of translations, but I can't imagine any of them show up in a situation where it really matters.
Note there was one broken one in the patch, which I fixed on commit (was not present in Drupal 8) via this interdiff: