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

          Attachments

            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