Have added following to the gerrit change comments There is a very remote chance that some people are still using 0.6, and there is a very remote chance that those people don't have a session dir.
Should we add in an upgrade script to make it, or just assume that everybody is using a current version so will have session dir at install time. I also think we don't need to bother caring whether people delete the session dir, it would be their own fault, so they should have to manually create it again. I'm sure there are other folders that would be assumed to be there since install (like the actual dataroot itself) If no-one complains, will merge. Cheers, Hugh -- You received this bug notification because you are a member of Mahara Core, 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 _______________________________________________ Mailing list: https://launchpad.net/~mahara-core Post to : [email protected] Unsubscribe : https://launchpad.net/~mahara-core More help : https://help.launchpad.net/ListHelp
