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.Problem/Motivation
When using Panels and Panelizer in admin mode the ajax call in Drupal.ajax.prototype.autologoutRefresh wasn't setting ajaxPageState in the request. This caused the response to override the ajaxPageState.theme and theme_token. This caused the next ajax call (usually made by interaction with Panels) to want to load any CSS or JS associated with the default theme. This cause styling conflicts when on a page using the admin theme.
Proposed resolution
Run the ajax.options through ajax.beforeSerialize to include the ajax_page_state to the ajax request.
Remaining tasks
Patch will be attached in first comment
User interface changes
None
API changes
None
| Comment | File | Size | Author |
|---|---|---|---|
| #15 | autologout-not-passing-ajax_page_state-2469177-15.patch | 288 bytes | dewalt |
| #9 | autologout-not-passing-ajax_page_state-2469177-9.patch | 464 bytes | dewalt |
| #1 | autologout-not-passing-ajax_page_state-2469177-1.patch | 288 bytes | xeraseth |
Comments
Comment #1
xeraseth CreditAttribution: xeraseth commentedAttaching patch for review
Comment #2
xeraseth CreditAttribution: xeraseth commentedComment #3
tame4tex CreditAttribution: tame4tex commentedThis patch worked for me ... also fixed another ajax - colorbox related bug
Comment #4
johnennew CreditAttribution: johnennew at Deeson commentedMany thanks for this patch - I can confirm it works as described.
Comment #6
johnennew CreditAttribution: johnennew at Deeson commentedCommitted with thanks!
Comment #7
johnennew CreditAttribution: johnennew at Deeson commentedComment #9
dewalt CreditAttribution: dewalt commentedThe patch solves issue for
autologoutGetTimeLeftcall only, the same trick should be added toautologoutRefreshcall. Full patch is attached.Comment #15
dewalt CreditAttribution: dewalt commentedUpdated patch for 7.x-4.x version. Contains not included part only.
Comment #16
dewalt CreditAttribution: dewalt commentedComment #18
johnennew CreditAttribution: johnennew at Deeson commentedCommitted, thanks!