1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Make "Remember Me" Cookie Duration An Option

Discussion in 'XenForo Suggestions' started by digitalpoint, Jun 25, 2012.

  1. digitalpoint

    digitalpoint Well-Known Member

    Having this hard coded to 7 days is a but obnoxious. I changed the behavior to 180 days in my setup, but it seems like something that should just be a setting.
     
  2. Chris D

    Chris D XenForo Developer Staff Member

    Where is this code, exactly?

    I'll see if this can be made into an add-on with configurable days.
     
  3. digitalpoint

    digitalpoint Well-Known Member

    It can... I already did it. Just seems like it should be a core thing. :)
     
  4. Chris D

    Chris D XenForo Developer Staff Member

    Sorry I thought you'd posted in the Request forum ;)

    I agree it should be core.
     
  5. Digital Doctor

    Digital Doctor Well-Known Member

    + core.
    +1 great thread.
     
  6. Andy.N

    Andy.N Well-Known Member

    Shawn,
    Where in the code we can change this cookie duration?
    Thanks
     
  7. digitalpoint

    digitalpoint Well-Known Member

    It's in XenForo_Model_User::setUserRememberCookie()
     
    Andy.N and melbo like this.
  8. Andy.N

    Andy.N Well-Known Member

    Changed
    Code:
    XenForo_Helper_Cookie::setCookie('user', $value, 7 * 86400, true);
    to
    Code:
    XenForo_Helper_Cookie::setCookie('user', $value, 180 * 86400, true);
    Thanks Shawn
     
  9. ragtek

    ragtek Guest

    Even i don't think we'll see any xenforo upgrades in 2012, you shouldn't edit the xenforo core files:D

    interested in an addon?
     

    Attached Files:

    FredC and Andy.N like this.
  10. Digital Doctor

    Digital Doctor Well-Known Member

    [ ] Remember me (for 7 days max) = alzheimer's setting.
     
  11. Andy.N

    Andy.N Well-Known Member

    of course. Bring it on!!!
     
  12. ragtek

    ragtek Guest

  13. Andy.N

    Andy.N Well-Known Member

Share This Page