Opened 8 years ago
Closed 4 years ago
#39547 closed defect (bug) (fixed)
options-permalink.php throws incorrect "You should update..." notice on multisite
Reported by: | boonebgorges | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.6 | Priority: | normal |
Severity: | normal | Version: | 4.6 |
Component: | Permalinks | Keywords: | has-patch needs-refresh |
Focuses: | administration, multisite | Cc: |
Description
Setup:
- Subdirectory multisite installation on Apache with mod_rewrite
/.htaccess/
and/
should *not* be writable by the webserver- On a secondary site (NOT blog 1), Dashboard > Options > Permalinks, ensure that you have pretty permalinks enabled (not index.php, not default)
- Click Save Changes without making any modifications
- You should see a green notice "You should update your .htaccess file now"
Prior to #37380, saving existing settings would result in a success notice. However, #37380 just fixed a bug in the implementation of [37747], so I think the bug comes from #35936.
Prior to [37747], the "You should update..." family of notices was never shown when is_multisite()
. (Multisite routing is taken care of by WordPress, not by the webserver; WP's generated RewriteBase
etc are incorrect in multisite.) I think that the fix here is to reintroduce the is_multisite()
check.
Attachments (1)
Change History (9)
Note: See
TracTickets for help on using
tickets.
See 39547.diff. Ping @swissspidy for another set of eyes.