Undefined variable: usingpi / Undefined variable: writable / Undefined variable: update_required

[andreasneuber]

Happens with 4.6-beta3-38065

Steps
1.Fresh WP
2.Login
3.Go to Settings > Permalinks
4.Press "Save Changes"
5.Check debug.log

debug.log has these entries

PHP Notice:  Undefined variable: usingpi in /var/www/html/tmp_phpnotices_60/wp-admin/options-permalink.php on line 113
PHP Stack trace:
PHP   1. {main}() /var/www/html/tmp_phpnotices_60/wp-admin/options-permalink.php:0
PHP Notice:  Undefined variable: writable in /var/www/html/tmp_phpnotices_60/wp-admin/options-permalink.php on line 113
PHP Stack trace:
PHP   1. {main}() /var/www/html/tmp_phpnotices_60/wp-admin/options-permalink.php:0
PHP Notice:  Undefined variable: update_required in /var/www/html/tmp_phpnotices_60/wp-admin/options-permalink.php on line 113
PHP Stack trace:
PHP   1. {main}() /var/www/html/tmp_phpnotices_60/wp-admin/options-permalink.php:0

My environment
-Apache/2.4.7 (Ubuntu)
-PHP version: 7.0.8-4 / Apache 2.0 Handler

[Clorith]

Hi there, and thanks for the report!

This is indeed introduced in 4.6-beta3, with revision r37747

The attached patch moves the definition of $usingpi to be available before the IF block.

[Clorith]

Moves a larger block up as $write and $update_required also need to be available sooner

[SergeyBiryukov]

In 38066:

Permalinks: After [37747], make sure $usingpi, $writable, and $update_required are defined before checking them on permalinks update.

Props Clorith.
Fixes #37380.

[SergeyBiryukov]

In 38067:

Permalinks: Rename $usingpi to $using_index_permalinks for clarity.

See #37380.