I've site which will be upgraded to 7.x, but meanwhile I'm looking for the same features as it's in 7.x
Please find the patch in the attachment.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kenorb’s picture

FileSize
12.39 KB

Ignore .info file changes.

kenorb’s picture

FileSize
19.12 KB

Added ctools plugins.

kenorb’s picture

FileSize
12.4 KB

Fixed warning:

Missing argument 2 for variable_get() called in snapengage.admin.inc
@@ -36,7 +36,7 @@ function snapengage_admin_settings_form(&$form_state) {
   $form['account']['snapengage_widget_advanced_script'] = array(
     '#type' => 'textarea',
     '#title' => t('Advanced script'),
-    '#default_value' => variable_get('snapengage_widget_advanced_script'),
+    '#default_value' => variable_get('snapengage_widget_advanced_script', NULL),
     '#description' => t("<strong>Beware</strong> messing around here can break your site. Leave blank if you just want to load the script and don't need adva
     '#states' => array(
       'invisible' => array(
arnested’s picture

I changed a few things in your patch:

  • I removed the visibility #states on the settings form since they are not part of Drupal 6 - instead I moved the advanced settings to a new, collapsed fieldset.
  • I added an hook_updated_N() which installs the supported languages on existing sites.
  • I removed the hook_ctools_plugin_directory() implementation since you didn't port the rest of the panels support anyway (and I haven't looked into what it would take to port it to Drupal 6 panels).

I have created a 6.x-1.1 release.

Greetings from DrupalCon Prague!

arnested’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

kenorb’s picture

Thank you.