If you have loaded an admin page in the full browser window (i.e. not in the overlay), and you click on a link to another admin page, the overlay will flash briefly before the next page is loading. This behaviour is not there if only the overlay module and not the overlay_paths modules is enabled.

The bug is in this line:

  $return = drupal_match_path($path, overlay_paths_get_overlay_patterns()) || (path_is_admin($path) && !overlay_paths_get_overlay_patterns(FALSE));

I suppose the intentions is to match $path against the patterns returned by overlay_paths_get_overlay_patterns().

CommentFileSizeAuthor
overlay-paths-1.patch741 bytesc960657
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

torotil’s picture

torotil CreditAttribution: torotil commented
Status: Needs review » Reviewed & tested by the community

I've tracked down the bug too and found the same solution. The patch works as expected. Thanks for providing it.

Steven Jones’s picture

Steven Jones CreditAttribution: Steven Jones commented
Status: Reviewed & tested by the community » Fixed

Thanks very much for the bug report, and the patch! Committed to 7.x-1.x

Status: Fixed » Closed (fixed)

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