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

rating API, check and update DocBlock

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2, 2.2.1
    • Fix Version/s: 2.3
    • Component/s: Documentation, Ratings
    • Labels:

      Description

      Check and update documentation, so that it should comply with moodle coding guidelines.
      Following needs to be updated/checked for rating api

      1. DocBlock for page and functions.
      2. All the files should be checked/updated.

      Note: You can create sub-tasks, so as to avoid bulk integration.

        Gliffy Diagrams

        1. smurf.xml
          6 kB
          Eloy Lafuente (stronk7)

          Activity

          Hide
          dougiamas Martin Dougiamas added a comment -

          Any progress on this yet?

          Show
          dougiamas Martin Dougiamas added a comment - Any progress on this yet?
          Hide
          phalacee Jason Fowler added a comment -

          Not yet, I've been working through the RSS API

          Show
          phalacee Jason Fowler added a comment - Not yet, I've been working through the RSS API
          Hide
          phalacee Jason Fowler added a comment -

          Please ignore the branch name, the MDL in it's description is correct though

          Show
          phalacee Jason Fowler added a comment - Please ignore the branch name, the MDL in it's description is correct though
          Hide
          dougiamas Martin Dougiamas added a comment -

          Docs still need a lot of work. This shouldn't be very hard. http://docs.moodle.org/dev/Rating_API

          Show
          dougiamas Martin Dougiamas added a comment - Docs still need a lot of work. This shouldn't be very hard. http://docs.moodle.org/dev/Rating_API
          Hide
          phalacee Jason Fowler added a comment -

          Just ran it through Marina's script to make sure I didn't miss anything

          Show
          phalacee Jason Fowler added a comment - Just ran it through Marina's script to make sure I didn't miss anything
          Hide
          phalacee Jason Fowler added a comment -

          Will get back to the wiki docs after I have finished my work on Sprint 17

          Show
          phalacee Jason Fowler added a comment - Will get back to the wiki docs after I have finished my work on Sprint 17
          Hide
          salvetore Michael de Raadt added a comment -

          Some comments:

          rating/lib.php

          • Line 324, some unnecessary whitespace in this parameter
          • Line 733, TODO should be @todo and should have an MDL included
          • Line 738, type should be int|null
          • Line 754, needs a blank comment line after description
          • Lines 762, 764, 767, some unnecessary blank lines, also, this comment is a bit inconsistent with previous class parameters and needs a closing brace, not that that's standard or anything
          • Line 891, needs a blank comment line after description
          • Line 920, needs a blank comment line after description
          • Line 931, needs a blank comment line after description
          • Line 1056, needs a blank comment line after description
          Show
          salvetore Michael de Raadt added a comment - Some comments: rating/lib.php Line 324, some unnecessary whitespace in this parameter Line 733, TODO should be @todo and should have an MDL included Line 738, type should be int|null Line 754, needs a blank comment line after description Lines 762, 764, 767, some unnecessary blank lines, also, this comment is a bit inconsistent with previous class parameters and needs a closing brace, not that that's standard or anything Line 891, needs a blank comment line after description Line 920, needs a blank comment line after description Line 931, needs a blank comment line after description Line 1056, needs a blank comment line after description
          Hide
          phalacee Jason Fowler added a comment -

          Thanks for the feedback Michael, those changes have now been made.

          Show
          phalacee Jason Fowler added a comment - Thanks for the feedback Michael, those changes have now been made.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Hi Jason, attaching another xml file, it shows the incorrect "asterisks" and some incorrect tabs/whitespace. I think you can ignore safely the rest.

          Ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Hi Jason, attaching another xml file, it shows the incorrect "asterisks" and some incorrect tabs/whitespace. I think you can ignore safely the rest. Ciao
          Hide
          phalacee Jason Fowler added a comment -

          Thanks for that Eloy, all cleaned up now.

          Show
          phalacee Jason Fowler added a comment - Thanks for that Eloy, all cleaned up now.
          Hide
          andyjdavis Andrew Davis added a comment -

          I believe the wiki docs are done. http://docs.moodle.org/dev/Rating_API

          Jason, feel free to reshuffle it and add anything I missed. Just make sure that to limit it to stuff someone adding ratings support to a custom activity module would be interested in. There were a few references to rating.php and rating_ajax.php which are internal files that someone using ratings doesn't need to be bothered with.

          Show
          andyjdavis Andrew Davis added a comment - I believe the wiki docs are done. http://docs.moodle.org/dev/Rating_API Jason, feel free to reshuffle it and add anything I missed. Just make sure that to limit it to stuff someone adding ratings support to a custom activity module would be interested in. There were a few references to rating.php and rating_ajax.php which are internal files that someone using ratings doesn't need to be bothered with.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Sorry, Jason... but where are the changes? I continue getting the same errors about bad asterisks and tabs/whitespace. Perhaps the work is into another branch? Note that the one above (wip-MDL-30991-master) doesn't match this issue number (MDL-30990).

          Holding this....ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Sorry, Jason... but where are the changes? I continue getting the same errors about bad asterisks and tabs/whitespace. Perhaps the work is into another branch? Note that the one above (wip- MDL-30991 -master) doesn't match this issue number ( MDL-30990 ). Holding this....ciao
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Ping!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Ping!
          Hide
          rajeshtaneja Rajesh Taneja added a comment -

          Hello Eloy,
          Jason is on leave and will be back next month

          Show
          rajeshtaneja Rajesh Taneja added a comment - Hello Eloy, Jason is on leave and will be back next month
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment - - edited

          I'm reopening this for Jason to take a look to it after he returns. Surely he will need to rebase (apart from fixes) as far as the branch has been on hold for many weeks.

          TIA!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - - edited I'm reopening this for Jason to take a look to it after he returns. Surely he will need to rebase (apart from fixes) as far as the branch has been on hold for many weeks. TIA!
          Hide
          phalacee Jason Fowler added a comment -

          All there now, I don't know why I forgot to push the changes ...

          Show
          phalacee Jason Fowler added a comment - All there now, I don't know why I forgot to push the changes ...
          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
          phalacee Jason Fowler added a comment -

          All rebased and ready for the integrators to work their magic

          Show
          phalacee Jason Fowler added a comment - All rebased and ready for the integrators to work their magic
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Nobody tested this, passing!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Nobody tested this, passing!
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Your changes are now upstream and will be included in the next minor released scheduled for March 13th (next Monday!).

          icao_reverse('arreis olik rebemevon afla letoh ognat');

          Closing, ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Your changes are now upstream and will be included in the next minor released scheduled for March 13th (next Monday!). icao_reverse('arreis olik rebemevon afla letoh ognat'); Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                25/Jun/12