[#MDL-11581] ouuser report needs to be upgraded to use all the new letters, displaytype and gradeitems settings - Moodle Tracker

Details

Type: Sub-task
Status: Closed
Priority: Blocker
Resolution: Fixed
Affects Version/s: 1.9
Fix Version/s: 1.9
Component/s: Gradebook
Labels:
None

Affected Branches:
MOODLE_19_STABLE
Fixed Branches:
MOODLE_19_STABLE

Description

Nicolas, can you please port all the recent changes in the user report back to the ouuser report so it works again? Letters for sure, not sure about decimals etc.

http://moodle.cvs.sourceforge.net/moodle/contrib/plugins/grade/report/ouuser/

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Nicolas Connault added a comment - 04/Oct/07 1:10 AM - edited

You mean it's not working? It works fine here...

Could you actually let me know what exact specifications are required? At the moment the OU user report only displays percentages, it doesn't care about the gradedisplaytype setting. It's the same actually with the normal user report: letter display isn't implemented.

As for decimals, it's working fine.

Show

Nicolas Connault added a comment - 04/Oct/07 1:10 AM - edited You mean it's not working? It works fine here... Could you actually let me know what exact specifications are required? At the moment the OU user report only displays percentages, it doesn't care about the gradedisplaytype setting. It's the same actually with the normal user report: letter display isn't implemented. As for decimals, it's working fine.

Hide

Permalink

Nicolas Connault added a comment - 04/Oct/07 1:18 PM

Implementing display of grade letters for both user reports...

Show

Nicolas Connault added a comment - 04/Oct/07 1:18 PM Implementing display of grade letters for both user reports...

Hide

Permalink

Nicolas Connault added a comment - 04/Oct/07 3:08 PM

More extensive testing still needed, but the implementation is complete for both user reports.

Show

Nicolas Connault added a comment - 04/Oct/07 3:08 PM More extensive testing still needed, but the implementation is complete for both user reports.

Hide

Permalink

Martin Dougiamas added a comment - 04/Oct/07 4:07 PM

OK, I'm testing it out now.

Show

Martin Dougiamas added a comment - 04/Oct/07 4:07 PM OK, I'm testing it out now.

People

Assignee:

Nicolas Connault

Reporter:

Martin Dougiamas

Tester:

Nobody

Participants:

Martin Dougiamas, Nicolas Connault

Vote (0)

Watch (0)

Dates

Created:

03/Oct/07 11:56 PM

Updated:

17/Dec/10 1:07 PM

Resolved:

04/Oct/07 3:08 PM