Needs review
Project:
Brilliant Gallery
Version:
6.x-3.6
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Dec 2009 at 20:27 UTC
Updated:
5 Jan 2019 at 23:58 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
ericwongcm commentedRandom is working fine for me using Brilliant Gallery 3.6..
Check your parameter and make sure you don't have empty ||..
----------
Example, I was using syntax like this which cause Brilliant Gallery 3.6 to break my gallery..
[bg|photo||100||9]
It turns out that Brilliant Gallery 3.5 ignore the missing parameter within || whereas Brilliant Gallery 3.6 doesn't accept it and show tiny thumbnails instead of what I had specified.
My syntax should not have any empty || and should look like this
[bg|photo|9|100|random|9]
Comment #2
TonyT commentedThis patches fixes the problem that when using old style syntax, empty arguments caused the following arguments to be misinterpreted.
The issue was that empty arguments were being skipped and not put into the old style array that is created. They should only be skipped when using the new style syntax.
Tested with brilliant_gallery 7.x-1.10.
Comment #3
TonyT commented