← Back to team overview

mahara-contributors team mailing list archive

[Bug 1170587] A change has been merged

 

Reviewed:  https://reviews.mahara.org/2131
Committed: http://gitorious.org/mahara/mahara/commit/2ae5cd5f267fb7f43e4c7ae33fba6b3f44092f42
Submitter: Aaron Wells (aaronw@xxxxxxxxxxxxxxx)
Branch:    master

commit 2ae5cd5f267fb7f43e4c7ae33fba6b3f44092f42
Author: Aaron Wells <aaronw@xxxxxxxxxxxxxxx>
Date:   Thu May 2 16:52:40 2013 +1200

Revert "Bug 1170587: Set mkdir() to create intermediate directories"

Replacing with a patch from Bug 1080498

This reverts commit bb099ec633f29a163b5458afefe2bfa5bdce0394

Change-Id: Ia19abc3271424c34a23a855abfff952b6a5e93eb

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contrib members
https://bugs.launchpad.net/bugs/1170587

Title:
  Problem with multi-level mkdir() in artefact/file/db/upgrade.php

Status in Mahara ePortfolio:
  Invalid
Status in Mahara 1.7 series:
  Invalid

Bug description:
  While upgrading demo.mahara.org from 1.6.4 to 1.7.0, I found the
  upgrade script errored out repeatedly, because in
  artefact/file/db/upgrade.php, in the section for version 2012092400,
  it tried to create a new directory "artefact/file/originals" under the
  dataroot. But, the base "artefact" directory itself didn't exist in my
  dataroot, and mkdir() defaults to NOT creating intermediate
  directories.

  So, I changed mkdir to create the intermediate directories, and then
  the installation went through okay.

  I'm not sure whether this is a problem other people might encounter,
  or whether it's due to the peculiarities of demo.mahara.org (which is
  reset by a script on a daily basis). But it wouldn't hurt to add it.

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


References