Fatal error: Function name must be a string

Hi @jenstechs,

I've saw the bug that duplicated the filter settings form and fixed that. I wonder if that fixed this as well? It's been fixed in latest dev.

It's not fixed in latest dev.

thx, @anrikun, would you be able to provide a backtrace? or any other additional info so that I could replicate it?

Sure @vitalie! This issue is caused by line 104 (and possibly line 111) in spamspan_admin.php:

<?php
    // we need this to insert our own validate/submit handlers
    // we use our own validate handler to extract use_form settings
    $settings['use_form']['#process'] = array(
      array($this, 'settings_form_process'), // Line 104
    );
    return $settings;
  }

  //attach our validation
  public function settings_form_process(&$element, &$form_state, &$complete_form) {
    $complete_form['#validate'][] = array($this, 'settings_form_validate'); // Line 111
    return $element;
  }
?>

#process and #validate expect arrays of strings, not arrays of arrays.
This syntax may work in PHP 5.4 (not tested though) but doesn't in PHP 5.3.
So unless Spamspan drops support for PHP 5.3, this syntax should not be used.

thank you @anrikun. Could you please test this patch?

The error is gone for me @vitalie!
But I'd prefer that someone who "uses form" tests it too before this gets "reviewed & tested by the community" as I don't use form myself.

ok, @anrikun, I will let it be for some time, but taking into account that 'uses form' is in essence non-functional in the 7.x-1.2 release, it is most likely that no-one will confirm this in a long time, so I will go ahead and commit after a while.

@jenstechs it was altered to support php 5.3.

@vitalie (#7): in that case, let's commit it now.