Support » Plugin: NS Cloner - Site Copier » Some problem with replace array

  • Resolved hanbr

    (@hanbr)


    Hi!

    Thanks for keeping this plugin updated!

    Though, I noticed some problem with the replace array. When cloning a site I get a lot of this errors:

    [08-Dec-2013 01:18:01 UTC] PHP Notice:  Undefined offset: 2 in /storage/content/19/179819/dinbrfsida.se/public_html/dev/wp-content/plugins/ns-cloner-site-copier/ns-cloner.php on line 1058
    [08-Dec-2013 01:18:01 UTC] PHP Notice:  Undefined offset: 3 in /storage/content/19/179819/dinbrfsida.se/public_html/dev/wp-content/plugins/ns-cloner-site-copier/ns-cloner.php on line 1058
    [08-Dec-2013 01:18:01 UTC] PHP Notice:  Undefined offset: 4 in /storage/content/19/179819/dinbrfsida.se/public_html/dev/wp-content/plugins/ns-cloner-site-copier/ns-cloner.php on line 1058

    I noticed that the $j variable used on line 1058 is increased inside this loop:

    foreach( $replace_array as $search_for => $replace_with) {
    
        $j++;
        ....

    Is that correct? Shouldn’t it be increased outside this loop?

    Br,
    Hans

    http://wordpress.org/plugins/ns-cloner-site-copier/

Viewing 1 replies (of 1 total)
  • Plugin Author Never Settle

    (@neversettle)

    Good eye and thanks for bringing that to our attention Hans! Sincerely appreciate your continued contribution.

    The plugin has been updated with fixes for both issues (PHP Notice and $j increment in the wrong place) as you suggest.

Viewing 1 replies (of 1 total)
  • The topic ‘Some problem with replace array’ is closed to new replies.