lossendae / Cliche

A media manager for MODx Revolution
14 stars 11 forks source link

Undefined variable: controllerName #15

Closed DragonSigh closed 12 years ago

DragonSigh commented 12 years ago

Here I go again. Using ClicheAlbum snippet call results in error: Notice: Undefined variable: controllerName in C:\xampp\htdocs\decoracryl\core\cache\includes\elements\modsnippet\96.include.cache.php on line 23 and the message in the bottom "[Cliche] Album not found" but the album exists.

I use this simple call:

[[ClicheAlbum? &id=1 ]]

[[Cliche]] constantly shows 1 photo with id 6

[[ClicheItem? &id=6 ]]

doesn't show anything at all

And 1 small question: сan I disable adding jquery in code, because I already have one

lossendae commented 12 years ago

Can you update your snippet clichealbum with my last commit ? You can disable jquery loading by setting the following property : &loadJquery=0

DragonSigh commented 12 years ago

Thanks it fixes the error with albums! Right now I've got the strange behavior. Seems that Cliche doesn't add JS at all. I see css (/assets/components/cliche/plugins/default/style.css) definition but no JS at all.

And

[[ClicheItem? &id=6 ]]

still doesn't show anything at all, no errors, no images

lossendae commented 12 years ago

I've changed the property by default, it should work now. Concerning the second call, the snippet is called ClicheImage, i have to change the doc :)

DragonSigh commented 12 years ago

Thanks! Amazing work.

The last thing to fix is the same as for Album, but now for Image snippet call:

Notice: Undefined variable: controllerName in C:\xampp\htdocs\decoracryl\core\cache\includes\elements\modsnippet\97.include.cache.php on line 23

lossendae commented 12 years ago

Yes it will be updated tonight (or tomorrow). Thanks for your feedbacks :)