Division by zero calculating enforce points weighting with zero included points

Description

WARN: FatalException: (2009-09-03 12:29:25,454
http-8443-Processor22_org.sakaiproject.gradebook.gwt.sakai.Gradebook2ResourceProducer)
java.lang.ArithmeticException: Division by zero
at java.math.BigDecimal.divide(BigDecimal.java:1568)
at org.sakaiproject.gradebook.gwt.sakai.Gradebook2ServiceImpl.calculateItemCategoryPercent(Gradebook2ServiceImpl.java:2693)
at org.sakaiproject.gradebook.gwt.sakai.Gradebook2ServiceImpl.getItemModel(Gradebook2ServiceImpl.java:3654)
at org.sakaiproject.gradebook.gwt.sakai.Gradebook2ServiceImpl.createGradebookModel(Gradebook2ServiceImpl.java:2831)
at org.sakaiproject.gradebook.gwt.sakai.Gradebook2ServiceImpl.createOrRetrieveGradebookModel(Gradebook2ServiceImpl.java:3232)
at org.sakaiproject.gradebook.gwt.sakai.Gradebook2ServiceImpl.getGradebookModels(Gradebook2ServiceImpl.java:3617)
at org.sakaiproject.gradebook.gwt.sakai.Gradebook2ServiceImpl.getApplicationModel(Gradebook2ServiceImpl.java:1190)
at

Activity

Show:

James Renfro September 11, 2009 at 9:50 AM

This item has been included in the 1.1.0-rc1 and rolled to UCD staging in preparation for production release.

James Renfro September 4, 2009 at 12:30 PM

Closing since this is a QA pass.

Nancy Wilkison September 4, 2009 at 11:20 AM

QA ENVIRONMENT -
vista enterprise, firefox, ie7
smartsite-test sakai_2-5-x-test-076 - Sakai 2.5.x-R57934 gb2-1.1.0-b2 - Server sakaitest2

QA SUMMARY -
duplicate of grbk-348. see there for complete test.
in short, on same site...
1 - added weight by points category, 1 item @ 100 points, 1 @ 50 points and ec at 20
2 - deleted item 1/100 points - ok
3 - deleted item 2/50 points, where the problem started yesterday - ok!
4 - category now shows %category as 0/100%, correct as only the ec item is there - ok
5 - deleted ec item/20 points - ok

QA RESULTS -
pass today on -test server

James Renfro September 3, 2009 at 10:52 PM

This item is now ready for testing.

James Renfro September 3, 2009 at 1:28 PM

Fixed in trunk. Please hold off until it's in the beta release to test.

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

Created September 3, 2009 at 1:24 PM
Updated June 29, 2010 at 2:01 PM
Resolved September 3, 2009 at 1:28 PM