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:
    • Rank:
      37608

      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.

        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: