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

          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.

            Gliffy Diagrams

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              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
              Permalink
              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
              Permalink
              Petr Skoda added a comment -

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

              Petr

              Show
              Petr Skoda added a comment - Hello Daren, I have added you to the list of pull-requesters. Petr
              Hide
              Permalink
              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: