[jboss-svn-commits] JBL Code SVN: r35182 - in labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src: main/java/org/drools/guvnor/server and 1 other directories.
jboss-svn-commits at lists.jboss.org
jboss-svn-commits at lists.jboss.org
Sat Sep 18 01:21:50 EDT 2010
Author: jervisliu
Date: 2010-09-18 01:21:50 -0400 (Sat, 18 Sep 2010)
New Revision: 35182
Modified:
labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/client/rulelist/AssetItemGrid.java
labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/server/ServiceImplementation.java
labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/java/org/drools/guvnor/server/ServiceImplementationTest.java
Log:
BRMS-378: Asset position displayed by Next/Previous button is not correct in "Knowledge Bases -> Packages -> defaultPackage -> Business Rule Assets"
BRMS-277: Administration -> Archive shows only 40 items
Modified: labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/client/rulelist/AssetItemGrid.java
===================================================================
--- labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/client/rulelist/AssetItemGrid.java 2010-09-17 18:30:56 UTC (rev 35181)
+++ labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/client/rulelist/AssetItemGrid.java 2010-09-18 05:21:50 UTC (rev 35182)
@@ -382,7 +382,7 @@
public void onClick(Button button,
EventObject e) {
if ( forward ) {
- int newPos = currentCursorPosition - 2;
+ int newPos = currentCursorPosition;
if ( newPos > 0 ) {
cursorPositions.push( newPos );
}
Modified: labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/server/ServiceImplementation.java
===================================================================
--- labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/server/ServiceImplementation.java 2010-09-17 18:30:56 UTC (rev 35181)
+++ labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/main/java/org/drools/guvnor/server/ServiceImplementation.java 2010-09-18 05:21:50 UTC (rev 35182)
@@ -887,7 +887,10 @@
TableDataResult table = new TableDataResult();
table.data = result.toArray( new TableDataRow[result.size()] );
- table.currentPosition = it.getPosition();
+ table.currentPosition = it.getPosition();
+ table.total = it.getSize();
+ table.hasNext = it.hasNext();
+
return table;
}
Modified: labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/java/org/drools/guvnor/server/ServiceImplementationTest.java
===================================================================
--- labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/java/org/drools/guvnor/server/ServiceImplementationTest.java 2010-09-17 18:30:56 UTC (rev 35181)
+++ labs/jbossrules/soa_branches/BRMS-5.1.1/drools-guvnor/src/test/java/org/drools/guvnor/server/ServiceImplementationTest.java 2010-09-18 05:21:50 UTC (rev 35182)
@@ -735,6 +735,120 @@
assertTrue( asset.content instanceof RuleContentText );
}
+ public void testListAssets() throws Exception {
+ ServiceImplementation impl = getService();
+ PackageItem pacakgeItem = impl.repository.createPackage("testListAssetsPackage",
+ "desc");
+ impl.createCategory("",
+ "testListAssetsCat",
+ "this is a cat");
+
+ String uuid1 = impl.createNewRule("testLoadArchivedAssets1",
+ "description",
+ "testListAssetsCat",
+ "testListAssetsPackage",
+ AssetFormats.BUSINESS_RULE);
+
+ String uuid2 = impl.createNewRule("testLoadArchivedAssets2",
+ "description",
+ "testListAssetsCat",
+ "testListAssetsPackage",
+ AssetFormats.BUSINESS_RULE);
+
+ String uuid3 = impl.createNewRule("testLoadArchivedAssets3",
+ "description",
+ "testListAssetsCat",
+ "testListAssetsPackage",
+ AssetFormats.BUSINESS_RULE);
+
+ String uuid4 = impl.createNewRule("testLoadArchivedAssets4",
+ "description",
+ "testListAssetsCat",
+ "testListAssetsPackage",
+ AssetFormats.BUSINESS_RULE);
+
+ String uuid5 = impl.createNewRule("testLoadArchivedAssets5",
+ "description",
+ "testListAssetsCat",
+ "testListAssetsPackage",
+ AssetFormats.BUSINESS_RULE);
+
+
+ TableDataResult res = impl.listAssets(pacakgeItem.getUUID(), new String[]{AssetFormats.BUSINESS_RULE}, 0, 2, AssetItemGrid.PACKAGEVIEW_LIST_TABLE_ID);
+
+ assertEquals(2, res.data.length);
+ assertTrue(5 == res.total);
+ assertTrue(res.currentPosition == 2);
+ assertTrue(res.hasNext);
+
+ res = impl.listAssets(pacakgeItem.getUUID(), new String[]{AssetFormats.BUSINESS_RULE}, 2, 2, AssetItemGrid.PACKAGEVIEW_LIST_TABLE_ID);
+ assertEquals(2, res.data.length);
+ assertTrue(5 == res.total);
+ assertTrue(res.currentPosition == 4);
+ assertTrue(res.hasNext);
+ }
+
+ public void testLoadArchivedAssets() throws Exception {
+ ServiceImplementation impl = getService();
+ impl.repository.createPackage("testLoadArchivedAssetsPackage",
+ "desc");
+ impl.createCategory("",
+ "testLoadArchivedAssetsCat",
+ "this is a cat");
+
+ String uuid1 = impl.createNewRule("testLoadArchivedAssets1",
+ "description",
+ "testLoadArchivedAssetsCat",
+ "testLoadArchivedAssetsPackage",
+ AssetFormats.DRL);
+ impl.archiveAsset(uuid1);
+
+ String uuid2 = impl.createNewRule("testLoadArchivedAssets2",
+ "description",
+ "testLoadArchivedAssetsCat",
+ "testLoadArchivedAssetsPackage",
+ AssetFormats.DRL);
+ impl.archiveAsset(uuid2);
+
+ String uuid3 = impl.createNewRule("testLoadArchivedAssets3",
+ "description",
+ "testLoadArchivedAssetsCat",
+ "testLoadArchivedAssetsPackage",
+ AssetFormats.DRL);
+ impl.archiveAsset(uuid3);
+
+ String uuid4 = impl.createNewRule("testLoadArchivedAssets4",
+ "description",
+ "testLoadArchivedAssetsCat",
+ "testLoadArchivedAssetsPackage",
+ AssetFormats.DRL);
+ impl.archiveAsset(uuid4);
+
+ String uuid5 = impl.createNewRule("testLoadArchivedAssets5",
+ "description",
+ "testLoadArchivedAssetsCat",
+ "testLoadArchivedAssetsPackage",
+ AssetFormats.DRL);
+ impl.archiveAsset(uuid5);
+
+ //We do not know how many archived assets we have in the test repo,
+ //but definitely more than 5 (as we just created 5)
+ TableDataResult res = impl.loadArchivedAssets(0, 2);
+
+ assertEquals(2, res.data.length);
+ //may return -1 as per JCR2.0 when precise count is not available due to performance reasons.
+ //assertTrue(-1 != res.total);
+ assertTrue(res.currentPosition == 2);
+ assertTrue(res.hasNext);
+
+ res = impl.loadArchivedAssets(2, 2);
+
+ assertEquals(2, res.data.length);
+ //assertTrue(-1 != res.total);
+ assertEquals(res.currentPosition, 4);
+ assertTrue(res.hasNext);
+ }
+
public void testTrackRecentOpenedChanged() throws Exception {
ServiceImplementation impl = getService();
UserInbox ib = new UserInbox( impl.repository );
More information about the jboss-svn-commits
mailing list