Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-25780

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

          Details

          • Type: Bug
          • Status: Closed
          • Priority: 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

                Attachments

                  Activity

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

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

                  Petr

                  Show
                  skodak Petr Skoda added a comment - Hello Daren, I have added you to the list of pull-requesters. Petr
                  Hide
                  Permalink
                  mudrd8mz David Mudrák added a comment -

                  Tested. Thanks for the patch Daren!

                  Show
                  mudrd8mz David Mudrák 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:
                        Fix Release Date:
                        21/Feb/11