Some webservice functions have the wrong explanatory comments

Bug #1396437 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Robert Lyon

Bug Description

It looks as though a function has been copied and pasted in a few places and tweaked to suit new role but the explanatory comment hasn't been updated for the new copy of the function.

Will need to go through and check that the explanatory comments match what the function says it does, eg check that the variables listed in comment match what the function actually accepts, or that it returns what the comment says it returns etc.

Tags: webservices
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/4112

Changed in mahara:
milestone: none → 15.04.0
status: New → In Progress
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/4112
Committed: http://gitorious.org/mahara/mahara/commit/bb7c65efb2f2a1fc487925882c803ce62972ad15
Submitter: Aaron Wells (<email address hidden>)
Branch: master

commit bb7c65efb2f2a1fc487925882c803ce62972ad15
Author: Robert Lyon <email address hidden>
Date: Tue Dec 23 16:11:56 2014 +1300

Making sure function comment matches function (Bug #1396437)

Change-Id: I573e6f354a2077afbaed5a501d2d99106b736e1a
Signed-off-by: Robert Lyon <email address hidden>

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
assignee: nobody → Robert Lyon (robertl-9)
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.