Closed (fixed)
Project:
Drupal core
Version:
6.x-dev
Component:
base system
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Sep 2007 at 12:27 UTC
Updated:
9 Oct 2007 at 15:41 UTC
The Javascript cookie detection needs to add the path=/ so that when the detection is done on one page, that it is available on another page. I discovered this while writing a new module (reindex) that uses the Batch API. The Batch API was not properly detecting that I had javascript enabled because (a) the detection was done on admin/build/modules and thus the path for the cookie "has_js" was "admin/build", and (b) batch API is handled on on the path "batch", which isn't under "admin/build".
This one line patch fixes the problem.
I've marked the issue as "critical" because batch API is pretty broken without it.
| Comment | File | Size | Author |
|---|---|---|---|
| cookie-path.patch | 587 bytes | douggreen |
Comments
Comment #1
chx commentedComment #2
chx commentedAlso see http://drupal.org/node/177927 which arrived to the same solution.
Comment #3
chx commentedComment #4
gábor hojtsyLooks right, thanks, committed.
Comment #5
(not verified) commented