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.
The confirm form for restoring defaults uses drupal_goto() - this prevents custom submit handlers from running etc. patch coming up.
Comment | File | Size | Author |
---|---|---|---|
#10 | homebox_drupal_goto_2118627-10.patch | 685 bytes | ptsimard |
#1 | homebox_drupal_goto_2118627.patch | 354 bytes | catch |
Comments
Comment #1
catchComment #2
catchComment #3
drummCommitted, http://drupalcode.org/project/homebox.git/commit/d6351e2.
Comment #5
yasAfter applying this patch, hombox gives a white screen by not completing to redirect. In my site, the patch doesn't look working well. I tried to revert to "drupal_goto(homebox_get_path($page));" at my site, and it is still working, though.
Comment #6
yasChanging the state to "Needs to review".
Comment #7
drummWhat error message did you get in the logs for the white screen. See https://drupal.org/node/158043 for help.
Comment #8
yas@drumm,
According to node/158043, I inserted the debug code into index.php but error.log (and access.log) didn't give anything else. It only showed a white screen...
Actually I tried the following system environments by increasing the memory (up to 256M from 128M)
Drupal 7.24 + PHP 5.5.34-0ubuntu0.12.04.1 (Ubuntu 12.04)
and
Drupal 7.24 + PHP 5.5.34-0ubuntu2 (Ubuntu 14.04 dev)
Comment #9
ptsimard CreditAttribution: ptsimard commentedHello Drumm,
Just want to add that I too am getting a WSOD when clicking on any links under "Add a block".
There is discrepancy between this issue's title/description and the committed patch:
The patch changed homebox_add_block() instead of homebox_restore_defaults_submit
Comment #10
ptsimard CreditAttribution: ptsimard commentedHere is a new patch to revert and fix the correct intended function.
Comment #11
drummCommitted