Pass options object to parser.parse in less.render

[rback]

The options object is not passed to less.Parser.parse in less.render. This means that variables can't be overridden with modifyVars.

The tests for this pass because lessTester.runTestSet does not call less.render, but instead calls less.Parser.parse directly.

test/modify-vars.js contains a test that uses less.render to demonstrate the bug.

[rback]

This is more of a bug report than a proper pull request. I ran into this issue when trying to use modifyVars with gulp-less. I quickly compared gulp-less to the command line lessc and grunt-contrib-less, to see how it differs.

Gulp-less uses less.render which won't work because of this issue.

The command line lessc calls less.Parser.parse directly without passing modifyVars. Instead it modifies the input data before the call.

The grunt task works similarly to lessc. It also modifies the input data before calling less.Parser.parse.

Modifying the input data before calling the parse method seems unnecessary, since the parser appears to do this, if passed the appropriate additionalData.

Is less.Parser.parse the recommended way to use less as a library? I can make a PR to gulp-less if this is the case.

[blaugueux]

+1

[ashley7070]

would like this to be implemented as well

[xpavelf]

+1