search

alooi14 / editthiscookie

Automatically exported from code.google.com/p/editthiscookie
0 stars 0 forks source link

Suggestion: Small Code changes #18

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
/trunk/js/cookies.js
Line 73 to 85
    var name            =  $(".name",           newCookie ).attr("value");
    var value           =  $(".value",          newCookie ).attr("value");
    var secure          =  $(".secure",         newCookie ).attr("checked");
    var httpOnly        =  $(".httpOnly",       newCookie ).attr("checked");
    var session         =  $(".session",        newCookie ).attr("checked");

    var expirationDate  =  $(".exp",            newCookie ).attr("value");
    var expirationDay   =  $(".expDay",         newCookie ).attr("value");
    var expirationMonth =  $(".expMonth",       newCookie ).attr("value");
    var expirationYear  =  $(".expYear",        newCookie ).attr("value");
    var expirationHour  =  $(".expHour",        newCookie ).attr("value");
    var expirationMin   =  $(".expMinutes",     newCookie ).attr("value");
    var expirationSec   =  $(".expSeconds",     newCookie ).attr("value");

Change to

    var name            =  $(".name",           newCookie ).val();
    var value           =  $(".value",          newCookie ).val();
    var secure          =  $(".secure",         newCookie ).is(':checked');  //to test
    var httpOnly        =  $(".httpOnly",       newCookie ).is(':checked');  //to test
    var session         =  $(".session",        newCookie ).is(':checked');  //to test

    var expirationDate  =  $(".exp",            newCookie ).val();
    var expirationDay   =  $(".expDay",         newCookie ).val();
    var expirationMonth =  $(".expMonth",       newCookie ).val();
    var expirationYear  =  $(".expYear",        newCookie ).val();
    var expirationHour  =  $(".expHour",        newCookie ).val();
    var expirationMin   =  $(".expMinutes",     newCookie ).val();
    var expirationSec   =  $(".expSeconds",     newCookie ).val();

Original issue reported on code.google.com by debn...@gmail.com on 24 Jun 2011 at 3:13

GoogleCodeExporter commented 8 years ago

Original comment by fcap...@gmail.com on 18 Jul 2011 at 4:50