Apache JDO Project - [jira] Commented: (JDO-402) checkRemoveGroup - removeGroup should

This is Interesting: Free IT Magazines  
Home > Archive > Apache JDO Project > August 2006 > [jira] Commented: (JDO-402) checkRemoveGroup - removeGroup should





You are viewing an archived Text-only version of the thread. To view this thread in it's original format and/or if you want to reply to this thread please [click here]

Author [jira] Commented: (JDO-402) checkRemoveGroup - removeGroup should
Erik Bengtson (JIRA)

2006-08-06, 7:11 am

[ http://issues.apache.org/jira/brows...action_12426048 ]

Erik Bengtson commented on JDO-402:
-----------------------------------

It was implicitly understood that the returned Set is immutable because the underlying Set is a reference. If the underlying set is a copy, it does not make sense to be immutable.

Anyway, I dont have any problem if it's changed to copy. In this case, making the returned set mutable is better for usability.

> checkRemoveGroup - removeGroup should be before getGroups
> ---------------------------------------------------------
>
> Key: JDO-402
> URL: http://issues.apache.org/jira/browse/JDO-402
> Project: JDO
> Issue Type: Bug
> Components: tck20
> Affects Versions: JDO 2 final
> Reporter: Ilan Kirsh
> Assigned To: Craig Russell
> Fix For: JDO 2 maintenance release 1
>
> Attachments: JDO-402.patch
>
>
> In org.apache.jdo.tck.api.persistencemanager.fetchplan.FetchPlanInterface, lines 281-282,
> it probably has to be:
> fp.removeGroup("default");
> Collection groups = fp.getGroups();
> Instead of:
> Collection groups = fp.getGroups();
> fp.removeGroup("default");


--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secur...nistrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira



Sponsored Links






Free braindumps | Software forum | Database administration forum

Copyright 2003 - 2008 webservertalk.com