Completed
Pull Request — master (#38)
by
unknown
09:58
created

Wrapper::php2XmlrpcType()   D

Complexity

Conditions 17
Paths 17

Size

Total Lines 35
Code Lines 30

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 17
eloc 30
nc 17
nop 1
dl 0
loc 35
rs 4.9807
c 1
b 0
f 0

How to fix   Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
/**
3
 * @author Gaetano Giunta
4
 * @copyright (C) 2006-2015 G. Giunta
5
 * @license code licensed under the BSD License: see file license.txt
6
 */
7
8
namespace PhpXmlRpc;
9
10
/**
11
 * PHP-XMLRPC "wrapper" class - generate stubs to transparently access xmlrpc methods as php functions and vice-versa.
12
 * Note: this class implements the PROXY pattern, but it is not named so to avoid confusion with http proxies.
13
 *
14
 * @todo use some better templating system for code generation?
15
 * @todo implement method wrapping with preservation of php objs in calls
16
 * @todo when wrapping methods without obj rebuilding, use return_type = 'phpvals' (faster)
17
 */
18
class Wrapper
19
{
20
    /// used to hold a reference to object instances whose methods get wrapped by wrapPhpFunction(), in 'create source' mode
21
    public static $objHolder = array();
22
23
    /**
24
     * Given a string defining a php type or phpxmlrpc type (loosely defined: strings
25
     * accepted come from javadoc blocks), return corresponding phpxmlrpc type.
26
     * Notes:
27
     * - for php 'resource' types returns empty string, since resources cannot be serialized;
28
     * - for php class names returns 'struct', since php objects can be serialized as xmlrpc structs
29
     * - for php arrays always return array, even though arrays sometimes serialize as json structs
30
     * - for 'void' and 'null' returns 'undefined'
31
     *
32
     * @param string $phpType
33
     *
34
     * @return string
35
     */
36
    public function php2XmlrpcType($phpType)
37
    {
38
        switch (strtolower($phpType)) {
39
            case 'string':
40
                return Value::$xmlrpcString;
41
            case 'integer':
42
            case Value::$xmlrpcInt: // 'int'
43
            case Value::$xmlrpcI4:
44
            case Value::$xmlrpcI8:
45
                return Value::$xmlrpcInt;
46
            case Value::$xmlrpcDouble: // 'double'
47
                return Value::$xmlrpcDouble;
48
            case 'bool':
49
            case Value::$xmlrpcBoolean: // 'boolean'
50
            case 'false':
51
            case 'true':
52
                return Value::$xmlrpcBoolean;
53
            case Value::$xmlrpcArray: // 'array':
54
                return Value::$xmlrpcArray;
55
            case 'object':
56
            case Value::$xmlrpcStruct: // 'struct'
57
                return Value::$xmlrpcStruct;
58
            case Value::$xmlrpcBase64:
59
                return Value::$xmlrpcBase64;
60
            case 'resource':
61
                return '';
62
            default:
63
                if (class_exists($phpType)) {
64
                    return Value::$xmlrpcStruct;
65
                } else {
66
                    // unknown: might be any 'extended' xmlrpc type
67
                    return Value::$xmlrpcValue;
68
                }
69
        }
70
    }
71
72
    /**
73
     * Given a string defining a phpxmlrpc type return the corresponding php type.
74
     *
75
     * @param string $xmlrpcType
76
     *
77
     * @return string
78
     */
79
    public function xmlrpc2PhpType($xmlrpcType)
80
    {
81
        switch (strtolower($xmlrpcType)) {
82
            case 'base64':
83
            case 'datetime.iso8601':
84
            case 'string':
85
                return Value::$xmlrpcString;
86
            case 'int':
87
            case 'i4':
88
            case 'i8':
89
                return 'integer';
90
            case 'struct':
91
            case 'array':
92
                return 'array';
93
            case 'double':
94
                return 'float';
95
            case 'undefined':
96
                return 'mixed';
97
            case 'boolean':
98
            case 'null':
99
            default:
100
                // unknown: might be any xmlrpc type
101
                return strtolower($xmlrpcType);
102
        }
103
    }
104
105
    /**
106
     * Given a user-defined PHP function, create a PHP 'wrapper' function that can
107
     * be exposed as xmlrpc method from an xmlrpc server object and called from remote
108
     * clients (as well as its corresponding signature info).
109
     *
110
     * Since php is a typeless language, to infer types of input and output parameters,
111
     * it relies on parsing the javadoc-style comment block associated with the given
112
     * function. Usage of xmlrpc native types (such as datetime.dateTime.iso8601 and base64)
113
     * in the @param tag is also allowed, if you need the php function to receive/send
114
     * data in that particular format (note that base64 encoding/decoding is transparently
115
     * carried out by the lib, while datetime vals are passed around as strings)
116
     *
117
     * Known limitations:
118
     * - only works for user-defined functions, not for PHP internal functions
119
     *   (reflection does not support retrieving number/type of params for those)
120
     * - functions returning php objects will generate special structs in xmlrpc responses:
121
     *   when the xmlrpc decoding of those responses is carried out by this same lib, using
122
     *   the appropriate param in php_xmlrpc_decode, the php objects will be rebuilt.
123
     *   In short: php objects can be serialized, too (except for their resource members),
124
     *   using this function.
125
     *   Other libs might choke on the very same xml that will be generated in this case
126
     *   (i.e. it has a nonstandard attribute on struct element tags)
127
     *
128
     * Note that since rel. 2.0RC3 the preferred method to have the server call 'standard'
129
     * php functions (ie. functions not expecting a single Request obj as parameter)
130
     * is by making use of the functions_parameters_type class member.
131
     *
132
     * @param callable $callable the PHP user function to be exposed as xmlrpc method/ a closure, function name, array($obj, 'methodname') or array('class', 'methodname') are ok
133
     * @param string $newFuncName (optional) name for function to be created. Used only when return_source in $extraOptions is true
134
     * @param array $extraOptions (optional) array of options for conversion. valid values include:
135
     *                            - bool return_source     when true, php code w. function definition will be returned, instead of a closure
136
     *                            - bool encode_php_objs   let php objects be sent to server using the 'improved' xmlrpc notation, so server can deserialize them as php objects
137
     *                            - bool decode_php_objs   --- WARNING !!! possible security hazard. only use it with trusted servers ---
138
     *                            - bool suppress_warnings remove from produced xml any warnings generated at runtime by the php function being invoked
139
     *
140
     * @return array|false false on error, or an array containing the name of the new php function,
141
     *                     its signature and docs, to be used in the server dispatch map
142
     *
143
     * @todo decide how to deal with params passed by ref in function definition: bomb out or allow?
144
     * @todo finish using phpdoc info to build method sig if all params are named but out of order
145
     * @todo add a check for params of 'resource' type
146
     * @todo add some trigger_errors / error_log when returning false?
147
     * @todo what to do when the PHP function returns NULL? We are currently returning an empty string value...
148
     * @todo add an option to suppress php warnings in invocation of user function, similar to server debug level 3?
149
     * @todo add a verbatim_object_copy parameter to allow avoiding usage the same obj instance?
150
     * @todo add an option to allow generated function to skip validation of number of parameters, as that is done by the server anyway
151
     */
152
    public function wrapPhpFunction($callable, $newFuncName = '', $extraOptions = array())
153
    {
154
        $buildIt = isset($extraOptions['return_source']) ? !($extraOptions['return_source']) : true;
155
156
        if (is_string($callable) && strpos($callable, '::') !== false) {
157
            $callable = explode('::', $callable);
158
        }
159
        if (is_array($callable)) {
160
            if (count($callable) < 2 || (!is_string($callable[0]) && !is_object($callable[0]))) {
161
                error_log('XML-RPC: ' . __METHOD__ . ': syntax for function to be wrapped is wrong');
162
                return false;
163
            }
164 View Code Duplication
            if (is_string($callable[0])) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
165
                $plainFuncName = implode('::', $callable);
166
            } elseif (is_object($callable[0])) {
167
                $plainFuncName = get_class($callable[0]) . '->' . $callable[1];
168
            }
169
            $exists = method_exists($callable[0], $callable[1]);
170
        } else if ($callable instanceof \Closure) {
171
            // we do not support creating code which wraps closures, as php does not allow to serialize them
172
            if (!$buildIt) {
173
                error_log('XML-RPC: ' . __METHOD__ . ': a closure can not be wrapped in generated source code');
174
                return false;
175
            }
176
177
            $plainFuncName = 'Closure';
178
            $exists = true;
179
        } else {
180
            $plainFuncName = $callable;
181
            $exists = function_exists($callable);
182
        }
183
184
        if (!$exists) {
185
            error_log('XML-RPC: ' . __METHOD__ . ': function to be wrapped is not defined: ' . $plainFuncName);
0 ignored issues
show
Bug introduced by
The variable $plainFuncName does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
186
            return false;
187
        }
188
189
        $funcDesc = $this->introspectFunction($callable, $plainFuncName);
0 ignored issues
show
Documentation introduced by
$plainFuncName is of type callable, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
190
        if (!$funcDesc) {
191
            return false;
192
        }
193
194
        $funcSigs = $this->buildMethodSignatures($funcDesc);
195
196
        if ($buildIt) {
197
            $callable = $this->buildWrapFunctionClosure($callable, $extraOptions, $plainFuncName, $funcDesc);
0 ignored issues
show
Documentation introduced by
$plainFuncName is of type callable, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
Documentation introduced by
$funcDesc is of type array<string,?,{"desc":"...returnsDocs":"string"}>, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
198
        } else {
199
            $newFuncName = $this->newFunctionName($callable, $newFuncName, $extraOptions);
200
            $code = $this->buildWrapFunctionSource($callable, $newFuncName, $extraOptions, $plainFuncName, $funcDesc);
0 ignored issues
show
Documentation introduced by
$plainFuncName is of type callable, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
201
        }
202
203
        $ret = array(
204
            'function' => $callable,
205
            'signature' => $funcSigs['sigs'],
206
            'docstring' => $funcDesc['desc'],
207
            'signature_docs' => $funcSigs['sigsDocs'],
208
        );
209
        if (!$buildIt) {
210
            $ret['function'] = $newFuncName;
211
            $ret['source'] = $code;
0 ignored issues
show
Bug introduced by
The variable $code does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
212
        }
213
        return $ret;
214
    }
215
216
    /**
217
     * Introspect a php callable and its phpdoc block and extract information about its signature
218
     *
219
     * @param callable $callable
220
     * @param string $plainFuncName
221
     * @return array|false
222
     */
223
    protected function introspectFunction($callable, $plainFuncName)
224
    {
225
        // start to introspect PHP code
226
        if (is_array($callable)) {
227
            $func = new \ReflectionMethod($callable[0], $callable[1]);
228
            if ($func->isPrivate()) {
229
                error_log('XML-RPC: ' . __METHOD__ . ': method to be wrapped is private: ' . $plainFuncName);
230
                return false;
231
            }
232
            if ($func->isProtected()) {
233
                error_log('XML-RPC: ' . __METHOD__ . ': method to be wrapped is protected: ' . $plainFuncName);
234
                return false;
235
            }
236
            if ($func->isConstructor()) {
237
                error_log('XML-RPC: ' . __METHOD__ . ': method to be wrapped is the constructor: ' . $plainFuncName);
238
                return false;
239
            }
240
            if ($func->isDestructor()) {
241
                error_log('XML-RPC: ' . __METHOD__ . ': method to be wrapped is the destructor: ' . $plainFuncName);
242
                return false;
243
            }
244
            if ($func->isAbstract()) {
245
                error_log('XML-RPC: ' . __METHOD__ . ': method to be wrapped is abstract: ' . $plainFuncName);
246
                return false;
247
            }
248
            /// @todo add more checks for static vs. nonstatic?
249
        } else {
250
            $func = new \ReflectionFunction($callable);
251
        }
252
        if ($func->isInternal()) {
253
            // Note: from PHP 5.1.0 onward, we will possibly be able to use invokeargs
254
            // instead of getparameters to fully reflect internal php functions ?
255
            error_log('XML-RPC: ' . __METHOD__ . ': function to be wrapped is internal: ' . $plainFuncName);
256
            return false;
257
        }
258
259
        // retrieve parameter names, types and description from javadoc comments
260
261
        // function description
262
        $desc = '';
263
        // type of return val: by default 'any'
264
        $returns = Value::$xmlrpcValue;
265
        // desc of return val
266
        $returnsDocs = '';
267
        // type + name of function parameters
268
        $paramDocs = array();
269
270
        $docs = $func->getDocComment();
271
        if ($docs != '') {
272
            $docs = explode("\n", $docs);
273
            $i = 0;
274
            foreach ($docs as $doc) {
275
                $doc = trim($doc, " \r\t/*");
276
                if (strlen($doc) && strpos($doc, '@') !== 0 && !$i) {
277
                    if ($desc) {
278
                        $desc .= "\n";
279
                    }
280
                    $desc .= $doc;
281
                } elseif (strpos($doc, '@param') === 0) {
282
                    // syntax: @param type $name [desc]
283
                    if (preg_match('/@param\s+(\S+)\s+(\$\S+)\s*(.+)?/', $doc, $matches)) {
284
                        $name = strtolower(trim($matches[2]));
285
                        //$paramDocs[$name]['name'] = trim($matches[2]);
0 ignored issues
show
Unused Code Comprehensibility introduced by
78% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
286
                        $paramDocs[$name]['doc'] = isset($matches[3]) ? $matches[3] : '';
287
                        $paramDocs[$name]['type'] = $matches[1];
288
                    }
289
                    $i++;
290
                } elseif (strpos($doc, '@return') === 0) {
291
                    // syntax: @return type [desc]
292
                    if (preg_match('/@return\s+(\S+)(\s+.+)?/', $doc, $matches)) {
293
                        $returns = $matches[1];
294
                        if (isset($matches[2])) {
295
                            $returnsDocs = trim($matches[2]);
296
                        }
297
                    }
298
                }
299
            }
300
        }
301
302
        // execute introspection of actual function prototype
303
        $params = array();
304
        $i = 0;
305
        foreach ($func->getParameters() as $paramObj) {
306
            $params[$i] = array();
307
            $params[$i]['name'] = '$' . $paramObj->getName();
308
            $params[$i]['isoptional'] = $paramObj->isOptional();
309
            $i++;
310
        }
311
312
        return array(
313
            'desc' => $desc,
314
            'docs' => $docs,
315
            'params' => $params, // array, positionally indexed
316
            'paramDocs' => $paramDocs, // array, indexed by name
317
            'returns' => $returns,
318
            'returnsDocs' =>$returnsDocs,
319
        );
320
    }
321
322
    /**
323
     * Given the method description given by introspection, create method signature data
324
     *
325
     * @todo support better docs with multiple types separated by pipes by creating multiple signatures
326
     *       (this is questionable, as it might produce a big matrix of possible signatures with many such occurrences)
327
     *
328
     * @param array $funcDesc as generated by self::introspectFunction()
329
     *
330
     * @return array
331
     */
332
    protected function buildMethodSignatures($funcDesc)
333
    {
334
        $i = 0;
335
        $parsVariations = array();
336
        $pars = array();
337
        $pNum = count($funcDesc['params']);
338 View Code Duplication
        foreach ($funcDesc['params'] as $param) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
339
            /* // match by name real param and documented params
0 ignored issues
show
Unused Code Comprehensibility introduced by
75% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
340
            $name = strtolower($param['name']);
341
            if (!isset($funcDesc['paramDocs'][$name])) {
342
                $funcDesc['paramDocs'][$name] = array();
343
            }
344
            if (!isset($funcDesc['paramDocs'][$name]['type'])) {
345
                $funcDesc['paramDocs'][$name]['type'] = 'mixed';
346
            }*/
347
348
            if ($param['isoptional']) {
349
                // this particular parameter is optional. save as valid previous list of parameters
350
                $parsVariations[] = $pars;
351
            }
352
353
            $pars[] = "\$p$i";
354
            $i++;
355
            if ($i == $pNum) {
356
                // last allowed parameters combination
357
                $parsVariations[] = $pars;
358
            }
359
        }
360
361
        if (count($parsVariations) == 0) {
362
            // only known good synopsis = no parameters
363
            $parsVariations[] = array();
364
        }
365
366
        $sigs = array();
367
        $sigsDocs = array();
368
        foreach ($parsVariations as $pars) {
369
            // build a signature
370
            $sig = array($this->php2XmlrpcType($funcDesc['returns']));
371
            $pSig = array($funcDesc['returnsDocs']);
372
            for ($i = 0; $i < count($pars); $i++) {
0 ignored issues
show
Performance Best Practice introduced by
It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

If the size of the collection does not change during the iteration, it is generally a good practice to compute it beforehand, and not on each iteration:

for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
Loading history...
373
                $name = strtolower($funcDesc['params'][$i]['name']);
374
                if (isset($funcDesc['paramDocs'][$name]['type'])) {
375
                    $sig[] = $this->php2XmlrpcType($funcDesc['paramDocs'][$name]['type']);
376
                } else {
377
                    $sig[] = Value::$xmlrpcValue;
378
                }
379
                $pSig[] = isset($funcDesc['paramDocs'][$name]['doc']) ? $funcDesc['paramDocs'][$name]['doc'] : '';
380
            }
381
            $sigs[] = $sig;
382
            $sigsDocs[] = $pSig;
383
        }
384
385
        return array(
386
            'sigs' => $sigs,
387
            'sigsDocs' => $sigsDocs
388
        );
389
    }
390
391
    /**
392
     * Creates a closure that will execute $callable
393
     * @todo validate params? In theory all validation is left to the dispatch map...
394
     * @todo add support for $catchWarnings
395
     *
396
     * @param $callable
397
     * @param array $extraOptions
398
     * @param string $plainFuncName
399
     * @param string $funcDesc
400
     * @return \Closure
401
     */
402
    protected function buildWrapFunctionClosure($callable, $extraOptions, $plainFuncName, $funcDesc)
0 ignored issues
show
Unused Code introduced by
The parameter $plainFuncName is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
403
    {
404
        $function = function($req) use($callable, $extraOptions, $funcDesc)
405
        {
406
            $nameSpace = '\\PhpXmlRpc\\';
407
            $encoderClass = $nameSpace.'Encoder';
408
            $responseClass = $nameSpace.'Response';
409
            $valueClass = $nameSpace.'Value';
410
411
            // validate number of parameters received
412
            // this should be optional really, as we assume the server does the validation
413
            $minPars = count($funcDesc['params']);
414
            $maxPars = $minPars;
415
            foreach ($funcDesc['params'] as $i => $param) {
0 ignored issues
show
Bug introduced by
The expression $funcDesc['params'] of type string is not traversable.
Loading history...
416
                if ($param['isoptional']) {
417
                    // this particular parameter is optional. We assume later ones are as well
418
                    $minPars = $i;
419
                    break;
420
                }
421
            }
422
            $numPars = $req->getNumParams();
423
            if ($numPars < $minPars || $numPars > $maxPars) {
424
                return new $responseClass(0, 3, 'Incorrect parameters passed to method');
425
            }
426
427
            $encoder = new $encoderClass();
428
            $options = array();
429
            if (isset($extraOptions['decode_php_objs']) && $extraOptions['decode_php_objs']) {
430
                $options[] = 'decode_php_objs';
431
            }
432
            $params = $encoder->decode($req, $options);
433
434
            $result = call_user_func_array($callable, $params);
435
436
            if (! is_a($result, $responseClass)) {
437
                if ($funcDesc['returns'] == Value::$xmlrpcDateTime || $funcDesc['returns'] == Value::$xmlrpcBase64) {
438
                    $result = new $valueClass($result, $funcDesc['returns']);
439
                } else {
440
                    $options = array();
441
                    if (isset($extraOptions['encode_php_objs']) && $extraOptions['encode_php_objs']) {
442
                        $options[] = 'encode_php_objs';
443
                    }
444
445
                    $result = $encoder->encode($result, $options);
446
                }
447
                $result = new $responseClass($result);
448
            }
449
450
            return $result;
451
        };
452
453
        return $function;
454
    }
455
456
    /**
457
     * Return a name for a new function, based on $callable, insuring its uniqueness
458
     * @param mixed $callable a php callable, or the name of an xmlrpc method
459
     * @param string $newFuncName when not empty, it is used instead of the calculated version
460
     * @return string
461
     */
462
    protected function newFunctionName($callable, $newFuncName, $extraOptions)
463
    {
464
        // determine name of new php function
465
466
        $prefix = isset($extraOptions['prefix']) ? $extraOptions['prefix'] : 'xmlrpc';
467
468
        if ($newFuncName == '') {
469
            if (is_array($callable)) {
470
                if (is_string($callable[0])) {
471
                    $xmlrpcFuncName = "{$prefix}_" . implode('_', $callable);
472
                } else {
473
                    $xmlrpcFuncName = "{$prefix}_" . get_class($callable[0]) . '_' . $callable[1];
474
                }
475
            } else {
476
                if ($callable instanceof \Closure) {
477
                    $xmlrpcFuncName = "{$prefix}_closure";
478
                } else {
479
                    $callable = preg_replace(array('/\./', '/[^a-zA-Z0-9_\x7f-\xff]/'),
480
                        array('_', ''), $callable);
481
                    $xmlrpcFuncName = "{$prefix}_$callable";
482
                }
483
            }
484
        } else {
485
            $xmlrpcFuncName = $newFuncName;
486
        }
487
488
        while (function_exists($xmlrpcFuncName)) {
489
            $xmlrpcFuncName .= 'x';
490
        }
491
492
        return $xmlrpcFuncName;
493
    }
494
495
    /**
496
     * @param $callable
497
     * @param string $newFuncName
498
     * @param array $extraOptions
499
     * @param string $plainFuncName
500
     * @param array $funcDesc
501
     * @return string
502
     *
503
     * @todo add a nice phpdoc block in the generated source
504
     */
505
    protected function buildWrapFunctionSource($callable, $newFuncName, $extraOptions, $plainFuncName, $funcDesc)
506
    {
507
        $namespace = '\\PhpXmlRpc\\';
508
509
        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool)$extraOptions['encode_php_objs'] : false;
510
        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool)$extraOptions['decode_php_objs'] : false;
511
        $catchWarnings = isset($extraOptions['suppress_warnings']) && $extraOptions['suppress_warnings'] ? '@' : '';
512
513
        $i = 0;
514
        $parsVariations = array();
515
        $pars = array();
516
        $pNum = count($funcDesc['params']);
517 View Code Duplication
        foreach ($funcDesc['params'] as $param) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
518
519
            if ($param['isoptional']) {
520
                // this particular parameter is optional. save as valid previous list of parameters
521
                $parsVariations[] = $pars;
522
            }
523
524
            $pars[] = "\$p[$i]";
525
            $i++;
526
            if ($i == $pNum) {
527
                // last allowed parameters combination
528
                $parsVariations[] = $pars;
529
            }
530
        }
531
532
        if (count($parsVariations) == 0) {
533
            // only known good synopsis = no parameters
534
            $parsVariations[] = array();
535
            $minPars = 0;
536
            $maxPars = 0;
537
        } else {
538
            $minPars = count($parsVariations[0]);
539
            $maxPars = count($parsVariations[count($parsVariations)-1]);
540
        }
541
542
        // build body of new function
543
544
        $innerCode = "\$paramCount = \$req->getNumParams();\n";
545
        $innerCode .= "if (\$paramCount < $minPars || \$paramCount > $maxPars) return new {$namespace}Response(0, " . PhpXmlRpc::$xmlrpcerr['incorrect_params'] . ", '" . PhpXmlRpc::$xmlrpcstr['incorrect_params'] . "');\n";
546
547
        $innerCode .= "\$encoder = new {$namespace}Encoder();\n";
548
        if ($decodePhpObjects) {
549
            $innerCode .= "\$p = \$encoder->decode(\$req, array('decode_php_objs'));\n";
550
        } else {
551
            $innerCode .= "\$p = \$encoder->decode(\$req);\n";
552
        }
553
554
        // since we are building source code for later use, if we are given an object instance,
555
        // we go out of our way and store a pointer to it in a static class var var...
556
        if (is_array($callable) && is_object($callable[0])) {
557
            self::$objHolder[$newFuncName] = $callable[0];
558
            $innerCode .= "\$obj = PhpXmlRpc\\Wrapper::\$objHolder['$newFuncName'];\n";
559
            $realFuncName = '$obj->' . $callable[1];
560
        } else {
561
            $realFuncName = $plainFuncName;
562
        }
563
        foreach ($parsVariations as $i => $pars) {
564
            $innerCode .= "if (\$paramCount == " . count($pars) . ") \$retval = {$catchWarnings}$realFuncName(" . implode(',', $pars) . ");\n";
565
            if ($i < (count($parsVariations) - 1))
566
                $innerCode .= "else\n";
567
        }
568
        $innerCode .= "if (is_a(\$retval, '{$namespace}Response')) return \$retval; else\n";
569
        if ($funcDesc['returns'] == Value::$xmlrpcDateTime || $funcDesc['returns'] == Value::$xmlrpcBase64) {
570
            $innerCode .= "return new {$namespace}Response(new {$namespace}Value(\$retval, '{$funcDesc['returns']}'));";
571
        } else {
572
            if ($encodePhpObjects) {
573
                $innerCode .= "return new {$namespace}Response(\$encoder->encode(\$retval, array('encode_php_objs')));\n";
574
            } else {
575
                $innerCode .= "return new {$namespace}Response(\$encoder->encode(\$retval));\n";
576
            }
577
        }
578
        // shall we exclude functions returning by ref?
579
        // if($func->returnsReference())
0 ignored issues
show
Unused Code Comprehensibility introduced by
78% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
580
        //     return false;
581
582
        $code = "function $newFuncName(\$req) {\n" . $innerCode . "\n}";
583
584
        return $code;
585
    }
586
587
    /**
588
     * Given a user-defined PHP class or php object, map its methods onto a list of
589
     * PHP 'wrapper' functions that can be exposed as xmlrpc methods from an xmlrpc server
590
     * object and called from remote clients (as well as their corresponding signature info).
591
     *
592
     * @param string|object $className the name of the class whose methods are to be exposed as xmlrpc methods, or an object instance of that class
593
     * @param array $extraOptions see the docs for wrapPhpMethod for basic options, plus
594
     *                            - string method_type    'static', 'nonstatic', 'all' and 'auto' (default); the latter will switch between static and non-static depending on whether $className is a class name or object instance
595
     *                            - string method_filter  a regexp used to filter methods to wrap based on their names
596
     *                            - string prefix         used for the names of the xmlrpc methods created
597
     *
598
     * @return array|false false on failure
599
     */
600
    public function wrapPhpClass($className, $extraOptions = array())
601
    {
602
        $methodFilter = isset($extraOptions['method_filter']) ? $extraOptions['method_filter'] : '';
603
        $methodType = isset($extraOptions['method_type']) ? $extraOptions['method_type'] : 'auto';
604
        $prefix = isset($extraOptions['prefix']) ? $extraOptions['prefix'] : '';
605
606
        $results = array();
607
        $mList = get_class_methods($className);
608
        foreach ($mList as $mName) {
609
            if ($methodFilter == '' || preg_match($methodFilter, $mName)) {
610
                $func = new \ReflectionMethod($className, $mName);
611
                if (!$func->isPrivate() && !$func->isProtected() && !$func->isConstructor() && !$func->isDestructor() && !$func->isAbstract()) {
612
                    if (($func->isStatic() && ($methodType == 'all' || $methodType == 'static' || ($methodType == 'auto' && is_string($className)))) ||
613
                        (!$func->isStatic() && ($methodType == 'all' || $methodType == 'nonstatic' || ($methodType == 'auto' && is_object($className))))
614
                    ) {
615
                        $methodWrap = $this->wrapPhpFunction(array($className, $mName), '', $extraOptions);
616
                        if ($methodWrap) {
617
                            if (is_object($className)) {
618
                                $realClassName = get_class($className);
619
                            }else {
620
                                $realClassName = $className;
621
                            }
622
                            $results[$prefix."$realClassName.$mName"] = $methodWrap;
623
                        }
624
                    }
625
                }
626
            }
627
        }
628
629
        return $results;
630
    }
631
632
    /**
633
     * Given an xmlrpc client and a method name, register a php wrapper function
634
     * that will call it and return results using native php types for both
635
     * params and results. The generated php function will return a Response
636
     * object for failed xmlrpc calls.
637
     *
638
     * Known limitations:
639
     * - server must support system.methodsignature for the wanted xmlrpc method
640
     * - for methods that expose many signatures, only one can be picked (we
641
     *   could in principle check if signatures differ only by number of params
642
     *   and not by type, but it would be more complication than we can spare time)
643
     * - nested xmlrpc params: the caller of the generated php function has to
644
     *   encode on its own the params passed to the php function if these are structs
645
     *   or arrays whose (sub)members include values of type datetime or base64
646
     *
647
     * Notes: the connection properties of the given client will be copied
648
     * and reused for the connection used during the call to the generated
649
     * php function.
650
     * Calling the generated php function 'might' be slow: a new xmlrpc client
651
     * is created on every invocation and an xmlrpc-connection opened+closed.
652
     * An extra 'debug' param is appended to param list of xmlrpc method, useful
653
     * for debugging purposes.
654
     *
655
     * @todo allow caller to give us the method signature instead of querying for it, or just say 'skip it'
656
     * @todo if we can not retrieve method signature, create a php function with varargs
657
     * @todo allow the created function to throw exceptions on method calls failures
658
     * @todo if caller did not specify a specific sig, shall we support all of them?
659
     *       It might be hard (hence slow) to match based on type and number of arguments...
660
     *
661
     * @param Client $client an xmlrpc client set up correctly to communicate with target server
662
     * @param string $methodName the xmlrpc method to be mapped to a php function
663
     * @param array $extraOptions array of options that specify conversion details. Valid options include
664
     *                            - integer signum              the index of the method signature to use in mapping (if method exposes many sigs)
665
     *                            - integer timeout             timeout (in secs) to be used when executing function/calling remote method
666
     *                            - string  protocol            'http' (default), 'http11' or 'https'
667
     *                            - string  new_function_name   the name of php function to create, when return_source is used. If unspecified, lib will pick an appropriate name
668
     *                            - string  return_source       if true return php code w. function definition instead of function itself (closure)
669
     *                            - bool    encode_php_objs     let php objects be sent to server using the 'improved' xmlrpc notation, so server can deserialize them as php objects
670
     *                            - bool    decode_php_objs     --- WARNING !!! possible security hazard. only use it with trusted servers ---
671
     *                            - mixed   return_on_fault     a php value to be returned when the xmlrpc call fails/returns a fault response (by default the Response object is returned in this case). If a string is used, '%faultCode%' and '%faultString%' tokens will be substituted with actual error values
672
     *                            - bool    debug               set it to 1 or 2 to see debug results of querying server for method synopsis
673
     *                            - int     simple_client_copy  set it to 1 to have a lightweight copy of the $client object made in the generated code (only used when return_source = true)
674
     *
675
     * @return \closure|array|false false on failure, closure by default and array for return_source = true
676
     */
677
    public function wrapXmlrpcMethod($client, $methodName, $extraOptions = array())
678
    {
679
        $newFuncName = isset($extraOptions['new_function_name']) ? $extraOptions['new_function_name'] : '';
680
681
        $buildIt = isset($extraOptions['return_source']) ? !($extraOptions['return_source']) : true;
682
683
        $mSig = $this->retrieveMethodSignature($client, $methodName, $extraOptions);
684
        if (!$mSig) {
685
            return false;
686
        }
687
688
        if ($buildIt) {
689
            return $this->buildWrapMethodClosure($client, $methodName, $extraOptions, $mSig);
0 ignored issues
show
Documentation introduced by
$mSig is of type array, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
690
        } else {
691
            // if in 'offline' mode, retrieve method description too.
692
            // in online mode, favour speed of operation
693
            $mDesc = $this->retrieveMethodHelp($client, $methodName, $extraOptions);
694
695
            $newFuncName = $this->newFunctionName($methodName, $newFuncName, $extraOptions);
696
697
            $results = $this->buildWrapMethodSource($client, $methodName, $extraOptions, $newFuncName, $mSig, $mDesc);
698
            /* was: $results = $this->build_remote_method_wrapper_code($client, $methodName,
0 ignored issues
show
Unused Code Comprehensibility introduced by
63% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
699
                $newFuncName, $mSig, $mDesc, $timeout, $protocol, $simpleClientCopy,
700
                $prefix, $decodePhpObjects, $encodePhpObjects, $decodeFault,
701
                $faultResponse, $namespace);*/
702
703
            $results['function'] = $newFuncName;
704
705
            return $results;
706
        }
707
708
    }
709
710
    /**
711
     * Retrieves an xmlrpc method signature from a server which supports system.methodSignature
712
     * @param Client $client
713
     * @param string $methodName
714
     * @param array $extraOptions
715
     * @return false|array
716
     */
717
    protected function retrieveMethodSignature($client, $methodName, array $extraOptions = array())
718
    {
719
        $namespace = '\\PhpXmlRpc\\';
720
        $reqClass = $namespace . 'Request';
721
        $valClass = $namespace . 'Value';
722
        $decoderClass = $namespace . 'Encoder';
723
724
        $debug = isset($extraOptions['debug']) ? ($extraOptions['debug']) : 0;
725
        $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
726
        $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
727
        $sigNum = isset($extraOptions['signum']) ? (int)$extraOptions['signum'] : 0;
728
729
        $req = new $reqClass('system.methodSignature');
730
        $req->addparam(new $valClass($methodName));
731
        $client->setDebug($debug);
732
        $response = $client->send($req, $timeout, $protocol);
733
        if ($response->faultCode()) {
734
            error_log('XML-RPC: ' . __METHOD__ . ': could not retrieve method signature from remote server for method ' . $methodName);
735
            return false;
736
        }
737
738
        $mSig = $response->value();
739
        if ($client->return_type != 'phpvals') {
740
            $decoder = new $decoderClass();
741
            $mSig = $decoder->decode($mSig);
742
        }
743
744
        if (!is_array($mSig) || count($mSig) <= $sigNum) {
745
            error_log('XML-RPC: ' . __METHOD__ . ': could not retrieve method signature nr.' . $sigNum . ' from remote server for method ' . $methodName);
746
            return false;
747
        }
748
749
        return $mSig[$sigNum];
750
    }
751
752
    /**
753
     * @param Client $client
754
     * @param string $methodName
755
     * @param array $extraOptions
756
     * @return string in case of any error, an empty string is returned, no warnings generated
757
     */
758
    protected function retrieveMethodHelp($client, $methodName, array $extraOptions = array())
759
    {
760
        $namespace = '\\PhpXmlRpc\\';
761
        $reqClass = $namespace . 'Request';
762
        $valClass = $namespace . 'Value';
763
764
        $debug = isset($extraOptions['debug']) ? ($extraOptions['debug']) : 0;
765
        $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
766
        $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
767
768
        $mDesc = '';
769
770
        $req = new $reqClass('system.methodHelp');
771
        $req->addparam(new $valClass($methodName));
772
        $client->setDebug($debug);
773
        $response = $client->send($req, $timeout, $protocol);
774
        if (!$response->faultCode()) {
775
            $mDesc = $response->value();
776
            if ($client->return_type != 'phpvals') {
777
                $mDesc = $mDesc->scalarval();
778
            }
779
        }
780
781
        return $mDesc;
782
    }
783
784
    /**
785
     * @param Client $client
786
     * @param string $methodName
787
     * @param array $extraOptions
788
     * @param string $mSig
789
     * @return \Closure
790
     *
791
     * @todo should we allow usage of parameter simple_client_copy to mean 'do not clone' in this case?
792
     */
793
    protected function buildWrapMethodClosure($client, $methodName, array $extraOptions, $mSig)
794
    {
795
        // we clone the client, so that we can modify it a bit independently of the original
796
        $clientClone = clone $client;
797
        $function = function() use($clientClone, $methodName, $extraOptions, $mSig)
798
        {
799
            $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
800
            $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
801
            $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool)$extraOptions['encode_php_objs'] : false;
802
            $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool)$extraOptions['decode_php_objs'] : false;
803 View Code Duplication
            if (isset($extraOptions['return_on_fault'])) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
804
                $decodeFault = true;
805
                $faultResponse = $extraOptions['return_on_fault'];
806
            } else {
807
                $decodeFault = false;
808
            }
809
810
            $namespace = '\\PhpXmlRpc\\';
811
            $reqClass = $namespace . 'Request';
812
            $encoderClass = $namespace . 'Encoder';
813
            $valueClass = $namespace . 'Value';
814
815
            $encoder = new $encoderClass();
816
            $encodeOptions = array();
817
            if ($encodePhpObjects) {
818
                $encodeOptions[] = 'encode_php_objs';
819
            }
820
            $decodeOptions = array();
821
            if ($decodePhpObjects) {
822
                $decodeOptions[] = 'decode_php_objs';
823
            }
824
825
            /// @todo check for insufficient nr. of args besides excess ones? note that 'source' version does not...
826
827
            // support one extra parameter: debug
828
            $maxArgs = count($mSig)-1; // 1st element is the return type
829
            $currentArgs = func_get_args();
830
            if (func_num_args() == ($maxArgs+1)) {
831
                $debug = array_pop($currentArgs);
832
                $clientClone->setDebug($debug);
833
            }
834
835
            $xmlrpcArgs = array();
836
            foreach($currentArgs as $i => $arg) {
837
                if ($i == $maxArgs) {
838
                    break;
839
                }
840
                $pType = $mSig[$i+1];
841
                if ($pType == 'i4' || $pType == 'i8' || $pType == 'int' || $pType == 'boolean' || $pType == 'double' ||
842
                    $pType == 'string' || $pType == 'dateTime.iso8601' || $pType == 'base64' || $pType == 'null'
843
                ) {
844
                    // by building directly xmlrpc values when type is known and scalar (instead of encode() calls),
845
                    // we make sure to honour the xmlrpc signature
846
                    $xmlrpcArgs[] = new $valueClass($arg, $pType);
847
                } else {
848
                    $xmlrpcArgs[] = $encoder->encode($arg, $encodeOptions);
849
                }
850
            }
851
852
            $req = new $reqClass($methodName, $xmlrpcArgs);
853
            // use this to get the maximum decoding flexibility
854
            $clientClone->return_type = 'xmlrpcvals';
855
            $resp = $clientClone->send($req, $timeout, $protocol);
856
            if ($resp->faultcode()) {
857
                if ($decodeFault) {
858
                    if (is_string($faultResponse) && ((strpos($faultResponse, '%faultCode%') !== false) ||
859
                            (strpos($faultResponse, '%faultString%') !== false))) {
860
                        $faultResponse = str_replace(array('%faultCode%', '%faultString%'),
861
                            array($resp->faultCode(), $resp->faultString()), $faultResponse);
0 ignored issues
show
Bug introduced by
The variable $faultResponse does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
862
                    }
863
                    return $faultResponse;
864
                } else {
865
                    return $resp;
866
                }
867
            } else {
868
                return $encoder->decode($resp->value(), $decodeOptions);
869
            }
870
        };
871
872
        return $function;
873
    }
874
875
    /**
876
     * @param Client $client
877
     * @param string $methodName
878
     * @param array $extraOptions
879
     * @param string $newFuncName
880
     * @param array $mSig
881
     * @param string $mDesc
882
     * @return array
883
     */
884
    public function buildWrapMethodSource($client, $methodName, array $extraOptions, $newFuncName, $mSig, $mDesc='')
885
    {
886
        $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
887
        $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
888
        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool)$extraOptions['encode_php_objs'] : false;
889
        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool)$extraOptions['decode_php_objs'] : false;
890
        $clientCopyMode = isset($extraOptions['simple_client_copy']) ? (int)($extraOptions['simple_client_copy']) : 0;
891
        $prefix = isset($extraOptions['prefix']) ? $extraOptions['prefix'] : 'xmlrpc';
892 View Code Duplication
        if (isset($extraOptions['return_on_fault'])) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
893
            $decodeFault = true;
894
            $faultResponse = $extraOptions['return_on_fault'];
895
        } else {
896
            $decodeFault = false;
897
            $faultResponse = '';
898
        }
899
900
        $namespace = '\\PhpXmlRpc\\';
901
902
        $code = "function $newFuncName (";
903
        if ($clientCopyMode < 2) {
904
            // client copy mode 0 or 1 == full / partial client copy in emitted code
905
            $verbatimClientCopy = !$clientCopyMode;
906
            $innerCode = $this->buildClientWrapperCode($client, $verbatimClientCopy, $prefix, $namespace);
907
            $innerCode .= "\$client->setDebug(\$debug);\n";
908
            $this_ = '';
909
        } else {
910
            // client copy mode 2 == no client copy in emitted code
911
            $innerCode = '';
912
            $this_ = 'this->';
913
        }
914
        $innerCode .= "\$req = new {$namespace}Request('$methodName');\n";
915
916
        if ($mDesc != '') {
917
            // take care that PHP comment is not terminated unwillingly by method description
918
            $mDesc = "/**\n* " . str_replace('*/', '* /', $mDesc) . "\n";
919
        } else {
920
            $mDesc = "/**\nFunction $newFuncName\n";
921
        }
922
923
        // param parsing
924
        $innerCode .= "\$encoder = new {$namespace}Encoder();\n";
925
        $plist = array();
926
        $pCount = count($mSig);
927 View Code Duplication
        for ($i = 1; $i < $pCount; $i++) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
928
            $plist[] = "\$p$i";
929
            $pType = $mSig[$i];
930
            if ($pType == 'i4' || $pType == 'i8' || $pType == 'int' || $pType == 'boolean' || $pType == 'double' ||
931
                $pType == 'string' || $pType == 'dateTime.iso8601' || $pType == 'base64' || $pType == 'null'
932
            ) {
933
                // only build directly xmlrpc values when type is known and scalar
934
                $innerCode .= "\$p$i = new {$namespace}Value(\$p$i, '$pType');\n";
935
            } else {
936
                if ($encodePhpObjects) {
937
                    $innerCode .= "\$p$i = \$encoder->encode(\$p$i, array('encode_php_objs'));\n";
938
                } else {
939
                    $innerCode .= "\$p$i = \$encoder->encode(\$p$i);\n";
940
                }
941
            }
942
            $innerCode .= "\$req->addparam(\$p$i);\n";
943
            $mDesc .= '* @param ' . $this->xmlrpc2PhpType($pType) . " \$p$i\n";
944
        }
945
        if ($clientCopyMode < 2) {
946
            $plist[] = '$debug=0';
947
            $mDesc .= "* @param int \$debug when 1 (or 2) will enable debugging of the underlying {$prefix} call (defaults to 0)\n";
948
        }
949
        $plist = implode(', ', $plist);
950
        $mDesc .= '* @return ' . $this->xmlrpc2PhpType($mSig[0]) . " (or an {$namespace}Response obj instance if call fails)\n*/\n";
951
952
        $innerCode .= "\$res = \${$this_}client->send(\$req, $timeout, '$protocol');\n";
953 View Code Duplication
        if ($decodeFault) {
0 ignored issues
show
Duplication introduced by
This code seems to be duplicated across your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
954
            if (is_string($faultResponse) && ((strpos($faultResponse, '%faultCode%') !== false) || (strpos($faultResponse, '%faultString%') !== false))) {
955
                $respCode = "str_replace(array('%faultCode%', '%faultString%'), array(\$res->faultCode(), \$res->faultString()), '" . str_replace("'", "''", $faultResponse) . "')";
956
            } else {
957
                $respCode = var_export($faultResponse, true);
958
            }
959
        } else {
960
            $respCode = '$res';
961
        }
962
        if ($decodePhpObjects) {
963
            $innerCode .= "if (\$res->faultcode()) return $respCode; else return \$encoder->decode(\$res->value(), array('decode_php_objs'));";
964
        } else {
965
            $innerCode .= "if (\$res->faultcode()) return $respCode; else return \$encoder->decode(\$res->value());";
966
        }
967
968
        $code = $code . $plist . ") {\n" . $innerCode . "\n}\n";
969
970
        return array('source' => $code, 'docstring' => $mDesc);
971
    }
972
973
    /**
974
     * Similar to wrapXmlrpcMethod, but will generate a php class that wraps
975
     * all xmlrpc methods exposed by the remote server as own methods.
976
     * For more details see wrapXmlrpcMethod.
977
     *
978
     * For a slimmer alternative, see the code in demo/client/proxy.php
979
     *
980
     * Note that unlike wrapXmlrpcMethod, we always have to generate php code here. It seems that php 7 will have anon classes...
981
     *
982
     * @param Client $client the client obj all set to query the desired server
983
     * @param array $extraOptions list of options for wrapped code. See the ones from wrapXmlrpcMethod plus
984
     *              - string method_filter      regular expression
985
     *              - string new_class_name
986
     *              - string prefix
987
     *              - bool   simple_client_copy set it to true to avoid copying all properties of $client into the copy made in the new class
988
     *
989
     * @return mixed false on error, the name of the created class if all ok or an array with code, class name and comments (if the appropriatevoption is set in extra_options)
990
     */
991
    public function wrapXmlrpcServer($client, $extraOptions = array())
992
    {
993
        $methodFilter = isset($extraOptions['method_filter']) ? $extraOptions['method_filter'] : '';
994
        $timeout = isset($extraOptions['timeout']) ? (int)$extraOptions['timeout'] : 0;
995
        $protocol = isset($extraOptions['protocol']) ? $extraOptions['protocol'] : '';
996
        $newClassName = isset($extraOptions['new_class_name']) ? $extraOptions['new_class_name'] : '';
997
        $encodePhpObjects = isset($extraOptions['encode_php_objs']) ? (bool)$extraOptions['encode_php_objs'] : false;
998
        $decodePhpObjects = isset($extraOptions['decode_php_objs']) ? (bool)$extraOptions['decode_php_objs'] : false;
999
        $verbatimClientCopy = isset($extraOptions['simple_client_copy']) ? !($extraOptions['simple_client_copy']) : true;
1000
        $buildIt = isset($extraOptions['return_source']) ? !($extraOptions['return_source']) : true;
1001
        $prefix = isset($extraOptions['prefix']) ? $extraOptions['prefix'] : 'xmlrpc';
1002
        $namespace = '\\PhpXmlRpc\\';
1003
1004
        $reqClass = $namespace . 'Request';
1005
        $decoderClass = $namespace . 'Encoder';
1006
1007
        $req = new $reqClass('system.listMethods');
1008
        $response = $client->send($req, $timeout, $protocol);
1009
        if ($response->faultCode()) {
1010
            error_log('XML-RPC: ' . __METHOD__ . ': could not retrieve method list from remote server');
1011
1012
            return false;
1013
        } else {
1014
            $mList = $response->value();
1015
            if ($client->return_type != 'phpvals') {
1016
                $decoder = new $decoderClass();
1017
                $mList = $decoder->decode($mList);
1018
            }
1019
            if (!is_array($mList) || !count($mList)) {
1020
                error_log('XML-RPC: ' . __METHOD__ . ': could not retrieve meaningful method list from remote server');
1021
1022
                return false;
1023
            } else {
1024
                // pick a suitable name for the new function, avoiding collisions
1025
                if ($newClassName != '') {
1026
                    $xmlrpcClassName = $newClassName;
1027
                } else {
1028
                    $xmlrpcClassName = $prefix . '_' . preg_replace(array('/\./', '/[^a-zA-Z0-9_\x7f-\xff]/'),
1029
                            array('_', ''), $client->server) . '_client';
1030
                }
1031
                while ($buildIt && class_exists($xmlrpcClassName)) {
1032
                    $xmlrpcClassName .= 'x';
1033
                }
1034
1035
                /// @todo add function setdebug() to new class, to enable/disable debugging
1036
                $source = "class $xmlrpcClassName\n{\npublic \$client;\n\n";
1037
                $source .= "function __construct()\n{\n";
1038
                $source .= $this->buildClientWrapperCode($client, $verbatimClientCopy, $prefix, $namespace);
1039
                $source .= "\$this->client = \$client;\n}\n\n";
1040
                $opts = array(
1041
                    'return_source' => true,
1042
                    'simple_client_copy' => 2, // do not produce code to copy the client object
1043
                    'timeout' => $timeout,
1044
                    'protocol' => $protocol,
1045
                    'encode_php_objs' => $encodePhpObjects,
1046
                    'decode_php_objs' => $decodePhpObjects,
1047
                    'prefix' => $prefix,
1048
                );
1049
                /// @todo build phpdoc for class definition, too
1050
                foreach ($mList as $mName) {
1051
                    if ($methodFilter == '' || preg_match($methodFilter, $mName)) {
1052
                        // note: this will fail if server exposes 2 methods called f.e. do.something and do_something
1053
                        $opts['new_function_name'] = preg_replace(array('/\./', '/[^a-zA-Z0-9_\x7f-\xff]/'),
1054
                            array('_', ''), $mName);
1055
                        $methodWrap = $this->wrapXmlrpcMethod($client, $mName, $opts);
1056
                        if ($methodWrap) {
1057
                            if (!$buildIt) {
1058
                                $source .= $methodWrap['docstring'];
1059
                            }
1060
                            $source .= $methodWrap['source'] . "\n";
1061
                        } else {
1062
                            error_log('XML-RPC: ' . __METHOD__ . ': will not create class method to wrap remote method ' . $mName);
1063
                        }
1064
                    }
1065
                }
1066
                $source .= "}\n";
1067
                if ($buildIt) {
1068
                    $allOK = 0;
1069
                    eval($source . '$allOK=1;');
0 ignored issues
show
Coding Style introduced by
It is generally not recommended to use eval unless absolutely required.

On one hand, eval might be exploited by malicious users if they somehow manage to inject dynamic content. On the other hand, with the emergence of faster PHP runtimes like the HHVM, eval prevents some optimization that they perform.

Loading history...
1070
                    if ($allOK) {
1071
                        return $xmlrpcClassName;
1072
                    } else {
1073
                        error_log('XML-RPC: ' . __METHOD__ . ': could not create class ' . $xmlrpcClassName . ' to wrap remote server ' . $client->server);
1074
                        return false;
1075
                    }
1076
                } else {
1077
                    return array('class' => $xmlrpcClassName, 'code' => $source, 'docstring' => '');
1078
                }
1079
            }
1080
        }
1081
    }
1082
1083
    /**
1084
     * Given necessary info, generate php code that will build a client object just like the given one.
1085
     * Take care that no full checking of input parameters is done to ensure that
1086
     * valid php code is emitted.
1087
     * @param Client $client
1088
     * @param bool $verbatimClientCopy when true, copy all of the state of the client, except for 'debug' and 'return_type'
1089
     * @param string $prefix used for the return_type of the created client
1090
     * @param string $namespace
1091
     *
1092
     * @return string
1093
     */
1094 View Code Duplication
    protected function buildClientWrapperCode($client, $verbatimClientCopy, $prefix = 'xmlrpc', $namespace = '\\PhpXmlRpc\\' )
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
1095
    {
1096
        $code = "\$client = new {$namespace}Client('" . str_replace("'", "\'", $client->path) .
1097
            "', '" . str_replace("'", "\'", $client->server) . "', $client->port);\n";
1098
1099
        // copy all client fields to the client that will be generated runtime
1100
        // (this provides for future expansion or subclassing of client obj)
1101
        if ($verbatimClientCopy) {
1102
            foreach ($client as $fld => $val) {
0 ignored issues
show
Bug introduced by
The expression $client of type object<PhpXmlRpc\Client> is not traversable.
Loading history...
1103
                if ($fld != 'debug' && $fld != 'return_type') {
1104
                    $val = var_export($val, true);
1105
                    $code .= "\$client->$fld = $val;\n";
1106
                }
1107
            }
1108
        }
1109
        // only make sure that client always returns the correct data type
1110
        $code .= "\$client->return_type = '{$prefix}vals';\n";
1111
        //$code .= "\$client->setDebug(\$debug);\n";
0 ignored issues
show
Unused Code Comprehensibility introduced by
43% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
1112
        return $code;
1113
    }
1114
}
1115