[bg random tag stopped working [#665218]

I have a gallery with 6 sub-galleries, and all pictures are in the sub-galleries with nothing in the gallery root.
On the gallery page I just had

[bg||3|150|random|9]

and it would show a random 9 images from the sub-galleries.
Sometime during one of the last few updates, it stopped working.
I no longer get any images displayed in the root page.
http://acef.org/Gallery

Comment	File	Size	Author
#2	brilliant_gallery-bg_random_tag_stopped_working-665218-2.patch	2.57 KB	TonyT

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

ericwongcm CreditAttribution: ericwongcm commented 3 July 2013 at 16:42

Random 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 CreditAttribution: TonyT commented 5 January 2019 at 23:58

Issue summary:

View changes

File	Size
brilliant_gallery-bg_random_tag_stopped_working-665218-2.patch	2.57 KB

This 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.