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

Forum rating display broken when rating value is 0

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.9.6
    • 2.9.7, 3.0.5
    • Forum, Ratings
    • MOODLE_29_STABLE
    • MOODLE_29_STABLE, MOODLE_30_STABLE
    • MDL-54620_master
    • Hide
      1. create a forum and a post from a student user
      2. Allow ratings on the forum and use an aggregation method such as 'minimum'
      3. As a teacher, rate the student's post, assigning it a score of zero.
      4. Refresh the page.
        • Confirm that display is 'Minimum rating: 0 (1)'
      Show
      create a forum and a post from a student user Allow ratings on the forum and use an aggregation method such as 'minimum' As a teacher, rate the student's post, assigning it a score of zero. Refresh the page. Confirm that display is 'Minimum rating: 0 (1)'

    Description

      When a student's post is rated with a value of 0, the student sees the grade display under their post as:
      Maximum rating: (1)
      When that same post is rated with a non-zero value, say 4, the student sees the grade display under their post as:
      Maximum rating: 4 (1)
      That is, when the value is 0, the rating display seems to use a space instead of the actual number.
      This currently exists in Moodle 2.3.2 and can be easily reproduced on demo.moodle.org as follows, in the CF101 demo course:
      1) Log in as teacher to the CF101 demo course
      2) Create a forum with default settings except for:
      2a) Change "Aggregate type" to "Maximum rating"
      2b) Change Scale dropdown to "5"
      3) Log in as student and add a post to the forum
      4) Log in as teacher and rate the student's forum post with a 0
      5) Log in as student and see the rating.
      Under the post, the rating text display should say:
      Maximum rating: 0 (1)
      Instead, as of 10/26/12 on Moodle 2.3.2, it says:
      Maximum rating: (1)
      See attached screenshot for the bug.

      Attachments

        Issue Links

          Activity

            People

              kwiliarty Kevin Wiliarty
              dmonllao David Monllaó
              Andrew Lyons Andrew Lyons
              Jake Dallimore Jake Dallimore
              Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Stevani Andolo, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                11/Jul/16