Moodle
  1. Moodle
  2. MDL-35238 Automatic plugin update deployment
  3. MDL-35240

URL to download a new plugin version is not returned by download.moodle.org REST API

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: Future Dev
    • Fix Version/s: 2.4
    • Component/s: Administration
    • Labels:
    • Fixed Branches:
      MOODLE_24_STABLE
    • Rank:
      43889

      Description

      The REST web service at http://download.moodle.org/api/1.0/updates.php does not seem to provide URL to download the new plugin version. Such an URL should be returned via the 'download' property as it is for the Moodle core. Such URL is populated in local_plugins_version::get_downloadlink() method. To be able to fetch ZIPs automatically, we need this info available in Moodle clients.

        Activity

        Hide
        David Mudrak added a comment -

        Assigning to Eloy as he seems to be the Best Man for this!

        Show
        David Mudrak added a comment - Assigning to Eloy as he seems to be the Best Man for this!
        Hide
        David Mudrak added a comment -

        Note that we might want to make this changes in a new version of the API at the URL http://download.moodle.org/api/1.1/updates.php

        Show
        David Mudrak added a comment - Note that we might want to make this changes in a new version of the API at the URL http://download.moodle.org/api/1.1/updates.php
        Hide
        Eloy Lafuente (stronk7) added a comment -

        It's already working, showing fake downloadurl and downloadmd5 until moodle.org gets updates with the new version of local_plugins.

        Tell me what do you want in the case of downloadurl/downloadmd5 not being available or being empty (to replace current fake values with expected behavior).

        I know it's 99.99% unlikely... but... ciao

        Show
        Eloy Lafuente (stronk7) added a comment - It's already working, showing fake downloadurl and downloadmd5 until moodle.org gets updates with the new version of local_plugins. Tell me what do you want in the case of downloadurl/downloadmd5 not being available or being empty (to replace current fake values with expected behavior). I know it's 99.99% unlikely... but... ciao
        Hide
        David Mudrak added a comment -

        Thanks Eloy. The only thing is the 'downloadurl' property name. It should be just 'download' as it's what all 2.3 sites around there expect and support (to display the 'Download' link).

        WRT missing value. If for some reason the value of the property can't be populated, just omit it from the response.

        Show
        David Mudrak added a comment - Thanks Eloy. The only thing is the 'downloadurl' property name. It should be just 'download' as it's what all 2.3 sites around there expect and support (to display the 'Download' link). WRT missing value. If for some reason the value of the property can't be populated, just omit it from the response.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Done! Let's wait to local_plugins to be upgraded @ moodle.org

        Show
        Eloy Lafuente (stronk7) added a comment - Done! Let's wait to local_plugins to be upgraded @ moodle.org
        Hide
        Dan Poltawski added a comment -

        Taking these issues out of current integration (I don't think they are moodle issues needing action).

        Show
        Dan Poltawski added a comment - Taking these issues out of current integration (I don't think they are moodle issues needing action).
        Hide
        Dan Poltawski added a comment -

        The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

        TIA and ciao

        Show
        Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Closing as fixed. The 1.1 api is already supporting this.

        Show
        Eloy Lafuente (stronk7) added a comment - Closing as fixed. The 1.1 api is already supporting this.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: