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

error when adding function to a new web service

    Details

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

      1- Create a service (Admin > Plugins > Web service > Manage service)
      2- Add few functions to service

      You never seen the error message:

      Invalid array parameter detected in required_param(): fid

      Show
      1- Create a service (Admin > Plugins > Web service > Manage service) 2- Add few functions to service You never seen the error message: Invalid array parameter detected in required_param(): fid
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:

      Description

      Invalid array parameter detected in required_param(): fid

      line 558 of /lib/moodlelib.php: call to debugging()
      line 32 of /admin/webservice/service_functions.php: call to optional_param()

      Notice: Undefined property: stdClass::$fid in /home/aparup/mcode/22/mysql/admin/webservice/service_functions.php on line 70 Warning: Invalid argument supplied for foreach() in /home/aparup/mcode/22/mysql/admin/webservice/service_functions.php on line 70

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

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

              Another issue to peer-review Rosie Thank you

              Show
              jerome Jérôme Mouneyrac added a comment - Another issue to peer-review Rosie Thank you
              Hide
              rwijaya Rossiani Wijaya added a comment -

              Hi Jerome,

              The patch looks good.

              Thanks for fixing this.

              Show
              rwijaya Rossiani Wijaya added a comment - Hi Jerome, The patch looks good. Thanks for fixing this.
              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
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Testing instructions, plz?

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Testing instructions, plz?
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Testing instructions, plz?

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Testing instructions, plz?
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              (Ops, sorry for dupe comment, I got an error and reloaded)

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - (Ops, sorry for dupe comment, I got an error and reloaded)
              Hide
              jerome Jérôme Mouneyrac added a comment -

              Instruction added

              Show
              jerome Jérôme Mouneyrac added a comment - Instruction added
              Hide
              nebgor Aparup Banerjee added a comment -

              Thanks guys,
              thats been integrated into 21, 22 and master.

              Show
              nebgor Aparup Banerjee added a comment - Thanks guys, thats been integrated into 21, 22 and master.
              Hide
              abgreeve Adrian Greeve added a comment -

              I tested this on versions 2.1, 2.2 and master. I checkout out master before hand to see what errors it was generating. I didn't come across any errors when adding functions to custom services.
              Test passed.

              Show
              abgreeve Adrian Greeve added a comment - I tested this on versions 2.1, 2.2 and master. I checkout out master before hand to see what errors it was generating. I didn't come across any errors when adding functions to custom services. Test passed.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              It is late here and I'm very tired but I didn't want to go to sleep before expressing my admiration for your amazing collaboration. Thanks!

              Closing as fixed, heading to zzzZZZzzz, niao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - It is late here and I'm very tired but I didn't want to go to sleep before expressing my admiration for your amazing collaboration. Thanks! Closing as fixed, heading to zzzZZZzzz, niao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    12/Mar/12