WordPress.org

1. michelwp
   Member
   Posted 1 year ago #

   Good plugin BUT dangerous set_include_path
   Please don’t modify the ini like here

   if( version_compare($wp_version, ‘2.5′, ‘>=’) ) {
   /****>>>*/set_include_path( dirname(__FILE__) . PATH_SEPARATOR . get_include_path() );
   /**
   * This file holds all the author plugins functions
   */
   require_once(dirname (__FILE__) . ‘/’ . ‘authorplugins.inc.php’);
   /***>>>*/restore_include_path();
   }

   It introduce big troubles in server and another plug-in like post-notification or wp-ecommerce. By using absolute path, all is possible without disturbing the others and configurations.

   After more than three hours of research...
   Best regards

   http://wordpress.org/extend/plugins/reveal-ids-for-wp-admin-25/

2. Otto42
   Moderator
   Posted 1 year ago #

   Agreed, that line makes no sense anyway.

   Thanks for the notification, I've modified my own local copy of this plugin to eliminate that flaw. I hope the author can integrate that in soon.

3. Alphawolf
   Member
   Posted 1 year ago #

   Thanks for the notification (and the e-mail).

   I've added these two lines in order to make my plugin run on IIS servers, which a user reported via e-mail some time ago. The restore_include_path() call was meant to set the include path to default afterwards so other plugins wont be affected, however I'll fix this ASAP.

4. Alphawolf
   Member
   Posted 1 year ago #

   Version 1.0.3 fixes that issue.

Topic Closed

This topic has been closed to new replies.