Moodle
  1. Moodle
  2. MDL-25780

php notice displayed when moodle installer can't write to the data directory

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0.2
    • Component/s: Installation
    • Labels:
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Rank:
      15569

      Description

      Installing Moodle and Moodle doesnt currently have write permission to the data directory. I get the following php notice displayed.

      Notice: Undefined variable: a in /home/andrew/Desktop/review/moodle/install.php on line 320
      Call Stack

      1. Time Memory Function Location
        1 0.0021 443200 {main}

        ( ) ../install.php:0

      Under the data directory text box is the following message:
      Data directory ({$a->dataroot}) cannot be created by the installer.

      The $a isnt being substituted.

        Activity

        Hide
        Daren Chandisingh added a comment -

        This is fixed and avilable from the LUNS public repository:

        https://git.luns.net.uk/?p=moodle.git;a=shortlog;h=refs/heads/MDL-25780

        Show
        Daren Chandisingh added a comment - This is fixed and avilable from the LUNS public repository: https://git.luns.net.uk/?p=moodle.git;a=shortlog;h=refs/heads/MDL-25780
        Hide
        Andrew Davis added a comment -

        Thanks Daren. Its 5:40pm on Friday here right now. On Monday morning I'll give this a look then open a pull request to get this integrated.

        Show
        Andrew Davis added a comment - Thanks Daren. Its 5:40pm on Friday here right now. On Monday morning I'll give this a look then open a pull request to get this integrated.
        Hide
        Petr Škoda added a comment -

        Hello Daren,
        I have added you to the list of pull-requesters.

        Petr

        Show
        Petr Škoda added a comment - Hello Daren, I have added you to the list of pull-requesters. Petr
        Hide
        David Mudrak added a comment -

        Tested. Thanks for the patch Daren!

        Show
        David Mudrak added a comment - Tested. Thanks for the patch Daren!

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: