I follow all procedures and the images are displayed but on the is a warning like this:

warning: Missing argument 8 for render_brilliant_gallery(), called in C:\xampp\htdocs\drupal\sites\all\Modules\brilliant_gallery-5.x-3.1\brilliant_gallery\brilliant_gallery.module on line 59 and defined in C:\xampp\htdocs\drupal\sites\all\Modules\brilliant_gallery-5.x-3.1\brilliant_gallery\brilliant_gallery.module on line 219.

hope it will be solved quckly, thanks

CommentFileSizeAuthor
#2 brilliant_gallery-5.x.3.1-render_brilliant_gallery-warning.patch935 bytesmeichr

Comments

ziadrupal’s picture

ziadrupal commented

this happened on unix hosting too WOW

warning: Missing argument 8 for render_brilliant_gallery(), called in /home/sonicspe/public_html/site/sites/all/modules/brilliant_gallery/brilliant_gallery.module on line 59 and defined in /home/sonicspe/public_html/site/sites/all/modules/brilliant_gallery/brilliant_gallery.module on line 219.

meichr’s picture

meichr commented
StatusFileSize
new brilliant_gallery-5.x.3.1-render_brilliant_gallery-warning.patch935 bytes

Hi ziadrupal,

I can confirm the warning you mentioned: it occurs when you call the standard menu of brilliant gallery, i.e. http://add_your_domain_here/brilliant_gallery/, in your browser. The reason for this warning is, that in the function render_brilliant_gallery all but the 8th parameter have a default value defined (line 219) and the function is called without any parameter (line 59). PHP then prints this warning. An easy solution would be to define a default value also for the 8th parameter on line 219, best default value I think would be yesnocaption = ''.

To help you with a quick correction, I have enclosed a patch by which you can correct the version 5.x-3.1 of the module. Download the the patch file into the directory brilliant_gallery and use the unix command "patch < brilliant_gallery-5.x-3.1-render_brilliant_gallery-warning.patch" to correct the module file. Please let me know here if you have problems with the patch.

I suggest to include this correction in the next version of the module.

This should help you for the time being,
regards,
Christian.

meichr’s picture

meichr commented
Status: Active » Needs review

Just changing the status.
reg, Christian.

bluenightdk’s picture

bluenightdk commented

Hi Christian!
Seems like a nice solution.
What do I do if I'm working on a wamp server on localhost, and furthermore is a newbie???? :-)
Regards, Frank

meichr’s picture

meichr commented

There is a utility allowing to patch on Windows systems. Please look at

http://gnuwin32.sourceforge.net/packages/patch.htm

Chris.

meichr’s picture

meichr commented
Status: Needs review » Fixed

This bug fix has been applied to the release 5.x-4.4 at the latest. Thanks a lot.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.