mahara-contributors team mailing list archive

[Mahara Bot <817940@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/616
Committed: http://gitorious.org/mahara/mahara/commit/8e8132fb22acabfa60f204bde595b23bd1a5e806
Submitter: Hugh Davenport (hugh@xxxxxxxxxxxxxxx)
Branch:    master

commit 8e8132fb22acabfa60f204bde595b23bd1a5e806
Author: Melissa Draper <melissa@xxxxxxxxxxxxxxx>
Date:   Fri Aug 26 17:00:43 2011 +1200

    Remove temporary session folder code (bug #817940)
    
    This patch removes the 'temporary' code that was checking for the
    existance of the session directories at each session creation. This
    check is performed at install and at update, however 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.
    
    Change-Id: Ia6020c0b73ce9e172e718c9638a312b1f0162295
    Signed-off-by: Melissa Draper <melissa@xxxxxxxxxxxxxxx>

-- 
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:
  In Progress

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