Pressflow 6.22
Perform a sitename/update.php and you get this error page:

Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'block_skinr_data' not found or invalid function name in module_invoke_all() (line 507 of /var/www/virtual/peterbowey.com.au/includes/module.inc).
Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'comment_skinr_data' not found or invalid function name in module_invoke_all() (line 507 of /var/www/virtual/peterbowey.com.au/includes/module.inc).
Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'node_skinr_data' not found or invalid function name in module_invoke_all() (line 507 of /var/www/virtual/peterbowey.com.au/includes/module.inc).
Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'views_skinr_data' not found or invalid function name in module_invoke_all() (line 507 of /var/www/virtual/peterbowey.com.au/includes/module.inc).

Errors do not occur for other Drupal sections, just UPDATE.PHP

Comments

Peter Bowey’s picture

Priority: Major » Critical

I have created a "Crude" solution:

Code changes in skinr.module: (see 3 new lines with '+')

/**
 * Implementation of hook_form_alter().
 */
function skinr_form_alter(&$form, $form_state, $form_id) {

+  if (defined('MAINTENANCE_MODE')) {
+    return;
+  }

  $skinr_data = skinr_fetch_data();
  $info = skinr_process_info_files();

  foreach ($skinr_data as $module => $settings) {
    if (isset($settings['form'][$form_id])) {
..
..

Why do I need do this obvious hack?

Peter Bowey’s picture

Status: Active » Needs work
moonray’s picture

That is actually the correct fix; core uses the same fix in some places. We've had this in the D7 version for a while. Not sure why it never got added to the D6 branch.

Peter Bowey’s picture

Refer #3

Thanks moonray,

Good to have it now declared public for other skinr D6 users!

moonray’s picture

Issue summary: View changes
Status: Needs work » Fixed

This was added with another patch. Fixed.

Status: Fixed » Closed (fixed)

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