stromguo / js-builder

Automatically exported from code.google.com/p/js-builder
0 stars 0 forks source link

New output targets overwrite existing ones #13

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Not sure yet what all the situations are, but if you simply create a new
output target using the default name and file name, then create a second
one using the same defaults, the second one will silently overwrite the
first.  This also occurs sometimes even after changing the names, although
I'm not sure what the pattern is.

Original issue reported on code.google.com by bmoes...@gmail.com on 28 Nov 2006 at 5:00

GoogleCodeExporter commented 8 years ago

Original comment by bmoes...@gmail.com on 29 Nov 2006 at 6:12

GoogleCodeExporter commented 8 years ago
Changed logic to differentiate between adding a new target vs. editing an 
existing
one.  The original code always treated the target as an edit if you changed the
default "New Target 1" name and could overwrite other targets incorrectly.

I also added a check that prompts the user if they attempt to save a target 
with a
name that already exists in the project's targets.

Original comment by bmoes...@gmail.com on 30 Nov 2006 at 6:37