Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.1.1
    • Fix Version/s: STABLE backlog
    • Component/s: Web Services
    • Labels:
      None
    • Testing Instructions:
      Hide

      Use the built-in web service client and try to create a user with an existing username. The error message should be explicit.

      Show
      Use the built-in web service client and try to create a user with an existing username. The error message should be explicit.
    • Affected Branches:
      MOODLE_21_STABLE
    • Pull from Repository:
      git@github.com:mouneyrac/moodle.git
    • Pull Master Branch:
    • Rank:
      18871

      Description

      use moodle_exception instead of the current not explicit invalid_param_exception

        Issue Links

          Activity

          Hide
          Jérôme Mouneyrac added a comment -

          I backed ported the improvement in 2.0 and 2.1

          Show
          Jérôme Mouneyrac added a comment - I backed ported the improvement in 2.0 and 2.1
          Hide
          Eloy Lafuente (stronk7) 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
          Eloy Lafuente (stronk7) 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
          Petr Škoda added a comment -

          Who is going to translate all these unlikely error messages? Web interface does not (and can not) return verbose error messages either, that is why we have the debug mode. The external lib should imo use the same logic! The iphone client and friends should display the returned debug info when you configure it to do that...

          my -1

          Show
          Petr Škoda added a comment - Who is going to translate all these unlikely error messages? Web interface does not (and can not) return verbose error messages either, that is why we have the debug mode. The external lib should imo use the same logic! The iphone client and friends should display the returned debug info when you configure it to do that... my -1
          Hide
          Sam Hemelryk added a comment -

          Reopening as discussed thanks Jerome. I agree with Petr here again sorry, keeping these messages generic and providing debugging information should be toggled upon configuration.

          Show
          Sam Hemelryk added a comment - Reopening as discussed thanks Jerome. I agree with Petr here again sorry, keeping these messages generic and providing debugging information should be toggled upon configuration.
          Hide
          Jérôme Mouneyrac added a comment -

          No need to be fixed as commented by Petr and Sam. Thanks for your reviews

          Show
          Jérôme Mouneyrac added a comment - No need to be fixed as commented by Petr and Sam. Thanks for your reviews
          Hide
          Jérôme Mouneyrac added a comment - - edited

          Note that when MDL-29435 is fixed, then the error message will be explicit in SOAP and XMLRPC. They already are explicit with the REST servers.

          Show
          Jérôme Mouneyrac added a comment - - edited Note that when MDL-29435 is fixed, then the error message will be explicit in SOAP and XMLRPC. They already are explicit with the REST servers.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: