Closed (fixed)
Project:
Brilliant Gallery
Version:
5.x-2.14
Component:
User interface
Priority:
Critical
Category:
Support request
Assigned:
Reporter:
Created:
19 Nov 2007 at 18:05 UTC
Updated:
23 Jul 2008 at 10:56 UTC
If I change the table background colour it isn't saved.
After saving the current settings the config page reloads and the old settings are still there.
Comments
Comment #1
laszlo.laszlo commentedI've got the same problem with the color picker.
Where can I found the saved background color in the database?
Comment #2
vacilando commentedTested it on my system, works fine. It definitely will not be a bug.
Do similar forms save all right in your system(s)? I'd suspect a wider problem with your Drupal install. I recently saw something similar after upgrading to Drupal 5.3.. eventually I found it was due to a module that was incompatible with that version...
laszlo.laszlo - such variables are stored in table "variable".
Comment #3
jaegerschnitzel commentedI searched my database and I found no item with the name "laszlo"
and yes all other options are fine for me
I installed drupal 5.3 a week ago on a empty webspace and database
Comment #4
vacilando commentedjaegerschnitzel - you should look in table "variables" for entries starting "brilliang_gallery..."
"laszlo.laszlo" is an user name -- I was just answering the other user's question (see above)
Comment #5
jaegerschnitzel commentedoh sorry ;)
I found the value and changed it to s:7:"#FFFFFF";
but the background is still black...
yes I cleaned my cache ;)
EDIT: some idea?
Comment #6
Nitagob commentedThat is my solution (edit the brillant_gallery.module) for the problem (not a Brillant Gallery bug, possibly):
$form['brilliant_gallery_bcgcolour'] = array(
// '#type' => 'colorpicker',
'#type' => 'textfield',
'#title' => t('Table background colour'),
'#default_value' => variable_get('brilliant_gallery_bcgcolour', '#000000'),
'#size' => 8,
'#maxlength' => 7,
'#description' => t("Pick colour of the background of the table that holds the images.")
);
Of course, no any colorpicker since than, but it's unnecessary for me.
Comment #7
jaegerschnitzel commentedok I changed my code in brillant_gallery.module and now I can set the colour manually
but still the problem that the background ist black whatever I enter...
the value is not accepted
Comment #8
Nitagob commentedBtw, try to rename bordercolor to bgcolor in the following string at render_brilliant_gallery() function in the .module file (there are no reason to set bordercolor there anyway - border width is zero, isn't it?)):
Comment #9
Nitagob commentedOr you even cannot see the colour value updated at the corresponding administration textfield??!
Comment #10
jaegerschnitzel commentedno effect
this is stored in my DB:
brilliant_gallery_bcgcolour s:7:"#ffffff";
is this right?
if so I think the DB is not readout correctly by the script
Comment #11
Nitagob commentedBG uses standard Drupal API function
variable_get('brilliant_gallery_bcgcolour', '<default_value_here>')so probably the function works incorrectly.
Try to check versions of installed modules (especially kernel's required and additional), and the Drupal's error log.
Comment #12
jaegerschnitzel commentedstrange thing
now it works, but only after I deactivated and activated the Colorpicker module
changing the backgroundcolour shows no effect until I deactivate and activate the Colorpicker module
Comment #13
Nitagob commented"It works?? Don't change it!" ;))
Comment #14
vacilando commentedUnfortunately, this problem appeared after Colorpicker module changed the way forms have to be built.
I have now modified this module's code as needed. Update Brilliant Gallery to version 2.15 and Colorpicker will work again.
Cheers,
Tomáš / Vacilando
Comment #15
jaegerschnitzel commentedgreat now it works
thanks :D
Comment #16
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #17
ziadrupal commentedThe picture are displayed, but on content there are warnings like these:
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.
what's happen?
Comment #18
johnhorning commentedI am getting the same error message.
warning: Missing argument 8 for render_brilliant_gallery(), called in /home/rivulcom/public_html/sites/all/modules/brilliant_gallery/brilliant_gallery.module on line 59 and defined in /home/rivulcom/public_html/sites/all/modules/brilliant_gallery/brilliant_gallery.module on line 219.
Comment #19
whatthestink commentedthey forgot to give captionyestnotext a default value so change:
render_brilliant_gallery( $thisfolder = '', $colcountoverride = '', $widthoverride = '', $sortoverride = '', $maximumnumbertoshow = '', $colouroverride = '', $beginfromoverride = 1, $captionyesnotext) {
to
render_brilliant_gallery( $thisfolder = '', $colcountoverride = '', $widthoverride = '', $sortoverride = '', $maximumnumbertoshow = '', $colouroverride = '', $beginfromoverride = 1, $captionyesnotext = '') {
and it should work like a champ
Comment #20
johnhorning commentedThanks - that does the trick!
Comment #21
vacilando commentedYes, a forgotten default value. Fixed in version 3.2.
Comment #22
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.