Community
Participate
Working Groups
currently all constants are generated in lower case. this might cause issue with the developer set the constant id to some java reserved word, such as private
Hello everyone, I am taking a look at this bug. I will soon attach a patch with the resolution. [], David Marques
Created attachment 119934 [details] Patch with bug resolution Hello everyone, I am attaching a patch with this bug resolution. [], David Marques
If the user add in one locale the key "key1" and in other "Key1" the generated L10nConstants, will have duplicated KEY1 constants. Shouldn't it be better to append an "_" before the constant name and keep it the in the same case of the key available in the location data file, for example, for the keys "key1" and "Key1", create the "_key1" and "_Key1" constants? (In reply to comment #2) > Created an attachment (id=119934) [details] > Patch with bug resolution > > Hello everyone, > > I am attaching a patch with this bug resolution. > > [], > > David Marques >
i think it make more sense that constants are not case sensitive. at least a never saw / used case sensitive constants on any midlet. when keys are added on the location editor we should be able to validate that and make sure that the constants are not duplicated. :) gep (In reply to comment #3) > If the user add in one locale the key "key1" and in other "Key1" the generated > L10nConstants, will have duplicated KEY1 constants. > > Shouldn't it be better to append an "_" before the constant name and keep it > the in the same case of the key available in the location data file, for > example, for the keys "key1" and "Key1", create the "_key1" and "_Key1" > constants? > > > (In reply to comment #2) > > Created an attachment (id=119934) [details] [details] > > Patch with bug resolution > > > > Hello everyone, > > > > I am attaching a patch with this bug resolution. > > > > [], > > > > David Marques > > >
is current patch valid? (In reply to comment #4) > i think it make more sense that constants are not case sensitive. at least a > never saw / used case sensitive constants on any midlet. > > when keys are added on the location editor we should be able to validate that > and make sure that the constants are not duplicated. > > :) > gep > > (In reply to comment #3) > > If the user add in one locale the key "key1" and in other "Key1" the generated > > L10nConstants, will have duplicated KEY1 constants. > > > > Shouldn't it be better to append an "_" before the constant name and keep it > > the in the same case of the key available in the location data file, for > > example, for the keys "key1" and "Key1", create the "_key1" and "_Key1" > > constants? > > > > > > (In reply to comment #2) > > > Created an attachment (id=119934) [details] [details] [details] > > > Patch with bug resolution > > > > > > Hello everyone, > > > > > > I am attaching a patch with this bug resolution. > > > > > > [], > > > > > > David Marques > > > > > >
Hi guys, The patch itself generates all keys in upper case. Although it does not validate user entry on the editor. [], David Marques (In reply to comment #5) > is current patch valid? > > > (In reply to comment #4) > > i think it make more sense that constants are not case sensitive. at least a > > never saw / used case sensitive constants on any midlet. > > > > when keys are added on the location editor we should be able to validate that > > and make sure that the constants are not duplicated. > > > > :) > > gep > > > > (In reply to comment #3) > > > If the user add in one locale the key "key1" and in other "Key1" the generated > > > L10nConstants, will have duplicated KEY1 constants. > > > > > > Shouldn't it be better to append an "_" before the constant name and keep it > > > the in the same case of the key available in the location data file, for > > > example, for the keys "key1" and "Key1", create the "_key1" and "_Key1" > > > constants? > > > > > > > > > (In reply to comment #2) > > > > Created an attachment (id=119934) [details] [details] [details] [details] > > > > Patch with bug resolution > > > > > > > > Hello everyone, > > > > > > > > I am attaching a patch with this bug resolution. > > > > > > > > [], > > > > > > > > David Marques > > > > > > > > > >
Comment on attachment 119934 [details] Patch with bug resolution This patch is obsolete.
This bug was fixed in bug 257362.
bug released into 0.9.1