It really needs some testing. I'll put it in HEAD first.

The actual creation of the scale I did directly in the database ... the GUI needs improvements to handle it.

The main part of what I did was to make these look nice in the forum.

There will be more work to test the gradebook to make sure everything works as expected there with 1 -point scales ... hopefully there are no assumptions in code about a minimum of two items.

Adding link to http://moodle.org/mod/forum/discuss.php?d=118821

where I suggest one simple change I've in some servers. I'd consider it incomplete because it just disables the validation. Instead we should be validating each element, looking for nulls / empties or so.

Ciao

If I remember correctly, there is an issue with aggregation in gradebook. If we have one item scales only, gradebook ends in division by zero in VALUE / (MAX - MIN) or so. Therefore, to have single item scales at the Czech community server, I have this hack:

I have prepared normal two items scale: "Užite?ný p?ísp?vek, Užite?ný p?ísp?vek" (which is "Useful post" in Czech). Then, I just patched forum to display just the second option:

diff --git a/mod/forum/lib.php b/mod/forum/lib.php

index 99022ad..fbd48c4 100644

--- a/mod/forum/lib.php

+++ b/mod/forum/lib.php

@@ -3638,6 +3638,11 @@ function forum_print_rating_menu($postid, $userid, $scale, $myrating=NULL) {

     if (empty($strrate)) {

         $strrate = get_string("rate", "forum");

     }

+    // hack mudrd8mz - enable one item scale in CSMUG forums

+    if ((count($scale) == 2) and ($scale[1] == 'Užite?ný p?ísp?vek') and ($scale[2] == 'Užite?ný p?ísp?vek')) {

+        unset($scale[1]);

+    }

+    // end hack

     $scale = array(FORUM_UNSET_POST_RATING => $strrate.'...') + $scale;

     choose_from_menu($scale, $postid, $myrating, '', '', '0', false, false, 0, '', false, false, 'forumpostratingmenu');

 }

This way, users can have either grade==0 (not graded post) or grade==2 on each post and the calculation in Gradebook works.

NOTE: This issue was assigned to the STABLE backlog without complete triaging process. Marking it as triaged, but with this note for future reference.

Adding 2.2.1 as an affected version, as this issue is again being asked for on moodle.org.

I'm one of the people asking for it: specifically for the Lang Teaching course. I don't understand why the clever hack implemented on Using Moodle and the French forum, can't also be installed on Lang Teaching. It seems to me much more in keeping with the Moodle ethos than judgemental scales. I'd also like to be able to use it on other Moodle platforms.

More requests for this feature:

http://moodle.org/mod/forum/discuss.php?d=180769
http://moodle.org/mod/forum/discuss.php?d=197048

Would someone mind answering if a single-point scale is introduced via a SQL change (to the scale) the grader report only make NULL/ZERO errors for that item using the one point scale?

My worry is it causes issues across the course's graded items and not just the single rated entity.

I don't mind if it just affects itself - I'll just hide it and exclude it from the course grade.

Hi Matt. I just experimentally applied the change to one of my production server and so far I can't see any issues. I haven't checked the grades calculations yet though. Anyway, now I think that the gradebook actually does support one-item scales and the problem with "division by zero" should not occur. Even with single-item scales, the minimal grade is 0 (no grade given) and maximal grade is 1 (graded using the single item scale). So the calculations should work. This requires deeper check and tests though.

Hi David,

Do you know what the status of this is? We're looking at it again for moodle.org in MDLSITE-1877 and I wonder if we could sort this out for the rest of the community at the same time.

(Note: i'm not really away of the implications )

Hi Dan. Thanks for raising this. Since my last comment I did not have much opportunity to look at this closely. However, I believe that we should be brave and simply try to remove the multiple items condition when editing a scale. Gradebook aggregation algorithms should be fine with it as they map "not selected" to 0 and then each scale item gets grade 1, 2, 3 etc. Thence a single-item scale operates over 0-1 interval and there should hopefully be no situation of dividing by zero.

This issue was assigned to me, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue. For more information, see http://docs.moodle.org/dev/Changes_to_issue_assignment

Just discussed this at http://docs.moodle.org/dev/Hackfest_UK_2014, and were generally thinking it was a good idea. Michael de Raadt asked me to mark it as a security issue.

For the record, we are using single item scales in production at moodle.org for some time already.

Hi David,
Yes, you are. Any chance of just adding that option to 2.7.1?

Even when this seems to be pretty trivial to implement and has been tested in production, I don't think we should change things like that on stable releases - and especially not at LTS branch. So my personal suggestion would be to add it as experimental flag for 2.8.

No need for a flag, let's just make it work for 2.8 when you only type one item in the scale.

We could even backport to 2.7 later, it's really a tiny feature (but useful) that doesn't affect APIs etc.

Just to comment that, related with this, MDL-45394 has a implementation about single-item "likes" that is being examined right now.

Also, other similar issues have been closed as dupe of both this and the "likes" one, FYI.

Ciao

Edited: Wrong link fixed.

Sorry Jason, I did not mean to steal your issue, I must have clicked the button to assign it to me after you assigned it to yourself.

S'all good, I should have checked my assigned issues first anyway, I have one on the go in this sprint already. Happy to have you take this off me.

I did not see any problem allowing the scales to have only 1 item. Most cases should still work as they did. I added a small unit tests that confirms that the API is OK with it.

Results for MDL-16474

• Remote repository: git://github.com/FMCorz/moodle.git

• Remote branch MDL-16474-master to be integrated into upstream master
  • Executed job http://integration.moodle.org/job/Precheck%20remote%20branch/4416
  • Details: http://integration.moodle.org/job/Precheck%20remote%20branch/4416/artifact/work/smurf.html

+1, thanks Fred (that does actually match our experience of using it on moodle.org)

Can I suggest expansion of testing instructions to include editing and removing the scale.

Thanks Dan.

Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!

Thanks Fred this has been integrated now. Yay that it finally lands!

Works as described. Passing.

Just in time for incoming releases, well done. And big thanks!

We must plan for freedom,
and not only for security,
if for no other reason than that
only freedom can make security secure

–  Karl Popper

Good work, thanks you for this.

Is there any plan to backport this to LTS 2.7?

Just noting that we now have a single-item scale QA test: MDLQA-7167. Please shout if the test needs amending or expanding at all.

Removing docs_required label as this is documented in https://docs.moodle.org/28/en/Scales