set_js_option doesnt seem to work with group left blank...

[canton7]

set_js_option doesnt seem to work with group left blank...

[leeovery]

As a follow up Ive now decided to use a closure in the casset config to achieve the desired result. In the min and combine config options I simply return from a closure an environment bool.

[canton7]

Right... Just trying to work out what's going on here.

Currently, set_*_option only applies itself to groups which already exist. The global group doesn't exist until you add something to it (e.g. using Casset::js()), so I suspect some effect of this is what is causing Casset to fall over. I'll do some testing when I get home tonight.

You probably want Casset:set_js_option('*', ...), as this will apply to all groups, instead of just the global group. Again, it won't apply to groups that don't exist yet, but I'll fix that tonight.

My normal approach is to define a constant based on whether I'm in a dev environment or not (you can set this from a .htaccess file with Setenv CONST value), then use this to determine what value is set into the config file. There should be nothing wrong with your approach, though, so I'll fix whatever's breaking tonight.

E.g.

// In your config
'min' => FUEL_DEV ? false : true,

[leeovery]

Thanks.

Im pretty sure I tried the asterix option as "group" too. Infact I used that first but it didnt seem to work with global. I was trying to do this before using Casset::js() so thats probably the issue.

[canton7]

Yeah. I'll force creating of the global group if it doesn't exist, and make '*' change the class property as well.

[leeovery]

Great stuff :)

[canton7]

Can you test the feature/set_group_option_fix branch? Thanks

[leeovery]

Sure Ill get on it now.

[leeovery]

sorry need to do something first. Ill check it out later tonight. within a few hours.

[canton7]

No worries

[leeovery]

Checked and have submitted pull request.

Its based off your feature branch.

Lee Overy

On 03/10/2011 18:21, "Antony Male"
[email protected]
wrote:

No worries

Reply to this email directly or view it on GitHub:
#9 (comment)

[canton7]

Good catch!

Thanks, tidied it up a little (watch those whitespace changes) and merged in. Congrats on your first included commit :)

Does that all work?

[leeovery]

Haha thanks.

I noticed those whitespace diffs. I strip all white space when I save each file using textmate on mac.

I'll check it out in a bit.

Lee
Sent from my iPhone

On 3 Oct 2011, at 21:37, Antony Male [email protected] wrote:

Good catch!

Thanks, tidied it up a little (watch those whitespace changes) and merged in. Congrats on your first included commit :)

Does that all work?

Reply to this email directly or view it on GitHub:
#9 (comment)

[canton7]

Netbeans did up until the point where I re-installed it... Oh well. I'll make a dedicated whitespace-fixing commit in a bit, keep all the noise in the same place :P

[leeovery]

All checked and working fine my end. Good stuff. (from your feature branch with my commit)

This is also getting me pretty sharp at Git too which is a good thing :)

I also use GitFlow btw.

[leeovery]

You probably know this but develop is broken at the moment but to this issue.

Sorry if Im stating obvious!

[canton7]

All merged in :)

Sorry for the delay, was busy with something.