Bug 257362 - [localization]: localization editor is allowing two constants with the same name
Summary: [localization]: localization editor is allowing two constants with the same name
Status: CLOSED FIXED
Alias: None
Product: MTJ (Archived)
Classification: Tools
Component: General (show other bugs)
Version: 0.9   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 0.9.1   Edit
Assignee: Diego Madruga Sandin CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed, core, ui
Depends on:
Blocks: 257363
  Show dependency tree
 
Reported: 2008-12-03 07:56 EST by Gustavo de Paula CLA
Modified: 2008-12-30 14:24 EST (History)
3 users (show)

See Also:

Attachments
Patch with bug resolution (29.97 KB, patch)
2008-12-15 14:09 EST, David Marques CLA 		no flags Details | Diff
Patch with editor state fixes (29.24 KB, patch)
2008-12-15 17:10 EST, David Marques CLA 		wds057: iplog+
wds057: review+
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gustavo de Paula CLA 2008-12-03 07:56:43 EST 
this needs to validates on the editor, since it doesn't make sense to have two constants with the same id
Comment 1 David Marques CLA 2008-12-11 08:52:44 EST 
Hello everyone,

I am taking a look at this bug. I will soon attach a patch with the resolution.

[],

David Marques
Comment 2 David Marques CLA 2008-12-15 14:09:08 EST 
Created attachment 120505 [details]
Patch with bug resolution

Hello everyone,

I am sending an attachment with the bug resolution. Can someone please revise it ?

[],

David Marques
Comment 3 Diego Madruga Sandin CLA 2008-12-15 16:28:19 EST 
I'll revise this patch.
Comment 4 Diego Madruga Sandin CLA 2008-12-15 16:50:31 EST 
Hi David,

I've found an problem when i try to save the localization data file. I try to save it, but the editor always show as the file still need to be saved. could you take a look at that?

Thanks.
Comment 5 David Marques CLA 2008-12-15 16:51:42 EST 
Comment on attachment 120505 [details]
Patch with bug resolution

obsolete patch
Comment 6 David Marques CLA 2008-12-15 17:10:26 EST 
Created attachment 120527 [details]
Patch with editor state fixes

Hello Diego,

I have not noticed the wrong editor behavior, thank you for finding it. Follows a patch with the resolution.

[],

David Marques
Comment 7 Diego Madruga Sandin CLA 2008-12-16 07:32:40 EST 
The patch was accepted and committed to SVN.
Comment 8 Gustavo de Paula CLA 2008-12-30 14:24:05 EST 
bug released into 0.9.1