IXR_Server->call() fails when calling method that uses __call()

[dave1010]

I have a class in a plugin that uses PHP's magic method __call() to add some XMl-RPC callback methods, but IXR_Server->call() throws an IXR_Error as it uses method_exists() instead of is_callable().

method_exists() returns true only if a method is specifically defined (and is visible). is_callable() should be identical to method_exists(), except also return true on any classes with a visible __call() method.

[dave1010]

Didn't mean to close, sorry

[josephscott]

Any reason you aren't using the xmlrpc_methods filter to add new XML-RPC methods? I wrote a simple example on how to do this ​http://josephscott.org/archives/2008/11/adding-xml-rpc-methods-to-wordpress/

[dave1010]

I am using the xml_rpc filter (inside a class):

// lots of code snipped
class MyXmlRpc {
  public function __construct() {
    add_filter( 'xmlrpc_methods', array($this, 'add_new_xmlrpc_methods') );
  }
  public function add_new_xmlrpc_methods($methods) {
    foreach ($this->methods as $method) {
      $methods[$this->PREFIX . $method] = array($this, $method);
    }
    return $methods;
  }
  public function __call($method, $args) {
    // ... does lots of things with $method and $args here
    return call_user_func_array(array($this, $method), $args);
  }
}

I think I even used your how to :-)

[hakre]

Patch does not apply clean against trunk.

---

call() will only be called if a certain function does not exists by name. So I don't understand your code example, because if so, your call() implementation will fail by call_user_func_array() as $method on $this does not exists and that's why call() will be invoked.

Or do I see something wrong here?

[hakre]

same patch against trunk.

[dave1010]

Thanks for the patch against trunk. I'm still learning SVN.

__call() is called when a method of that name is not visible. I.e. if you have private / protected methods with the same name. This means certain non-public methods can be run, but they are filtered through __call(). In my code I'm also modifying $method in some cases.

Some random examples of what is possible using __call() in this situation:

if (strpos($method, 'secure', 0) !== false) {
  $this->authenticate();
}
if ($args[0] == 7 || date('D') == 'Mon') {
  $method .= '_awesome';
  $args[1]++;
}

I guess I could work around it by sending all XML-RPC requests to 1 method and adding an extra arg, but that's a bit messy.

[hakre]

I had missed the protected / private members, my fault. is_callable() looks valid to me. +1.

[nacin]

(In [16809]) s/method_exists/is_callable/. props hakre, fixes #14933.