Hi,

As I've needed to use the others cookies settings with your module, I've added the support of all cookie settings in the admin form. Now you can define:

  • The cookie path.
  • The cookie domain.
  • If the language cookie use the secure mode.
  • If the language cookie use the HTTP only mode.

I've created a patch (#1) to add these new features.
Enjoy it.

CommentFileSizeAuthor
#4 locale-cookie_adding_of_all_settable_settings_for_a_cookie-2074991-4.patch11.69 KBBabou
#3 locale-cookie_adding_of_all_settable_settings_for_a_cookie-2074991-3.patch11.88 KBBabou
#2 Patch to add of all settable settings for a cookie.5.06 KBEchofive
#1 Patch to add of all settable settings for a cookie.5.09 KBEchofive

Comments

Echofive’s picture

Echofive commented
StatusFileSize
new Patch to add of all settable settings for a cookie.5.09 KB

This is the patch...

Echofive’s picture

Echofive commented
StatusFileSize
new Patch to add of all settable settings for a cookie.5.06 KB

Little corrections into the default variable values.

Babou’s picture

Babou commented
StatusFileSize
new locale-cookie_adding_of_all_settable_settings_for_a_cookie-2074991-3.patch11.88 KB

This patch should work when using drush make or when patching from the terminal.

Babou’s picture

Babou commented
StatusFileSize
new locale-cookie_adding_of_all_settable_settings_for_a_cookie-2074991-4.patch11.69 KB

New attempt for drush make without windows line endings

bart.hanssens’s picture

bart.hanssens commented
Status: Needs review » Needs work

Not sure what happened here, but the patch in #4 seems to replace complete files, instead of just a patch...