By using this site, you agree to our updated Privacy Policy and our Terms of Use. Manage your Cookies Settings.
437,605 Members | 2,153 Online
Bytes IT Community
+ Ask a Question
Need help? Post your question and get tips & solutions from a community of 437,605 IT Pros & Developers. It's quick & easy.

Trying to set a cookie onto self.parent.window.opener

P: n/a
I thought this would work but it seems to not work neither in Netscape
nor in IE:

Expand|Select|Wrap|Line Numbers
  1. <script type="text/javascript">
  2. <!--
  3.  
  4. // OBTAINED FROM http://www.javascripter.net/faq/settinga.htm
  5.  
  6. // NOTE THAT IF YOU SET days TO -1 THE COOKIE WILL BE SET TO YESTERDAY
  7. AND THUS EXPIRE
  8. function setCookie(name, value, days) {
  9. var today = new Date();
  10. var expire = new Date();
  11. if (days == null || isNaN(days) || days == 0) days = 1;
  12. if (days >= 1 || days < 0) expire.setTime(today.getTime() + 3600000 *
  13. 24 * days);
  14. document.cookie = name + '=' + escape(value) + ';expires=' +
  15. expire.toGMTString();
  16. }
  17.  
  18.  
  19. setCookie('val_chat', '**DELETE**', -1);
  20. self.parent.window.opener.document.writeln(document.cookie); //
  21. DELETE PARENT WINDOW'S COOKIE
  22. self.parent.window.opener.location.href = 'chat.jsp';
  23. self.parent.close();
  24. //-->
  25. </script>
  26. <noscript><!-- NO EQUIVALENT --></noscript>
  27.  
However, instead of the expired cookie being set into the frame's
parent's window.opener document, nothing is set, however, redirection
takes place but incorrectly since the cookie's instance still exists
when the window.opener is refreshed.

So how have you guys done this correctly where I have gone wrong?

Thanx
Phil

Feb 6 '06 #1
Share this Question
Share on Google+
4 Replies


P: n/a
> self.parent.window.opener.document.writeln(documen t.cookie); //
DELETE PARENT WINDOW'S COOKIE


This code write something to parent window document, but not operate
with its cookie.
Try to add extra parameter to the function "setCookie":
function setCookie(name, value, days, docCookie)
{
.....

Change:
document.cookie = name + '=' + escape(value) + ';expires='+
expire.toGMTString();

To:
docCookie.cookie = name + '=' + escape(value) + ';expires='
+expire.toGMTString();
......

And then:
setCookie('val_chat', '**DELETE**', -1,
self.parent.window.opener.document);
Maybe it works, but I don't test.

Feb 6 '06 #2

P: n/a
I'm sorry I do not understand this. :(

Phil

marss wrote:
self.parent.window.opener.document.writeln(documen t.cookie); //
DELETE PARENT WINDOW'S COOKIE


This code write something to parent window document, but not operate
with its cookie.
Try to add extra parameter to the function "setCookie":
function setCookie(name, value, days, docCookie)
{
....

Change:
document.cookie = name + '=' + escape(value) + ';expires='+
expire.toGMTString();

To:
docCookie.cookie = name + '=' + escape(value) + ';expires='
+expire.toGMTString();
.....

And then:
setCookie('val_chat', '**DELETE**', -1,
self.parent.window.opener.document);
Maybe it works, but I don't test.


Feb 6 '06 #3

P: n/a

Phil Powell wrote:
I thought this would work but it seems to not work neither in Netscape
nor in IE:

Expand|Select|Wrap|Line Numbers
  1.   <script type="text/javascript">
  2.   <!--
  3.  // OBTAINED FROM http://www.javascripter.net/faq/settinga.htm
  4.  // NOTE THAT IF YOU SET days TO -1 THE COOKIE WILL BE SET TO YESTERDAY
  5.  AND THUS EXPIRE
  6.  function setCookie(name, value, days) {
  •  
  • Add extra parameter docCookie:
  • function setCookie(name, value, days, docCookie) {
  •  
  •   var today = new Date();
  •   var expire = new Date();
  •   if (days == null || isNaN(days) || days == 0) days = 1;
  •   if (days >= 1 || days < 0) expire.setTime(today.getTime() + 3600000 *
  •  24 * days);
  •   document.cookie = name + '=' + escape(value) + ';expires=' +
  •  expire.toGMTString();
  •  
  • Set desired value to docCookie:
  • docCookie = name + '=' + escape(value) + ';expires=' +
  • expire.toGMTString();
  •  
  •  }
  •      setCookie('val_chat', '**DELETE**', -1);
  •  
  • Sorry, I was wrong here, last parameter must be
  • self.parent.window.opener.document.cookie
  • (not self.parent.window.opener.document)
  •  
  • setCookie('val_chat', '**DELETE**', -1,
  • self.parent.window.opener.document.cookie);
  •      self.parent.window.opener.document.writeln(document.cookie); //
  •  
  • Function writeln() writes content of document.cookie to
  • self.parent.window.opener.document but not set its cookie. To set
  • cookie of self.parent.window.opener.document you should do it directly:
  • self.parent.window.opener.document.cookie="...". I guess this line of
  • code is wrong.
  •  DELETE PARENT WINDOW'S COOKIE
  •      self.parent.window.opener.location.href = 'chat.jsp';
  •      self.parent.close();
  •  
  • I don't understand what are you going to achieve with this code and
  • can't comment it.
  • Sorry for my English
  •    //-->
  •   </script>
  •   <noscript><!-- NO EQUIVALENT --></noscript>
  •  

  • However, instead of the expired cookie being set into the frame's
    parent's window.opener document, nothing is set, however, redirection
    takes place but incorrectly since the cookie's instance still exists
    when the window.opener is refreshed.

    So how have you guys done this correctly where I have gone wrong?

    Thanx
    Phil

    Feb 7 '06 #4

    P: n/a
    JRS: In article <11**********************@z14g2000cwz.googlegroups .com>
    , dated Mon, 6 Feb 2006 00:50:21 remote, seen in
    news:comp.lang.javascript, Phil Powell <ph**************@gmail.com>
    posted :
    I thought this would work but it seems to not work neither in Netscape
    nor in IE:

    [code]
    <script type="text/javascript">
    <!--

    // OBTAINED FROM http://www.javascripter.net/faq/settinga.htm
    Clearly a site to be avoided.

    // NOTE THAT IF YOU SET days TO -1 THE COOKIE WILL BE SET TO YESTERDAY
    AND THUS EXPIRE
    function setCookie(name, value, days) {
    var today = new Date();
    var expire = new Date();
    if (days == null || isNaN(days) || days == 0) days = 1;
    if (days >= 1 || days < 0) expire.setTime(today.getTime() + 3600000 *
    24 * days);
    document.cookie = name + '=' + escape(value) + ';expires=' +
    expire.toGMTString();
    }
    There's no need to create two Date Objects just to get an Object set to
    24 hours ahead of now.

    To get a Date Object whose value does not matter, use new Date(0)
    which is quicker. To get a new Date Object E with the same value as a
    Date Object D, use E = new Date(+D) .

    3600000 * 24 is better written as 864e5.

    Code posted should be executable as posted. Do not allow your posting
    agent to wrap lines.

    You can advance 24 hours, but the description is in terms of days.
    Civil days do not always have 24 hours; if you describe the expiry in
    days, your users may occasionally get surprised.

    Use var T = new Date() ; T.setDate(T.getDate()+1)

    Your code seems to assume that the value of days is integer, without
    ensuring that.
    So how have you guys done this correctly where I have gone wrong?


    Your code no doubt does exactly what it should do. You have failed to
    say what you want it to do.

    It seems that days is read from a user control. If you check the
    control with a RegExp you can assure a good value; consider and test

    S = control.value
    OK = /^[+-]?\d+$/.test(S) // repeat if false?
    days = OK ? +S : 1

    I suppose that you have checked the incipient cookie string S with, say,
    alert(S); probably the error whose consequences you refer to lies
    elsewhere.

    --
    John Stockton, Surrey, UK. ?@merlyn.demon.co.uk Turnpike v4.00 IE 4
    <URL:http://www.jibbering.com/faq/> JL/RC: FAQ of news:comp.lang.javascript
    <URL:http://www.merlyn.demon.co.uk/js-index.htm> jscr maths, dates, sources.
    <URL:http://www.merlyn.demon.co.uk/> TP/BP/Delphi/jscr/&c, FAQ items, links.
    Feb 7 '06 #5

    This discussion thread is closed

    Replies have been disabled for this discussion.