← Back to team overview

dhis2-devs team mailing list archive

[Branch ~dhis2-devs-core/dhis2/trunk] Rev 18355: remove all Query/Order methods from IdObjectManager, should be used through QueryService instead

 

------------------------------------------------------------
revno: 18355
committer: Morten Olav Hansen <mortenoh@xxxxxxxxx>
branch nick: dhis2
timestamp: Fri 2015-02-20 16:31:37 +0700
message:
  remove all Query/Order methods from IdObjectManager, should be used through QueryService instead
modified:
  dhis-2/dhis-api/src/main/java/org/hisp/dhis/common/IdentifiableObjectManager.java
  dhis-2/dhis-services/dhis-service-core/src/main/java/org/hisp/dhis/common/DefaultIdentifiableObjectManager.java
  dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/common/IdentifiableObjectManagerTest.java


--
lp:dhis2
https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk

Your team DHIS 2 developers is subscribed to branch lp:dhis2.
To unsubscribe from this branch go to https://code.launchpad.net/~dhis2-devs-core/dhis2/trunk/+edit-subscription
=== modified file 'dhis-2/dhis-api/src/main/java/org/hisp/dhis/common/IdentifiableObjectManager.java'
--- dhis-2/dhis-api/src/main/java/org/hisp/dhis/common/IdentifiableObjectManager.java	2015-02-18 13:43:10 +0000
+++ dhis-2/dhis-api/src/main/java/org/hisp/dhis/common/IdentifiableObjectManager.java	2015-02-20 09:31:37 +0000
@@ -29,10 +29,6 @@
  */
 
 import org.hisp.dhis.common.NameableObject.NameableProperty;
-import org.hisp.dhis.query.Order;
-import org.hisp.dhis.query.Query;
-import org.hisp.dhis.query.Result;
-import org.hisp.dhis.query.ResultTransformer;
 
 import java.util.Collection;
 import java.util.Date;
@@ -73,10 +69,6 @@
 
     <T extends IdentifiableObject> Collection<T> getAll( Class<T> clazz );
 
-    <T extends IdentifiableObject> Collection<T> getAll( Class<T> clazz, Order order );
-
-    <T extends IdentifiableObject> Collection<T> getAll( Class<T> clazz, List<Order> order );
-
     <T extends IdentifiableObject> Collection<T> getAllByName( Class<T> clazz, String name );
 
     <T extends IdentifiableObject> Collection<T> getAllByNameIgnoreCase( Class<T> clazz, String name );
@@ -93,10 +85,6 @@
 
     <T extends IdentifiableObject> List<T> getBetween( Class<T> clazz, int first, int max );
 
-    <T extends IdentifiableObject> List<T> getBetween( Class<T> clazz, int first, int max, Order order );
-
-    <T extends IdentifiableObject> List<T> getBetween( Class<T> clazz, int first, int max, List<Order> order );
-
     <T extends IdentifiableObject> List<T> getBetweenSorted( Class<T> clazz, int first, int max );
 
     <T extends IdentifiableObject> List<T> getBetweenLikeName( Class<T> clazz, String name, int first, int max );
@@ -145,10 +133,6 @@
 
     <T extends IdentifiableObject> int getCountLikeShortName( Class<T> clazz, String shortName );
 
-    Result query( Query query );
-
-    Result query( Query query, ResultTransformer transformer );
-
     void refresh( Object object );
 
     // -------------------------------------------------------------------------

=== modified file 'dhis-2/dhis-services/dhis-service-core/src/main/java/org/hisp/dhis/common/DefaultIdentifiableObjectManager.java'
--- dhis-2/dhis-services/dhis-service-core/src/main/java/org/hisp/dhis/common/DefaultIdentifiableObjectManager.java	2015-02-18 13:43:10 +0000
+++ dhis-2/dhis-services/dhis-service-core/src/main/java/org/hisp/dhis/common/DefaultIdentifiableObjectManager.java	2015-02-20 09:31:37 +0000
@@ -28,17 +28,12 @@
  * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-import com.google.common.collect.Lists;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.hibernate.SessionFactory;
 import org.hisp.dhis.common.NameableObject.NameableProperty;
 import org.hisp.dhis.common.comparator.IdentifiableObjectNameComparator;
-import org.hisp.dhis.query.Order;
-import org.hisp.dhis.query.Query;
 import org.hisp.dhis.query.QueryService;
-import org.hisp.dhis.query.Result;
-import org.hisp.dhis.query.ResultTransformer;
 import org.hisp.dhis.user.UserCredentials;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.transaction.annotation.Transactional;
@@ -275,26 +270,6 @@
     }
 
     @Override
-    public <T extends IdentifiableObject> Collection<T> getAll( Class<T> clazz, Order order )
-    {
-        return getAll( clazz, Lists.newArrayList( order ) );
-    }
-
-    @Override
-    @SuppressWarnings( "unchecked" )
-    public <T extends IdentifiableObject> Collection<T> getAll( Class<T> clazz, List<Order> order )
-    {
-        GenericIdentifiableObjectStore<IdentifiableObject> store = getIdentifiableObjectStore( clazz );
-
-        if ( store == null )
-        {
-            return new ArrayList<>();
-        }
-
-        return (Collection<T>) store.getAll( order );
-    }
-
-    @Override
     @SuppressWarnings( "unchecked" )
     public <T extends IdentifiableObject> Collection<T> getAllByName( Class<T> clazz, String name )
     {
@@ -498,26 +473,6 @@
     }
 
     @Override
-    public <T extends IdentifiableObject> List<T> getBetween( Class<T> clazz, int first, int max, Order order )
-    {
-        return getBetween( clazz, first, max, Lists.newArrayList( order ) );
-    }
-
-    @Override
-    @SuppressWarnings( "unchecked" )
-    public <T extends IdentifiableObject> List<T> getBetween( Class<T> clazz, int first, int max, List<Order> order )
-    {
-        GenericIdentifiableObjectStore<IdentifiableObject> store = getIdentifiableObjectStore( clazz );
-
-        if ( store == null )
-        {
-            return new ArrayList<>();
-        }
-
-        return (List<T>) store.getAll( first, max, order );
-    }
-
-    @Override
     @SuppressWarnings( "unchecked" )
     public <T extends IdentifiableObject> List<T> getBetweenSorted( Class<T> clazz, int first, int max )
     {
@@ -863,18 +818,6 @@
     }
 
     @Override
-    public Result query( Query query )
-    {
-        return queryService.query( query );
-    }
-
-    @Override
-    public Result query( Query query, ResultTransformer transformer )
-    {
-        return queryService.query( query, transformer );
-    }
-
-    @Override
     @SuppressWarnings( "unchecked" )
     public <T extends IdentifiableObject> T getNoAcl( Class<T> clazz, String uid )
     {

=== modified file 'dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/common/IdentifiableObjectManagerTest.java'
--- dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/common/IdentifiableObjectManagerTest.java	2015-02-17 08:00:44 +0000
+++ dhis-2/dhis-services/dhis-service-core/src/test/java/org/hisp/dhis/common/IdentifiableObjectManagerTest.java	2015-02-20 09:31:37 +0000
@@ -571,70 +571,4 @@
         assertTrue( cd.contains( dataElementC ) );
         assertTrue( cd.contains( dataElementD ) );
     }
-
-    @Test
-    public void orderTest()
-    {
-        DataElement dataElementC = createDataElement( 'C' );
-        DataElement dataElementA = createDataElement( 'A' );
-        DataElement dataElementD = createDataElement( 'D' );
-        DataElement dataElementB = createDataElement( 'B' );
-
-        identifiableObjectManager.save( dataElementA );
-        identifiableObjectManager.save( dataElementB );
-        identifiableObjectManager.save( dataElementC );
-        identifiableObjectManager.save( dataElementD );
-
-        Property property = new Property( DataElement.class, null, null );
-        property.setSimple( true );
-        property.setPersisted( true );
-        property.setFieldName( "name" );
-
-        List<DataElement> asc = new ArrayList<>( identifiableObjectManager.getAll( DataElement.class, Order.asc( property ) ) );
-        assertTrue( "DataElementA".equals( asc.get( 0 ).getName() ) );
-        assertTrue( "DataElementB".equals( asc.get( 1 ).getName() ) );
-        assertTrue( "DataElementC".equals( asc.get( 2 ).getName() ) );
-        assertTrue( "DataElementD".equals( asc.get( 3 ).getName() ) );
-
-        List<DataElement> desc = new ArrayList<>( identifiableObjectManager.getAll( DataElement.class, Order.desc( property ) ) );
-        assertTrue( "DataElementD".equals( desc.get( 0 ).getName() ) );
-        assertTrue( "DataElementC".equals( desc.get( 1 ).getName() ) );
-        assertTrue( "DataElementB".equals( desc.get( 2 ).getName() ) );
-        assertTrue( "DataElementA".equals( desc.get( 3 ).getName() ) );
-    }
-
-    @Test
-    public void pagedOrderTest()
-    {
-        DataElement dataElementC = createDataElement( 'C' );
-        DataElement dataElementA = createDataElement( 'A' );
-        DataElement dataElementD = createDataElement( 'D' );
-        DataElement dataElementB = createDataElement( 'B' );
-
-        identifiableObjectManager.save( dataElementA );
-        identifiableObjectManager.save( dataElementB );
-        identifiableObjectManager.save( dataElementC );
-        identifiableObjectManager.save( dataElementD );
-
-        Property property = new Property( DataElement.class, null, null );
-        property.setSimple( true );
-        property.setPersisted( true );
-        property.setFieldName( "name" );
-
-        List<DataElement> asc = new ArrayList<>( identifiableObjectManager.getBetween( DataElement.class, 0, 2, Order.asc( property ) ) );
-        assertTrue( "DataElementA".equals( asc.get( 0 ).getName() ) );
-        assertTrue( "DataElementB".equals( asc.get( 1 ).getName() ) );
-
-        asc = new ArrayList<>( identifiableObjectManager.getBetween( DataElement.class, 2, 2, Order.asc( property ) ) );
-        assertTrue( "DataElementC".equals( asc.get( 0 ).getName() ) );
-        assertTrue( "DataElementD".equals( asc.get( 1 ).getName() ) );
-
-        List<DataElement> desc = new ArrayList<>( identifiableObjectManager.getBetween( DataElement.class, 0, 2, Order.desc( property ) ) );
-        assertTrue( "DataElementD".equals( desc.get( 0 ).getName() ) );
-        assertTrue( "DataElementC".equals( desc.get( 1 ).getName() ) );
-
-        desc = new ArrayList<>( identifiableObjectManager.getBetween( DataElement.class, 2, 2, Order.desc( property ) ) );
-        assertTrue( "DataElementB".equals( desc.get( 0 ).getName() ) );
-        assertTrue( "DataElementA".equals( desc.get( 1 ).getName() ) );
-    }
 }