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.
Went to this URL:
http://dev.ngphost.com/?q=admin/settings/clean-urls
Clicked "Run the clean URL test" which directed me to this URL:
http://dev.ngphost.com/admin/settings/clean-urls
Page pulls up again as expected, but radio buttons are still disabled.
Comment | File | Size | Author |
---|---|---|---|
#9 | query_string_1.patch | 875 bytes | ChrisKennedy |
#5 | query_string.patch | 986 bytes | ChrisKennedy |
#4 | patch_73.txt | 1.74 KB | webernet |
Comments
Comment #1
seanrPossible association with this:
http://cvs.drupal.org/viewcvs/drupal/drupal/includes/bootstrap.inc?r1=1....
Comment #2
ChrisKennedy CreditAttribution: ChrisKennedy commentedFrom http://api.drupal.org/api/HEAD/function/system_clean_url_settings:
Thus http://drupal.org/node/87310 is what caused it.
Comment #3
webernet CreditAttribution: webernet commentedI can confirm the problem. Marking as critical.
Comment #4
webernet CreditAttribution: webernet commentedAttached patch rolls back http://drupal.org/node/87310 until that issue can come up with a better solution.
Comment #5
ChrisKennedy CreditAttribution: ChrisKennedy commentedThe attached patch restores functionality for me on apache - please verify. It theoretically preserves the fixes from#87310 too.
Comment #6
webernet CreditAttribution: webernet commentedPatch in #5 fixes the issue for apache. Needs to be tested on IIS.
Comment #7
alanburke CreditAttribution: alanburke commentedPatch at #5 works as advertised for me.
Alan
Comment #8
chx CreditAttribution: chx commentedWorking w/ ChrisKen on a better one.
Comment #9
ChrisKennedy CreditAttribution: ChrisKennedy commentedChx and I talked out the logic and improved it. This patch also uses his proposed formatting to help with the parentheses.
Comment #10
Dries CreditAttribution: Dries commentedThat code is getting real ugly -- I think I'm in favor of a rollback.
Comment #11
ChrisKennedy CreditAttribution: ChrisKennedy commentedUsing the rollback patch in #4 works fine. Any new attempt to fix the original problem should indicate which variable tests and logic are being used for which servers/configurations.
Comment #12
Dries CreditAttribution: Dries commentedCommitted #4. Thanks.
Comment #13
(not verified) CreditAttribution: commentedComment #14
petrotool CreditAttribution: petrotool commentedare there any instructions for adding this patch? as i just get parse errors and the patches take the whole site down.
either im doing somthing wrong or it doesnt work. im hosted with streamline.net and im about to ditch them as this is more hassle than its worth