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

web service server die without helping the developer

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.6, 2.2.3
    • Fix Version/s: 2.1.7, 2.2.4
    • Component/s: Web Services
    • Labels:

      Description

      For each web service server.php:
      before that the server die() because the web services or the protocol are not enable, add:
      if debugging()

      { error_log(print_r('The server die because web service or protocol are not enable', true)); }

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              jerome Jérôme Mouneyrac added a comment -

              You can cherry-pick in 2.2 and 2.1, it should not conflict and it's nice for the devs.

              Show
              jerome Jérôme Mouneyrac added a comment - You can cherry-pick in 2.2 and 2.1, it should not conflict and it's nice for the devs.
              Hide
              stronk7 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
              stronk7 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
              poltawski Dan Poltawski added a comment -

              Thanks Jerome, i've integrated this now. I agree this is useful for web service developers.

              Show
              poltawski Dan Poltawski added a comment - Thanks Jerome, i've integrated this now. I agree this is useful for web service developers.
              Hide
              phalacee Jason Fowler added a comment -

              works fine - error message makes sense too

              Show
              phalacee Jason Fowler added a comment - works fine - error message makes sense too
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

              Thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks!

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    9/Jul/12