mahara-contributors team mailing list archive

[Mahara Bot <817940@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/900
Committed: http://gitorious.org/mahara/mahara/commit/b5277b715febfab085005cd5c7a0d35fd2650105
Submitter: Richard Mansfield (richardm@xxxxxxxxxx)
Branch:    master

commit b5277b715febfab085005cd5c7a0d35fd2650105
Author: Richard Mansfield <richard.mansfield@xxxxxxxxxxxxxxx>
Date:   Wed Nov 30 12:07:30 2011 +1300

    Revert "Remove temporary session folder code (bug #817940)"
    
    This reverts commit 8e8132fb22acabfa60f204bde595b23bd1a5e806.
    
    That commit was not ready to be merged.  As it says in the commit
    message, "if the session directory goes away somehow between updates,
    there is currently _no_ way to restore it without one of these two
    options. This may need to be considered before merging in this
    change."
    
    We can revert this revert when we have a good alternative for creating
    the session directories.
    
    Change-Id: I5141832da3225e202feef19dccabfea9ff35beeb

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/817940

Title:
  Session directories are created on every init

Status in Mahara ePortfolio:
  Fix Committed

Bug description:
  htdocs/auth/session.php has a "TEMPORARY: this will be REMOVED..." which has been in there for four years and since 0.7.0
  It's not a major issue, and it serves the job of ensuring that the session directories exist, but this should be done at install time, and should have been part of the upgrade really.

  I think that we should either remove the comment, or remove the code
  depending on what is required.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/817940/+subscriptions