Author: thomas.heute(a)jboss.com Date: 2007-08-15 07:01:57 -0400 (Wed, 15 Aug 2007) New Revision: 7930 Modified: branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/identity/db/HibernateRoleModuleImpl.java branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/identity/db/HibernateUserModuleImpl.java branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/test/identity/IdentityTest.java branches/JBoss_Portal_Branch_2_6/identity/src/resources/hibernate/domain-identity.hbm.xml Log: Return ordered Set of users and roles when searching on database Modified: branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/identity/db/HibernateRoleModuleImpl.java =================================================================== --- branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/identity/db/HibernateRoleModuleImpl.java 2007-08-15 09:17:32 UTC (rev 7929) +++ branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/identity/db/HibernateRoleModuleImpl.java 2007-08-15 11:01:57 UTC (rev 7930) @@ -273,9 +273,9 @@ try { Session session = getCurrentSession(); - Query query = session.createQuery("from HibernateRoleImpl"); + Query query = session.createQuery("from HibernateRoleImpl as r order by r.name"); Iterator iterator = query.iterate(); - return Tools.toSet(iterator); + return Tools.toSet(iterator, true); } catch (HibernateException e) { @@ -300,19 +300,19 @@ userNameFilter = "%" + userNameFilter.replaceAll("%", "") + "%"; // - query = session.createQuery("from HibernateUserImpl as user left join user.roles role where role.name=:name" + " AND user.userName LIKE :filter"); + query = session.createQuery("from HibernateUserImpl as user left join user.roles role where role.name=:name" + " AND user.userName LIKE :filter order by user.userName"); query.setString("filter", userNameFilter); } else { - query = session.createQuery("from HibernateUserImpl as user left join user.roles role where role.name=:name"); + query = session.createQuery("from HibernateUserImpl as user left join user.roles role where role.name=:name order by user.userName"); } query.setString("name", roleName); query.setFirstResult(offset); query.setMaxResults(limit); Iterator iterator = query.iterate(); - Set result = Tools.toSet(iterator); + Set result = Tools.toSet(iterator, true); Set newResult = new HashSet(); Iterator cleaner = result.iterator(); Modified: branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/identity/db/HibernateUserModuleImpl.java =================================================================== --- branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/identity/db/HibernateUserModuleImpl.java 2007-08-15 09:17:32 UTC (rev 7929) +++ branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/identity/db/HibernateUserModuleImpl.java 2007-08-15 11:01:57 UTC (rev 7930) @@ -231,11 +231,11 @@ try { Session session = getCurrentSession(); - Query query = session.createQuery("from HibernateUserImpl"); + Query query = session.createQuery("from HibernateUserImpl as u order by u.userName"); query.setFirstResult(offset); query.setMaxResults(limit); Iterator iterator = query.iterate(); - return Tools.toSet(iterator); + return Tools.toSet(iterator, true); } catch (HibernateException e) { @@ -254,12 +254,12 @@ // Session session = getCurrentSession(); - Query query = session.createQuery("from HibernateUserImpl as u where u.userName like :filter"); + Query query = session.createQuery("from HibernateUserImpl as u where u.userName like :filter order by u.userName"); query.setString("filter", filter); query.setFirstResult(offset); query.setMaxResults(limit); Iterator iterator = query.iterate(); - return Tools.toSet(iterator); + return Tools.toSet(iterator, true); } catch (HibernateException e) { Modified: branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/test/identity/IdentityTest.java =================================================================== --- branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/test/identity/IdentityTest.java 2007-08-15 09:17:32 UTC (rev 7929) +++ branches/JBoss_Portal_Branch_2_6/identity/src/main/org/jboss/portal/test/identity/IdentityTest.java 2007-08-15 11:01:57 UTC (rev 7930) @@ -75,46 +75,31 @@ { ctx.begin(); Set set1 = userModule.findUsersFilteredByUserName("user", 0, 10); - Set nameSet1 = new HashSet(); - for (Iterator i = set1.iterator(); i.hasNext();) - { - User user = (User)i.next(); - nameSet1.add(user.getUserName()); - } - Set expectedNameSet1 = new HashSet(); - expectedNameSet1.add("user1"); - expectedNameSet1.add("user2"); - assertEquals(expectedNameSet1, nameSet1); - + Iterator it = set1.iterator(); + User user = (User)it.next(); + assertEquals("user1", user.getUserName()); + user = (User)it.next(); + assertEquals("user2", user.getUserName()); + // Set set2 = userModule.findUsersFilteredByUserName("blah", 0, 10); assertEquals(0, set2.size()); // Set set3 = userModule.findUsersFilteredByUserName("", 0, 10); - Set nameSet3 = new HashSet(); - for (Iterator i = set3.iterator(); i.hasNext();) - { - User user = (User)i.next(); - nameSet3.add(user.getUserName()); - } - Set expectedNameSet3 = new HashSet(); - expectedNameSet3.add("user1"); - expectedNameSet3.add("user2"); - expectedNameSet3.add("admin"); - assertEquals(expectedNameSet3, nameSet3); + it = set3.iterator(); + user = (User)it.next(); + assertEquals("admin", user.getUserName()); + user = (User)it.next(); + assertEquals("user1", user.getUserName()); + user = (User)it.next(); + assertEquals("user2", user.getUserName()); // Set set4 = userModule.findUsersFilteredByUserName("user1", 0, 10); - Set nameSet4 = new HashSet(); - for (Iterator i = set4.iterator(); i.hasNext();) - { - User user = (User)i.next(); - nameSet4.add(user.getUserName()); - } - Set expectedNameSet4 = new HashSet(); - expectedNameSet4.add("user1"); - assertEquals(expectedNameSet4, nameSet4); + it = set4.iterator(); + user = (User)it.next(); + assertEquals("user1", user.getUserName()); ctx.commit(); } Modified: branches/JBoss_Portal_Branch_2_6/identity/src/resources/hibernate/domain-identity.hbm.xml =================================================================== --- branches/JBoss_Portal_Branch_2_6/identity/src/resources/hibernate/domain-identity.hbm.xml 2007-08-15 09:17:32 UTC (rev 7929) +++ branches/JBoss_Portal_Branch_2_6/identity/src/resources/hibernate/domain-identity.hbm.xml 2007-08-15 11:01:57 UTC (rev 7930) @@ -124,7 +124,7 @@ lazy="false" inverse="false" cascade="none" - sort="unsorted"> + sort="name"> <!--<cache usage=&quot;@portal.hibernate.cache.usage(a)&quot;/&gt;--&gt; <key column="jbp_uid"/> <many-to-many