Viewing 9 replies - 1 through 9 (of 9 total)
  • Plugin Author Ella

    (@ellatrix)

    Which version of WordPress are you running? Could you var_dump( $GLOBALS[‘wp_version’] );? If you get this error it means it doesn’t have a minor version number, which is weird…

    Thread Starter timbr

    (@brugman)

    That var outputs 3.8.1, but I’m also running a security plugin which might obscure the version number when obtained through a WP function.

    Plugin Author Ella

    (@ellatrix)

    It’s not obtained through a function… Could you var_dump it just were the plugin is giving a notice? Sorry, I really need to know why there is unexpected output there. I’m also guessing that when you go to your login page, there’s something wrong with the stylesheets? You’re running rwl-login-3.9.php instead of rwl-login-3.8.php.
    I’d be really grateful if you could check, because this is pretty critical for future WP versions…

    Thread Starter timbr

    (@brugman)

    No problem. I can confirm the security plugin hides up the version when pre-printed from within the plugin. This is just me F5ing: https://i.imgur.com/X076sLX.png Could you make it work with a fake version number?

    Plugin Author Ella

    (@ellatrix)

    That’s bad. No plugin should change that. Unfortunately this plugin requires 3.8 and uses different files based on the version. If your security plugin changes it to a version lower than 3, this plugin will not work at all because of

    if ( version_compare( $wp_version, '3.8', '<' ) ) {
    
    	return;
    
    }
    Thread Starter timbr

    (@brugman)

    It’s a feature of Better WP Security (soon iThemes Security), they warn it might break plugins so I can’t fault either of you but I value the vulnerability it patches.

    To make rename wp-login compatible I would check if there is a sub-version, if not, disregard the version and assume its running the latest WordPress.

    Plugin Author Ella

    (@ellatrix)

    Not easy to do. I added 3.9 compatibility before 3.9 gets released. So the latest version would be 3.9, but you’re running 3.8. So I’d have to manually change the version on the date of the release or after (will cause breakage), or guess the date and time WordPress upgrades itself (impossible). All I can say it’s a very bad idea to change the version number. A lot of things rely on it, even in core. If you want to hide the version, you should just make sure it’s not revealed on the front-end. You should hide the generator tag and query strings attached to scripts.

    Plugin Author Ella

    (@ellatrix)

    I’ll fix this problem, but by fixing it, it will be unusable if you change the version. I’ll have to require a minor version number.

    Thread Starter timbr

    (@brugman)

    I see, I didn’t think about the 3.9 part. For now I’ve disabled the security feature. Thanks for your very speedy help. ??

Viewing 9 replies - 1 through 9 (of 9 total)
  • The topic ‘Tiny critical error’ is closed to new replies.