From 598b24910189c1ce3184cb80ecc64c843df453b8 Mon Sep 17 00:00:00 2001 From: Alexis Sellier Date: Tue, 28 Feb 2012 17:06:45 +0100 Subject: [PATCH] Fix '@arguments' behaviour, Closes #632 `.mixin` & `.mixin()` had differing behaviours. --- lib/less/parser.js | 2 +- test/css/mixins-args.css | 3 +++ test/less/mixins-args.less | 3 +++ 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/lib/less/parser.js b/lib/less/parser.js index 35d7f382..37376c37 100644 --- a/lib/less/parser.js +++ b/lib/less/parser.js @@ -763,7 +763,7 @@ less.Parser = function Parser(env) { } if (elements.length > 0 && ($(';') || peek('}'))) { - return new(tree.mixin.Call)(elements, args, index, env.filename, important); + return new(tree.mixin.Call)(elements, args || [], index, env.filename, important); } }, diff --git a/test/css/mixins-args.css b/test/css/mixins-args.css index 0544b828..e878ef6e 100644 --- a/test/css/mixins-args.css +++ b/test/css/mixins-args.css @@ -59,6 +59,9 @@ body { .arguments2 { border: 0px; } +.arguments3 { + border: 0px; +} .edge-case { border: "{"; } diff --git a/test/less/mixins-args.less b/test/less/mixins-args.less index 468d69ad..26e1d7a8 100644 --- a/test/less/mixins-args.less +++ b/test/less/mixins-args.less @@ -116,6 +116,9 @@ body { .arguments2 { .mixin-arguments(); } +.arguments3 { + .mixin-arguments; +} // Edge cases