[isidorus-cvs] r859 - branches/gdl-frontend/src/anaToMia/GDL_Widgets/src/us/isidor/gdl/anaToMia/Widgets/base

lgiessmann at common-lisp.net lgiessmann at common-lisp.net
Fri Sep 9 15:25:43 UTC 2011


Author: lgiessmann
Date: Fri Sep  9 08:25:42 2011
New Revision: 859

Log:
gdl-frontend: Widgets: fixed a a potential bug in TmHelper.hasRoles(...)

Modified:
   branches/gdl-frontend/src/anaToMia/GDL_Widgets/src/us/isidor/gdl/anaToMia/Widgets/base/TmHelper.java

Modified: branches/gdl-frontend/src/anaToMia/GDL_Widgets/src/us/isidor/gdl/anaToMia/Widgets/base/TmHelper.java
==============================================================================
--- branches/gdl-frontend/src/anaToMia/GDL_Widgets/src/us/isidor/gdl/anaToMia/Widgets/base/TmHelper.java	Thu Sep  8 07:54:00 2011	(r858)
+++ branches/gdl-frontend/src/anaToMia/GDL_Widgets/src/us/isidor/gdl/anaToMia/Widgets/base/TmHelper.java	Fri Sep  9 08:25:42 2011	(r859)
@@ -246,7 +246,7 @@
 	// the array list rolePlayersAndTypes
 	public static boolean hasRoles(Association assoc, ArrayList<Pair<Topic, Topic>> rolePlayertypesAndTypes){
 		if(assoc == null) return false;
-		if(assoc.getRoles().length() == 0 && rolePlayertypesAndTypes == null) return true;
+		if(rolePlayertypesAndTypes == null) return assoc.getRoles().length() == 0;
 		if(assoc.getRoles().length() != rolePlayertypesAndTypes.size()) return false;
 		
 		for (Pair<Topic, Topic> pair : rolePlayertypesAndTypes) {




More information about the Isidorus-cvs mailing list