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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pancho’s picture

Please review.

cburschka’s picture

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

Version: 6.x-dev » 7.x-dev

Committed, thanks. RTBC for 7.x.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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