Closed (fixed)
Project:
Image watermark
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Jun 2007 at 15:54 UTC
Updated:
11 Jul 2007 at 17:46 UTC
Jump to comment: Most recent file
Comments
Comment #1
kbahey commentedThis is the wrong project.
It has to go into image_gallery, not image watermark.
Comment #2
schnizZzla commentedPlease tell me, when or where did this error occur. Please post the url.
Gonna fix this bug soon.
For now if you want, i suppose to circumvent the problem you only need image gallery module installed and enabled. Also at least one gallery (vocabulary term) should be created.
Comment #3
schnizZzla commented@khalid
I dont know, sorry, i've posted a follow up the same time
Comment #4
schnizZzla commentedMy Follow up hast overwritten your changes, change it back, if you're sure. But I really think this is connected to a new feature (toggle). Unfortunately flaminhoon did not post enough information...
Comment #5
schnizZzla commentedI was right, I've forgotten a simple check...
Without image_gallery installed you get the error when you go to admin/settings/image_watermark.
@flaminhoon
follow my instructions above. If you do not want to install image gallery, you can install it, create the first gallery and then disable it. This is going to create the neccessary db entry to circumvent this problem...
Comment #6
drewish commentedthe error message make it seem like there's a bug in the query. i just checked the current version of image_gallery and it's properly quoted. make sure you're running the latest version.
Comment #7
schnizZzla commentedHere is a patch.
I've made an mistake in code which I ported from image gallery module. Missing quotes around the string messed the query up.
I was not sure about calling the original _image_gallery_get_vid() function, because this function creates a new gallery, when no galleries exist. That isn't neccessary on watermark configuration. So I ported code from image gallery into almost the same function _watermark_get_gallery_vid(), that only gets vids.
Comment #8
schnizZzla commentedthis patch belongs to watermark
Comment #9
kbahey commentedFixed in 5.x and HEAD.
Comment #10
schnizZzla commentedmerci
Comment #11
(not verified) commented