iznogoudatall / phamlp

Automatically exported from code.google.com/p/phamlp
0 stars 0 forks source link

load_paths option disappears when loading Sass extensions #108

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
What is the expected output? What do you see instead?

OK behaviour:
1. define a SassParser config with 'load_path' => array('whatever');
2. SassParser->getLoad_paths() returns the propper array.

BUG:
1. put in an extension, say 'extensions' => array('Compass');
2. SassParser->getLoad_paths() is empty.

What version of the product are you using? On what operating system?
phamlp 3.2 on wamp, php 5.3, but it's irrelevant

Please provide any additional information below.

I found the bug to be on lines 306 and 308 in SassParser.php
306: array($lp) : array_merge($options['load_paths'], $lp));
the $lp is a string, and array_merge cannot merge paths. It should be:
array($lp) : array_merge($options['load_paths'], array($lp)));

line 308 has the same bug, only with $fp.

Original issue reported on code.google.com by m...@chionsas.lt on 29 Aug 2011 at 10:53

GoogleCodeExporter commented 8 years ago
Thanks

Original comment by zeus.shl...@gmail.com on 1 Nov 2011 at 11:45