Hello - the configuration settings don't seem to save properly. When I update, for example, SOOPER DropDownKit delay settings, and click the [ ] enable checkbox, the settings do appear to save in the MySQL database, but then they are not retrieved or displayed when I return to the configuration screen - the value fields are blank and I must re-enter the values every time I am on the screen, as well the checkboxes have been cleared and I must re-check them every time. Please advise why this is happening.

Comments

meramo’s picture

Subscribing

JurriaanRoelofs’s picture

For me they work fine, are other people having this problem too?
Please let me know what Operating System and Apache/MyQSL versions you work on.

Anonymous’s picture

I am having all of the reported issues (error messages that I had to turn off in error reporting settings; no response to changes in the configuration page; all settings have to be re-entered if I make one change to the configuration...).

According to PHPMyAdmin, I have MySQL client version: 5.1.47.

Not sure where to find out which version of Apache, or which OS. I am using Bluehost.com, if that helps: I believe they are Linux, but I could be wrong.

mediahackers’s picture

Subscribing. Same issues. Mac 10.5, MAMP MYSQL 5.137, PHP 5.2.11 Cheers.

gorsti’s picture

subscribing - same issues

Drupal 6.19
Access to update.php Protected
CKEditor 3.4
Configuration file Protected
Cron maintenance tasks Last run 9 min 4 sec ago
You can run cron manually.
Database updates Up to date
Drupal core update status No update data available
No information is available about potential new releases for currently installed modules and themes. To check for updates, you may need to run cron or you can check manually. Please note that checking for available updates can take a long time, so please be patient.
File system Writable (public download method)
GD library bundled (2.0.34 compatible)
HTML Purifier Library 4.2.0
Hierarchical Select All updates installed. Implementation modules are installed correctly.
Image import Import directory sites/default/files/images/import exists.
Image module directories Exists (sites/default/files/images).
Image toolkit The gd toolkit is installed.
Mime type detection PHP Fileinfo Extension
MySQL database 5.0.51a
PHP 5.2.4-2ubuntu5.12
PHP memory limit 128M
PHP register globals Disabled
Unicode library PHP Mbstring Extension
Update notifications Enabled
Upload progress Enabled (PECL uploadprogress)
Web server Apache/2.2.8 (Ubuntu) mod_python/3.3.1 Python/2.5.2 PHP/5.2.4-2ubuntu5.12 with Suhosin-Patch mod_ssl/2.2.8 OpenSSL/0.9.8g mod_perl/2.0.3 Perl/v5.8.8
getID3() 1.7.9-20090308
jQuery UI 1.6
jQuery Update 1.2.6

JurriaanRoelofs’s picture

it would be helpful if you guys could show me the errors that you see

quiquedb’s picture

same problem on 6.19 but working ok on 6.16...

subscribing.

JurriaanRoelofs’s picture

Assigned: Unassigned » JurriaanRoelofs
Status: Active » Postponed (maintainer needs more info)

It is working fine for me on 6.19 on both my LAMP server and my XAMPP Windows 7x64 system.

One thing that could cause this is when an admin theme is showing at /admin/build/themes/settings/sooper_tundra

You should always have Tundra enabled and set as default when editing theme settings. It is ok if you have a normal admin theme because Tundra will force itself on the theme settings page by terminating the admin theme before it loads, but some modules that mess with the admin interface can still mess things up, but if the Tundra theme loads at the theme settings form I have no idea what your problem is if you dont see any errors.
Did you guys check your status report? file system working ok?

Anonymous’s picture

Same issue here on 6.20. In addition on the settings page I get this PHP warning:

warning: array_merge() [function.array-merge]: Argument #2 is not an array in /var/www/web9/web/sites/all/themes/sooper_tundra/theme-settings-defaults.php on line 21.

zandros’s picture

Same errors here.

heliogabal’s picture

Same issue here, Acquia Drupal 1.2.32 (Drupal 6.20). I get the same error as #9 and the same behavior as described by tgroff.

Platform:
Debian Lenny 64bit
Apache/2.2.9 (Debian)
PHP 5.2.6-1+lenny9
MySQL 5.0.51a

zandros’s picture

I think the maintainer should check the warnings that are written down on a relevant issue. This issue may be related with that one.

zandros’s picture

As I can see in the coding, the author of this module uses a lot the following line:
include($file->filename);

After fixing the SOOPER Background-Image Kit I should ask him, what does he actually accept from filename? The path or the name of the file? If it is the path then it's ok. But if the author looked for the name of the file then it is not correct and should have used the following: $file->name

wd0013’s picture

I performed an echo in theme-settings-defaults.php here:

// Load SOOPER Features
foreach (file_scan_directory($theme_path .'features', 'defaults.inc', array('.', '..', 'CVS')) as $file) {
echo $file->filename . "
";
include($file->filename);

$defaults = array_merge($defaults, $sooperdefaults);
}

and produced this:

themes/tundra/sooper_tundra/features/sooper-slideshowkit/slideshowkit-theme-settings-defaults.inc
themes/tundra/sooper_tundra/features/sooper-linkicons/linkicons-theme-settings-defaults.inc
themes/tundra/sooper_tundra/features/sooper-dropdownkit/dropdownkit-theme-settings-defaults.inc
themes/tundra/sooper_tundra/features/sooper-layoutkit/layoutkit-theme-settings-defaults.inc
themes/tundra/sooper_tundra/features/sooper-slideshowkit/slideshowkit-theme-settings-defaults.inc
themes/tundra/sooper_tundra/features/sooper-linkicons/linkicons-theme-settings-defaults.inc
themes/tundra/sooper_tundra/features/sooper-dropdownkit/dropdownkit-theme-settings-defaults.inc
themes/tundra/sooper_tundra/features/sooper-layoutkit/layoutkit-theme-settings-defaults.inc

This tells me that $file->filename is correct. However I still get the following:
* warning: array_merge() [function.array-merge]: Argument #2 is not an array in /opt/lampp/htdocs/drupal6/themes/tundra/sooper_tundra/theme-settings-defaults.php on line 25.
* warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/drupal6/includes/form.inc on line 1213.
* warning: Invalid argument supplied for foreach() in /opt/lampp/htdocs/drupal6/includes/form.inc on line 1213.

So I added the print_r statement to this line in theme-settings-defaults.php:
// Merge the saved variables and their default values
print_r($saved_settings);

$settings = array_merge($defaults, $saved_settings);

And it returns the following:

Array
(
[mission] =>;
[default_logo] =>; 0
[logo_path] =>; sites/default/files/logo.png
[default_favicon] =>; 1
[favicon_path] =>;
[primary_links] =>; 1
[secondary_links] =>; 1
[toggle_logo] =>; 1
[toggle_favicon] =>; 1
[toggle_name] =>; 0
[toggle_search] =>; 0
[toggle_slogan] =>; 0
[toggle_mission] =>; 0
[toggle_node_user_picture] =>; 0
[toggle_comment_user_picture] =>; 0
[toggle_primary_links] =>; 1
[toggle_secondary_links] =>; 1
[toggle_node_info_page] =>; 0
[toggle_node_info_story] =>; 1
[logo_upload] =>;
[favicon_upload] =>;
[iepngfix] =>; 1
[block_edit] =>; 1
[force_eq_heights] =>; 1
[commentheader] =>; Comments:
[timestamp] =>; 0
[slideshowkit_enable] =>; 1
[cycle_invoke] =>; div.slideshow div >; ul
[cycle_fx] =>; scrollHorz
[cycle_timeout] =>; 4000
[cycle_continuous] =>; 0
[cycle_speed] =>; 800
[cycle_pagerEvent] =>; mouseover
[cycle_easing] =>; linear
[cycle_random] =>; 0
[cycle_pause] =>; 0
[cycle_pauseOnPagerHover] =>; 1
[cycle_delay] =>; 0
[link_icons_enable] =>; px
[link_icons_set] =>; 80
[sooperfish_invoke] =>; #navbar >; ul
[sooperfish_width] =>; 200
[sooperfish_enable] =>; 1
[sooperfish_delay] =>; 500
[sooperfish_dualcolumn] =>; 5
[sooperfish_triplecolumn] =>; 12
[sooperfish_speed_show] =>; 500
[sooperfish_speed_hide] =>; 500
[sooperfish_easing_show] =>; swing
[sooperfish_easing_hide] =>; swing
[sooperfish_properties_show] =>; Array
(
[height] =>; 1
)

[sooperfish_properties_hide] =>; Array
(
[opacity] =>; 1
[height] =>; 1
)

[fixedfluid] =>; px
[layout_width_fluid] =>; 80
[layout_width_fixed] =>; 960
[sidebar_width] =>; 25
[expand_admin] =>; 0
[layout_min_width] =>; 0
[layout_max_width] =>; 0
)

This leaves me more confused that ever, as the second argument is alive and well (and very much an array). I'll keep looking, but a clue would be nice.

BTW - I've tried this on XAMPP and in a standard LAMP stack in Ubuntu 64.

zandros’s picture

Yes, same here. Array seems to be fine, although I have to say that when I am trying to save a background image, the variable that stores it in the array seems empty. (but that's a different issue I guess)

JurriaanRoelofs’s picture

hey guys I mgoing on vacation 2morrow until the 16th so I'll probably look at this after I return, unless I break a leg on the slopes.

nicktr’s picture

Have a good trip! Look forward to getting some fixes when you're back! ;)

JurriaanRoelofs’s picture

I've fixed the warning about argument 2 in the theme-settings defaults not being an array, as well as the errors from common.inc - they were caused by some drupal_add_js functions in the font kit that were in Drupal 7 syntax instead of Drupal 6. Going to submit it to cvs soon, just need to do some more checks.

JurriaanRoelofs’s picture

Please update your Tundra installations to version Beta 2. I got a bunch of cvs errors before I could tag the beta so also please let me know if you spot a problem with this, I really dont get along well with CVS myself..

I put this important message on the project page, please make sure you update the features along with the Tundra theme itself:

"Make sure you download the zip files for the right version of the Tundra theme, If you update your Tundra theme to a newer version, you must also download the new zips from here. If you hover the zip links you can see they point to a specific version, this version should always match the version of the Tundra theme."

p.s. the CVS server needs some time from now before the beta2 package will show up

JurriaanRoelofs’s picture

Status: Postponed (maintainer needs more info) » Closed (won't fix)