Issue Details (XML | Word | Printable)

Key: SAK-16166
Type: Sub-task Sub-task
Status: Closed Closed
Resolution: Fixed
Priority: Critical Critical
Assignee: Lydia Li
Reporter: Jean-François Lévêque
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Sakai
SAK-14386

Samigo: resources are not correctly released [Static Code Review]

Created: 24-Apr-2009 01:49   Updated: 27-Oct-2009 13:40
Component/s: Tests & Quizzes (Samigo)
Affects Version/s: 2.6.0
Fix Version/s: 2.6.1, 2.7.0

Time Tracking:
Not Specified

2.6.x Status: Resolved
2.5.x Status: None
2.4.x Status: None


 Description  « Hide
There is a new FindBugs detector for finding leaking resources: OBL_UNSATISFIED_OBLIGATION

See for an overview of all relevant issues: http://qa1-nl.sakaiproject.org/codereview/bug_dashboard/findbugs_database.html



 All   Comments   Work Log   Change History   Subversion Commits   git Commits      Sort Order: Ascending order - Click to sort in descending order
Jean-François Lévêque added a comment - 24-Apr-2009 01:51
This was done in SAK-14386.

Jean-François Lévêque added a comment - 24-Apr-2009 01:51
http://qa1-nl.sakaiproject.org/codereview/bug_dashboard/findbugs_database.html does not show OBL_UNSATISFIED_OBLIGATION for samigo anymore.

Peter Peterson added a comment - 15-May-2009 00:18
Changed merge request for 2.6.x to "None" per Jean Francious comments.

Jean-François Lévêque added a comment - 15-May-2009 02:16
r54945 trunk changes have not been merged into 2.6.x yet.

Karen Tsao added a comment - 17-Sep-2009 10:43 - edited
For 2.6.x:

coursework-dev4:~/sakai2.6/sakai2.6-oracle/sakai/sam >svn commit -m "SAK-16166"
Sending samigo-app/src/java/org/sakaiproject/tool/assessment/devtools/SubstituteProperties.java
Sending samigo-app/src/java/org/sakaiproject/tool/assessment/settings/PathInfo.java
Sending samigo-qti/src/java/org/sakaiproject/tool/assessment/qti/util/PathInfo.java
Sending samigo-qti/src/java/org/sakaiproject/tool/assessment/qti/util/XmlUtil.java
Transmitting file data ....
Committed revision 66580.