Correct the logic in extract_from_markers()

[stodorovic]

I created simple PHP script to test function extract_from_markers:

<?php
include_once( __DIR__.'/wp-load.php' );
require_once(ABSPATH . 'wp-admin/includes/admin.php');

$wprules = extract_from_markers( ABSPATH.'.htaccess', 'WordPress' );

var_dump( $wprules );

Result (WP 4.9):

array(0) {
}

Result (WP 4.8.3):

array(9) {
  [0] =>
  string(24) "<IfModule mod_rewrite.c>"
  [1] =>
  string(16) "RewriteEngine On"
  [2] =>
  string(13) "RewriteBase /"
  [3] =>
  string(30) "RewriteRule ^index\.php$ - [L]"
  [4] =>
  string(35) "RewriteCond %{REQUEST_FILENAME} !-f"
  [5] =>
  string(35) "RewriteCond %{REQUEST_FILENAME} !-d"
  [6] =>
  string(28) "RewriteRule . /index.php [L]"
  [7] =>
  string(11) "</IfModule>"
  [8] =>
  string(0) ""
}

It seems that's bug and correct code should be (function extract_from_markers):

        foreach ( $markerdata as $markerline ) {
                if ( false !== strpos( $markerline, '# END ' . $marker ) ) {
                        $state = false;  
                }
                if ( $state ) {
                        $result[] = $markerline;
                }
                if ( false !== strpos( $markerline, '# BEGIN ' . $marker ) ) {
                                $state = true;
                }
        }

Related topic: ​https://wordpress.org/support/topic/rewrite-rules-must-be-updated-message-on-wordpress-4-9/

Thanks in advance,
Sasa

[SergeyBiryukov]

In 42199:

Rewrite Rules: Correct the logic in extract_from_markers() after [41928].

Props stodorovic.
Fixes #42579. See #39920.

[SergeyBiryukov]

Reopening for 4.9 branch.

[dd32]

In 42213:

Rewrite Rules: Correct the logic in extract_from_markers() after [41928].

Props stodorovic, SergeyBiryukov.
Merges [42199] to the 4.9 branch.
Fixes #42579. See #39920.

[dd32]

#42657 was marked as a duplicate.