WordPress.org

1. Knut Sparhell
   Member
   Posted 10 months ago #
   

   In data.php the XML PI is echoed before plugins.php 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/

2. ericjuden
   Member
   Plugin Author
   Posted 10 months ago #
   

   Thanks, I've added ini_set("display_errors", 0) as the first line in the xml file.

   Eric

3. ericjuden
   Member
   Plugin Author
   Posted 10 months ago #
   

   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

4. Knut Sparhell
   Member
   Posted 10 months ago #
   

   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.

5. Knut Sparhell
   Member
   Posted 10 months ago #
   

   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?

6. ericjuden
   Member
   Plugin Author
   Posted 10 months ago #
   

   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

Reply

You must log in to post.