View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
18335 | Bug reports | RemoteControl | public | 2022-09-06 22:33 | 2022-09-26 16:45 |
Reporter | jared.kail | Assigned To | |||
Priority | none | Severity | minor | ||
Status | closed | Resolution | no change required | ||
Product Version | 5.3.x | ||||
Summary | 18335: jsonrpcphp client constructor not honoring debug | ||||
Description | $myJSONRPCClient = new \org\jsonrpcphp\JsonRPCClient( LS_RPCURL, true) does not turn on debugging. Note: public function __construct($url, $debug = false, $proxy = null) I believe this should be changed to be: public function __construct($url, $debug = false, $proxy = null) | ||||
Steps To Reproduce | Steps to reproduce$myJSONRPCClient = new \org\jsonrpcphp\JsonRPCClient( LS_RPCURL, true); Expected resultOutput should include debug info for RPC client; none produced currently Actual result(Write here what happened instead) | ||||
Tags | No tags attached. | ||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | weberhofer/jsonrpcphp 2.2.2 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | n/a | ||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | 7.4 | ||||
Thanks! Can you create a PR so we can review it and merge it? |
|
But : we didn't use in LimeSurvey core JsonRPCClient … |
|
Strange. But that code seems outdated. Look at this. |
|
FYI the version I have was from composer: { |
|
To be reported at this repo : https://github.com/weberhofer/jsonrpcphp Already reported by @mbischof |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2022-09-06 22:33 | jared.kail | New Issue | |
2022-09-23 23:56 | gabrieljenik | Status | new => acknowledged |
2022-09-23 23:56 | gabrieljenik | Note Added: 71928 | |
2022-09-23 23:56 | gabrieljenik | Bug heat | 0 => 2 |
2022-09-24 17:05 | DenisChenu | Note Added: 71931 | |
2022-09-24 17:05 | DenisChenu | Bug heat | 2 => 4 |
2022-09-26 15:06 | gabrieljenik | Note Added: 71943 | |
2022-09-26 16:43 | jared.kail | Note Added: 71945 | |
2022-09-26 16:43 | jared.kail | Bug heat | 4 => 6 |
2022-09-26 16:45 | DenisChenu | Note Added: 71946 | |
2022-09-26 16:45 | DenisChenu | Status | acknowledged => closed |
2022-09-26 16:45 | DenisChenu | Resolution | open => no change required |