Bug 109955 - [nls tooling] Properties file editor should detect duplicate properties.
Summary: [nls tooling] Properties file editor should detect duplicate properties.
Status: ASSIGNED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: JDT-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 276710 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-09-20 02:06 EDT by trevor campbell CLA
Modified: 2013-10-15 04:46 EDT (History)
2 users (show)

See Also:

Attachments
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 trevor campbell CLA 2005-09-20 02:06:14 EDT 
It would be nice if Properties file editor detected duplicate properties.
In long properties files on mature projects with many programmers these files
often end up in poor shape and having duplicate properties is a real problem.
Comment 1 Dani Megert CLA 2006-03-21 05:09:55 EST 
Try Source > Find Broken NLS Keys... (needs latest 3.2 I-build)
Comment 2 Dani Megert CLA 2009-05-18 10:20:12 EDT 
*** Bug 276710 has been marked as a duplicate of this bug. ***
Comment 3 Luciano CLA 2013-10-15 04:25:51 EDT 
Eclipse should NOT delete lines with "duplicate" key, because they can be loaded as String array by org.apache.commons.configuration.Configuration.getStringArray

http://commons.apache.org/proper/commons-configuration/apidocs/org/apache/commons/configuration/Configuration.html#getStringArray(java.lang.String)

Deleting lines with the same property and different values leads to loss of data (I'm experimenting now with Eclipse Kepler)!

This feature should be implemented as a command on demand (not automatic).
Thanks

PS: this issue is old and not up-to-date, I think I'm gonna create a new one
Comment 4 Luciano CLA 2013-10-15 04:46:40 EDT 
Sorry, I made a mistake: I agree that Eclipse should *only detect*.
I opened issue #419427 because Kepler actually deletes.

Thanks