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

Add functions to a service has a bad url

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.5, 2.2.2, 2.3
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Web Services
    • Labels:
    • Testing Instructions:
      Hide

      Log in as admin
      Navigate to External services (Settings -> Site administration -> Plugins -> web services -> External services)
      Add new external service and save
      Click on functions and then click on "Add functions"
      Try adding some functions and see you don't see any error.

      Show
      Log in as admin Navigate to External services (Settings -> Site administration -> Plugins -> web services -> External services) Add new external service and save Click on functions and then click on "Add functions" Try adding some functions and see you don't see any error.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:

      Description

      When adding functions to a web service client I got redirected to a non-existant page

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            poltawski Dan Poltawski added a comment -

            Hi Jerome,

            Please could you peer review this..

            Show
            poltawski Dan Poltawski added a comment - Hi Jerome, Please could you peer review this..
            Hide
            jerome Jérôme Mouneyrac added a comment -

            It's a bit of a tangential decision: +1

            Show
            jerome Jérôme Mouneyrac added a comment - It's a bit of a tangential decision: +1
            Hide
            jerome Jérôme Mouneyrac added a comment -

            cherry-pick on 2.1/2.2 too, I checked it's in these branch too. Thanks for catching it Dan.

            Show
            jerome Jérôme Mouneyrac added a comment - cherry-pick on 2.1/2.2 too, I checked it's in these branch too. Thanks for catching it Dan.
            Hide
            poltawski Dan Poltawski added a comment -

            Shamelessly stolen testing instructions from linked issue.

            Show
            poltawski Dan Poltawski added a comment - Shamelessly stolen testing instructions from linked issue.
            Hide
            poltawski Dan Poltawski added a comment -

            (sorry for missing them before)

            Show
            poltawski Dan Poltawski added a comment - (sorry for missing them before)
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (21, 22 & master). And passed. Thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (21, 22 & master). And passed. Thanks!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            UPDATE tracker_issues
               SET status = 'Closed',
                  comment = 'Thanks!'
            WHEN participants = 'Did a gorgeous work'

            This landed upstream some hours ago (some - me - developer fell slept in the sofa yesterday before spamming this).

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - UPDATE tracker_issues SET status = 'Closed', comment = 'Thanks!' WHEN participants = 'Did a gorgeous work' This landed upstream some hours ago (some - me - developer fell slept in the sofa yesterday before spamming this).

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/12