Ver Fonte

Add FIXME

pdontthink há 17 anos atrás
pai
commit
cdaebdad32
1 ficheiros alterados com 1 adições e 0 exclusões
  1. 1 0
      templates/default_advanced/js/dtree.js

+ 1 - 0
templates/default_advanced/js/dtree.js

@@ -289,6 +289,7 @@ dTree.prototype.clearCookie = function() {
 };
 
 // [Cookie] Sets value in a cookie
+// FIXME: although setCookie() supports the secure flag, it isn't used when called anywhere in this file; ideally it should correspond to how the flag is determined in the SM core in sqsetcookie() (including the admin config that turns the secure flag off)... also, would be good to add the HTTP only flag
 dTree.prototype.setCookie = function(cookieName, cookieValue, expires, path, domain, secure) {
 	document.cookie =
 		escape(cookieName) + '=' + escape(cookieValue)