Forbid empty custom property values

sass-spec: sass/sass-spec#1385 Fixes #2655
sass · Apr 20, 2019 · c51f442 · c51f442
1 parent f2db048
commit c51f442
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/expand.cpp b/src/expand.cpp
@@ -253,7 +253,13 @@ namespace Sass {
     if (value) value = value->perform(&eval);
     Block_Obj bb = ab ? operator()(ab) : NULL;
     if (!bb) {
-      if (!value || (value->is_invisible() && !d->is_important())) return 0;
+      if (!value || (value->is_invisible() && !d->is_important())) {
+        if (d->is_custom_property()) {
+          error("Custom property values may not be empty.", d->value()->pstate(), traces);
+        } else {
+          return nullptr;
+        }
+      }
     }
     Declaration* decl = SASS_MEMORY_NEW(Declaration,
                                         d->pstate(),

diff --git a/src/parser.cpp b/src/parser.cpp
@@ -1859,7 +1859,7 @@ namespace Sass {
       css_error("Invalid CSS", " after ", message);
     }
 
-    if (schema->empty()) return {};
+    if (schema->empty()) error("Custom property values may not be empty.");
     return schema.detach();
   }