Example code to replicate this can be found on the github project for lessphp where I originally reported this, only to discover the latest version of lessc.inc.php fixes this: https://github.com/leafo/lessphp/issues/139

Having replaced the lessc.inc.php within this module, with the one from github master, this appears to have fixed it, so I think you just need to update the lib.

Comments

Bill Choy’s picture

Version: 7.x-2.4 » 6.x-2.7

Following.... Have the same problem. Replacing the "lessc.inc.php" fixed in 6.x-2.7 worked.
Both lessc file where marked, annoyingly, as v.0.2.1

The downloaded lessc actually look more like their v.0.3.0 branch.. which has an unidentified syntax change.

PS: I hope they fix the mixin's parameter delimiter from ";" to "," [ex @gradiant(@start; @end){ ... } ].
To better match the Less Function delimiter [ex @greylight:lighten(@grey, 10%); ].

thedavidmeister’s picture

same problem here, proposed fix of cloning the latest version of lessphp seems to make things work.

was breaking bootstrap (by twitter) until i updated lessc.inc.php

deciphered’s picture

Status: Active » Needs review
StatusFileSize
new36.33 KB

Find attached a patch that simply updates to the latest lessc.inc.php

corey.aufang’s picture

The current version of the module has Libraries API support.

If you download the latest version of lessphp and place in the libraries folder, it should be automatically detected and used in place of the bundled version.

I will be working on an update to the module to include the 3.0 version of lessphp, but until that occurs the best solution is to use the libraries method.

I'm going to update the module page to show that the module has Libraries API support.

corey.aufang’s picture

Status: Needs review » Closed (won't fix)

Closing as Libraries API is the current and most likely only future solution.

Reopen if you're still having issues.

thedavidmeister’s picture

Status: Closed (won't fix) » Active

why should the module knowingly ship with a broken version of the library?

the resolution to this issue should either be the module shipping with a version of the LESS library that functions as expected, or should not include any version of the library at all and always require the user to BYO library, preferably using the Libraries API as you suggested.

corey.aufang’s picture

Status: Active » Fixed

I agree.

The newest version requires that you download the lessphp library separately.

thedavidmeister’s picture

Status: Fixed » Closed (fixed)

great, i think that's a good solution. I've been having a few annoying bugs while implementing LESS on a few different projects and doing things this way means I always know who's issue queue to follow up when something ruins my day :)

thedavidmeister’s picture

Issue summary: View changes

missed url before