WordPress.org

Support

Support » Plugins and Hacks » MAJpage Menu Class Extender » [Resolved] [PATCH] Only variables should be passed by reference

[Resolved] [PATCH] Only variables should be passed by reference

  • PHP 5.4.10 on WP 3.6 with WP_DEBUG turned on. Getting the warning:

    Strict standards: Only variables should be passed by reference in /path/to/wp-content/plugins/majpage-menu-class-extender/majpage-menu-class-extender.php on line 21

    In function majpage_mce change

    list( , $item ) = each( $xml->xpath( 'ul' ) );
    if( ! $item )
    	list( , $item ) = each( $xml->xpath( 'menu' ) );

    to

    $elems = $xml->xpath( 'ul' );
    list( , $item ) = each( $elems );
    if( ! $item ) {
    	$elems = $xml->xpath( 'menu' );
    	list( , $item ) = each( $elems );
    }

    Annooying and wasteful but it’ll stop your plugin filling up users’ error logs with warnings.

    http://wordpress.org/plugins/majpage-menu-class-extender/

Viewing 1 replies (of 1 total)
  • Plugin Author duzymaju

    @duzymaju

    Hello flynsarmy,

    Thanks a lot for your notice. Today I corrected that strict standards warning’s cause in new 1.4 plugin’s version. 🙂

    There are also a number of other changes so I will be grateful for checking new version of plugin.

    If you have other suggestions connected with this plugin, I’ll wait for your information. 😉

Viewing 1 replies (of 1 total)
  • The topic ‘[Resolved] [PATCH] Only variables should be passed by reference’ is closed to new replies.
Skip to toolbar