Group reordering isn't working (Safari/EE2/Low Vars 1.3.6)
There seems to be an issue with the JS that saves changes to group ordering. Whenever I drag and drop a group I'm seeing the following error in my console:
TypeError: 'undefined' is not an object (evaluating '$(this).attr('id').replace')
I've updated the DB manually so I'm in no hurry for a fix, but I thought you might want a heads-up.
Great addon btw, I don't know why it's taken me so long to give it a try.
Replies
Low 3 Aug 2011 15:38
Hey Dom,
Confirmed! I think the reordering of groups will work if there is no ungrouped group, tho.
To fix, open /themes/third_party/low_variables/scripts/low_variables.js, go to line #93 and replace it with this:
if ($(this).attr('id')) new_order.push($(this).attr('id').replace('group_id_', ''));Dom S 3 Aug 2011 15:53
Hi Low,
Thanks for such a speedy fix, that's done the trick. For what it's worth I haven't got an ungrouped group - all my vars are in groups - but nonetheless that has sorted it.
Now I just need to figure out why I'm unable to use Low Vars as parameters for a channel entries tag. Are there any known issues with sites using the NSM bootstrap script? I'm using it to assign various additional global vars and I wonder if that's what's giving me problems. I am using early parsing and I don't have anything else on the sessions_end hook. It's probably a stupid error on my part but I can't seem to pin it down.
Thanks,
Dom
Low 3 Aug 2011 15:58
I take it you've enabled Early Parsing in both the extension settings and the vars in question?
To check whether they're parsed correctly, turn on template debugging. There you'll see a list of Snippets. The early parsed LVs should be in that list.
In the same debug info, you can see the opening tag for your channel:entries tag pair. If you can see parsed vars as parameters, things should work out okay.
I'm not aware of any conflicts with NSM Bootstrap...
Dom S 3 Aug 2011 16:00
Well my hunch was right, it was a stupid error on my part. I didn't realise that the early variable parsing needed to be set on individual vars as well as globally, I guess I overlooked the setting when I was creating them. Switching that on's got everything working perfectly. Thanks Low, it's much appreciated.
Low 3 Aug 2011 16:02
Yeah, it's also a per-var option, as you wouldn't want all vars to be parsed early, for security reasons.
You're welcome! Glad it worked out!