WPIntell

Source evidence

[Plugin: Client Status] PHP Warnings destroy XML

Client Status · support · 2012-07-31T17:13:00+00:00

complaintsentiment
highseverity
0.96relevance
7replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

3 / 22 rows with source links

13.6% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

19 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
Knut Sparhell unresolved
In data.php the XML PI is echoed before plugins.ph p is required. This will generate a warning when others plugins try something like session_start or to set headers. Please move this echo line to below the require_once statements. There must be no output before loading WP core. May be also set PHP display_errors false to ensure the XML is not polluted with any warnings or errors coming from other plugins that should be irrelevant to producing the XML. http://wordpress.org/extend/plugins/client-status/ Thanks, I’ve added ini_set(“display_errors”, 0) as the first line in the xml file. Eric Knut, If you are interested in how 1.4 is coming, you can check out what I have here: http://plugins.trac.wordpress.org/browser/client-status/tags/1.4_BETA It is usable, but I’m still trying to figure out the client updates from the dashboard site. If the site has more than 3 updates, it finishes 3 and leaves the site in maintenance mode. Eric Thank you! Testing it …. Tried to install it in folder beta-client-status just to see that you have hard coded all the paths. Please see http://codex.wordpress.org/Function_Reference/plugin_dir_path and also http://codex.wordpress.org/Function_Reference/admin_url will probably be handy. Replaced all occurrences of client-status with beta-client-status in constants.php and go on … Fatal error: Call-time pass-by-reference has been removed in PHP 5.4 (Deprecated since PHP 5.3). Many places where functions are called with the & operator as parameter prefix. Fixed all occurrences and now it activates and looks just fine. Will take a closer look later. When I have activated the beta on a client, the server installation refuses to show any status, saying “You must enter a site URL to retrieve data”. But I have a site URL for the client, of course. Strange, since the beta actually retrieves all the statuses when the client runs version 1.3.3 (except for the new status XML-RPC on/off, as expected). I could contribute by writing tickets on Trac, but it seems I need to add a component keyword, like client-status to address this plugin? Thanks for the feedback. I’ll work on the hard coded paths and the call-time pass-by-reference errors. As for Trac, I’m fairly certain that is only for core. Best thing for now would be to keep posting in the forums. Eric Do you plan to update this plugin? Knut, I have not had any time to commit to this project in a while. I’m sorry for that. I would still like to make another release. Eric

Comments

7 shown
ericjuden 2012-08-15T16:13:00+00:00

Thanks, I’ve added ini_set(“display_errors”, 0) as the first line in the xml file. Eric

ericjuden 2012-08-15T16:47:00+00:00

Knut, If you are interested in how 1.4 is coming, you can check out what I have here: http://plugins.trac.wordpress.org/browser/client-status/tags/1.4_BETA It is usable, but I’m still trying to figure out the client updates from the dashboard site. If the site has more than 3 updates, it finishes 3 and leaves the site in maintenance mode. Eric

Knut Sparhell 2012-08-15T23:56:00+00:00

Thank you! Testing it …. Tried to install it in folder beta-client-status just to see that you have hard coded all the paths. Please see http://codex.wordpress.org/Function_Reference/plugin_dir_path and also http://codex.wordpress.org/Function_Reference/admin_url will probably be handy. Replaced all occurrences of client-status with beta-client-status in constants.php and go on … Fatal error: Call-time pass-by-reference has been removed in PHP 5.4 (Deprecated since PHP 5.3). Many places where functions are called with the & operator as parameter prefix. Fixed all occurrences and now it activates and looks just fine. Will take a closer look later.

Knut Sparhell 2012-08-16T00:21:00+00:00

When I have activated the beta on a client, the server installation refuses to show any status, saying “You must enter a site URL to retrieve data”. But I have a site URL for the client, of course. Strange, since the beta actually retrieves all the statuses when the client runs version 1.3.3 (except for the new status XML-RPC on/off, as expected). I could contribute by writing tickets on Trac, but it seems I need to add a component keyword, like client-status to address this plugin?

ericjuden 2012-08-16T13:59:00+00:00

Thanks for the feedback. I’ll work on the hard coded paths and the call-time pass-by-reference errors. As for Trac, I’m fairly certain that is only for core. Best thing for now would be to keep posting in the forums. Eric

Knut Sparhell 2013-06-27T00:17:00+00:00

Do you plan to update this plugin?

ericjuden 2013-06-29T18:20:00+00:00

Knut, I have not had any time to commit to this project in a while. I’m sorry for that. I would still like to make another release. Eric