Fix wrong identification of invalid effects (for unknown effects, it used to default...

diff --git a/src/live_effects/lpeobject.cpp b/src/live_effects/lpeobject.cpp
index bcb01463a741e609253fc29149996df32576424c..e7810fc28a81f0b6ccb6f8813c0df292ca8956c7 100644 (file)
--- a/src/live_effects/lpeobject.cpp
+++ b/src/live_effects/lpeobject.cpp
@@ -182,11 +182,9 @@ livepatheffect_set(SPObject *object, unsigned key, gchar const *value)
                 lpeobj->lpe = NULL;
             }
 
-            if (value) {
+            if ( value && Inkscape::LivePathEffect::LPETypeConverter.is_valid_key(value) ) {
                 lpeobj->effecttype = Inkscape::LivePathEffect::LPETypeConverter.get_id_from_key(value);
-                if (lpeobj->effecttype != Inkscape::LivePathEffect::INVALID_LPE) {
-                    lpeobj->lpe = Inkscape::LivePathEffect::Effect::New(lpeobj->effecttype, lpeobj);
-                }
+                lpeobj->lpe = Inkscape::LivePathEffect::Effect::New(lpeobj->effecttype, lpeobj);
                 lpeobj->effecttype_set = true;
             } else {
                 lpeobj->effecttype = Inkscape::LivePathEffect::INVALID_LPE;