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

invalid PARAM_PATH cleanup in CLI install

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.1
    • Fix Version/s: 2.2.2, 2.3
    • Component/s: Administration
    • Labels:

      Description

      PARAM_PATH was originally intended for relative paths created from CFG->dataroot, now it is also used in new file API.
      It is not possible to use it for full OS paths.

      This issue affects sites that use exotic characters in dataroot path. It is not compatible with Windows too, but it CLI was never supported there.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            nebgor Aparup Banerjee added a comment -

            thanks,integrated.

            just to clarify the test here: simply see that theres no issues with a CLI install (non-windows).

            Show
            nebgor Aparup Banerjee added a comment - thanks,integrated. just to clarify the test here: simply see that theres no issues with a CLI install (non-windows).
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success

            Tested under both Windows and Unix. Install worked nicely.

            Show
            salvetore Michael de Raadt added a comment - Test result: Success Tested under both Windows and Unix. Install worked nicely.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This is now available in the git and cvs repositories.

            Consider the responsibility of your fingerprints engraved there for future generations!

            Thanks for the work, closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This is now available in the git and cvs repositories. Consider the responsibility of your fingerprints engraved there for future generations! Thanks for the work, closing, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/Mar/12