Ticket #377 (closed defect: fixed)

Opened 1 year ago

Last modified 1 year ago

values in ratings stars must be multiples of 10

Reported by: stevec Assigned to: jsuttor
Priority: critical Milestone:
Component: ambra Version: 0.7
Keywords: plosone, ratings, testing Cc:
Blocking: Blocked By:

Description

to correctly render start, the value passed in to the css pct%% must be a multiple of 10. On the all ratings page, the overall value is not set that way, thus it can result in a blank rating.

Dependency Graph

Change History

06/13/07 15:03:47 changed by amit

  • owner changed from somebody to jsuttor.
  • priority changed from unassigned to critical.
  • version set to 0.7.
  • milestone changed from TBD to 0.7.

06/13/07 16:05:08 changed by jsuttor

  • keywords set to plosone, ratings, testing.
  • status changed from new to assigned.
  • component changed from topaz to plos-one.

all pct%% are set as multiples of 20%, e.g. 1 star = 20%, this is cloned from the original template.

any other comments about what exactly is needed?

06/13/07 16:18:48 changed by stevec

As mentioned in the original report, check the values on the overall rating on the all ratings page. Rate with three different users so that you get a non .5 value. Values will not be a multiple of 10.

06/13/07 16:21:31 changed by stevec

To clarify further. 20 * X where is is not an integer value does not make the result a multiple of 10. If you have an overall value of 3.7 you will get 74. On the article pages, there is a value called roundedAverage to solve this problem.

06/14/07 11:36:48 changed by jsuttor

  • status changed from assigned to closed.
  • resolution set to fixed.

fixed in r2936. there is now consistent handling of overall & averages in all contexts.

10/29/07 21:13:36 changed by

  • milestone deleted.

Milestone 0.7 deleted