Moodle
  1. Moodle
  2. MDL-31166

invalid PARAM_PATH cleanup in CLI install

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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
            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
            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
            Michael de Raadt added a comment -

            Test result: Success

            Tested under both Windows and Unix. Install worked nicely.

            Show
            Michael de Raadt added a comment - Test result: Success Tested under both Windows and Unix. Install worked nicely.
            Hide
            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
            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: