Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
When clean URLs are set to "off" (on IIS), I see no overlay on node/x/edit and other places where it should appear. Instead, the full admin page is shown.
The moment clean URLs are enabled, Overlay works again.
Comment | File | Size | Author |
---|---|---|---|
#10 | overlay_indexphp_no_capture.patch | 822 bytes | Heine |
#3 | overlaycleanurl.patch | 776 bytes | casey |
Comments
Comment #1
casey CreditAttribution: casey commentedCan't reproduce. Which browser/OS are you using?
Comment #2
Heine CreditAttribution: Heine commentedIE 8, Chrome 4, Opera 10.5 (alpha) and Firefox 3.5.7 on Windows 7 with IIS 7.5 as backend.
I've tested with both /drupal-7.0-alpha1/ and root (/) as basepath and in both cases, clean urls disabled, means no proper overlay.
When clean URLs are off, index.php ends up being used in urls (due to IIS detection code?):
Comment #3
casey CreditAttribution: casey commentedAhh it was the index.php. This should do.
Comment #4
casey CreditAttribution: casey commentedComment #5
Heine CreditAttribution: Heine commentedAnd so it does, but is there any reason for the backreference? Wouldn't (?:index.php)? suffice there?
Comment #6
casey CreditAttribution: casey commentedWhere do you see a backreference?
Comment #7
aspilicious CreditAttribution: aspilicious commentedI think Heine wonna change
(Drupal.settings.basePath + "(index.php)?")
to
(?:index.php)?
Comment #8
casey CreditAttribution: casey commentedThat's no backreference and it also can't be changed into that.
Comment #9
aspilicious CreditAttribution: aspilicious commentedLol, ok then Heine needs to explain ;)
Comment #10
Heine CreditAttribution: Heine commentedSorry, capture not backreference:
But the non capture pattern ?: is not well supported in older browsers.
Comment #11
casey CreditAttribution: casey commented???? Why do you need a non-capture modifier?
Comment #12
casey CreditAttribution: casey commentedComment #13
mfer CreditAttribution: mfer commentedI ran some tests and the non-capture is ever so slightly faster. Since we aren't using the back reference we might as well keep the ?:.
Comment #14
webchickI can't tell from mfer's comment whether this still needs work or not. Splitting the difference at 'needs review' :)
Comment #15
casey CreditAttribution: casey commentedMe neither... mfer?
Comment #16
bgm CreditAttribution: bgm commentedI have run into this issue, but the patch in #10 did not solve the problem for me (I did many refresh/cache flush to make sure that the patched JS was loaded in the browser).
Comment #17
casey CreditAttribution: casey commented@bgm According to @Heine in #5 and me the patch makes it work. What browser/OS are you using?
Let's commit patch in #10. It's at least an improvement.
Comment #18
webchickSounds good. Committed to HEAD.
If this is still an issue, please post back with steps to reproduce.