The message "Testing clean URLs..." isn't removed in three out of four cases (install: success/error and settings: error). The enclosed patch fixes this (+ a minor codestyle improvement).

CommentFileSizeAuthor
cleanurl-hide-testing.patch1.78 KBpancho

Comments

pancho’s picture

pancho
Location UTC+2 🇪🇺 EU
commented

Please review.

cburschka’s picture

cburschka
they
commented
Status: Needs review » Reviewed & tested by the community

On installation:
Pre-patch: Line does not appear after testing.
Post-patch (and after clearing the browser cache to refresh the js!): Line disappears.

On installation, without the .htaccess file (to break clean URLs):
Pre-patch: Line does not appear after testing.
Post-patch: Line disappears.

On visiting ?q=admin/settings/ with the previous install (.htaccess still broken)
Pre-patch: Line does not appear after testing.
Post-patch: Line disappears.

Appears to be okay.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Version: 6.x-dev » 7.x-dev

Committed, thanks. RTBC for 7.x.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.