search

Dexels / navajo

Navajo Service-oriented Applications
GNU Affero General Public License v3.0
9 stars 5 forks source link

… #568

Closed roelofkemp closed 3 years ago

roelofkemp commented 3 years ago

proper support for removeExisting=false, in this case the existing property should be kept, rather than throwing an exception. Also already set selections should not be removed. Note that all existing scripts have removingExisting=true, so they wont be affected.

roelofkemp commented 3 years ago

After discussion with @kharybdys (Marte), this change shouldn't be applied. Not specifying removeExisting is equivalent to removeExisting=false and there are many scripts that rely on this to result in an exception.