I don't think the cookie mode ever worked right, as it used an uninitialized variable for the $expire.

Here's a version that seems to work fine.

I've changed it to either set a cookie with expiration near the end of time() or to remove the cookie. The correct button shows in each case.

In the process I cleaned up some out-of-date comments.

CommentFileSizeAuthor
#2 kiosk.cookie_improvements_783554_02.patch5.69 KBrfay
#1 kiosk.cookie_improvements_783554_01.patch4.06 KBrfay
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented
FileSize
kiosk.cookie_improvements_783554_01.patch4.06 KB

Here's the patch. I don't know why they seem to get dropped so often lately.

rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented
FileSize
kiosk.cookie_improvements_783554_02.patch5.69 KB

Here's an improvement: Cookie deletion wasn't working quite right.