← Back to team overview

dhis2-devs team mailing list archive

[Branch ~dhis2-devs-core/dhis2/trunk] Rev 3013: Removed check for compulsory dataelement/indicator group sets in integrity check as there is no s...

 

------------------------------------------------------------
revno: 3013
committer: Lars Helge Overland <larshelge@xxxxxxxxx>
branch nick: dhis2
timestamp: Sun 2011-03-13 06:02:23 +0100
message:
  Removed check for compulsory dataelement/indicator group sets in integrity check as there is no such thing anymore
modified:
  dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataintegrity/DataIntegrityService.java
  dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/dataintegrity/DefaultDataIntegrityService.java
  dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataadmin/src/main/java/org/hisp/dhis/dataadmin/action/dataintegrity/GetDataIntegrityAction.java
  dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataadmin/src/main/webapp/dhis-web-maintenance-dataadmin/dataIntegrityForm.vm


--
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/dataintegrity/DataIntegrityService.java'
--- dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataintegrity/DataIntegrityService.java	2010-11-29 17:16:41 +0000
+++ dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataintegrity/DataIntegrityService.java	2011-03-13 05:02:23 +0000
@@ -65,12 +65,6 @@
     Collection<DataElement> getDataElementsWithoutGroups();
     
     /**
-     * Gets all data elements which are not members of one or more groups
-     * which enter into a compulsory group set.
-     */
-    Collection<DataElement> getDataElementsViolatingCompulsoryGroupSets();
-    
-    /**
      * Gets all data elements units which are members of more than one group
      * which enter into an exclusive group set.
      */    
@@ -126,12 +120,6 @@
     SortedMap<Indicator, String> getInvalidIndicatorDenominators();
     
     /**
-     * Gets all indicators which are not members of one or more groups
-     * which enter into a compulsory group set.
-     */
-    Collection<Indicator> getIndicatorsViolatingCompulsoryGroupSets();
-    
-    /**
      * Gets all indicators units which are members of more than one group
      * which enter into an exclusive group set.
      */    

=== modified file 'dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/dataintegrity/DefaultDataIntegrityService.java'
--- dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/dataintegrity/DefaultDataIntegrityService.java	2010-11-29 17:16:41 +0000
+++ dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/dataintegrity/DefaultDataIntegrityService.java	2011-03-13 05:02:23 +0000
@@ -188,30 +188,6 @@
     }
 
     @Override
-    public Collection<DataElement> getDataElementsViolatingCompulsoryGroupSets()
-    {
-        Collection<DataElementGroupSet> groupSets = dataElementService.getAllDataElementGroupSets();
-
-        Collection<DataElement> dataElements = dataElementService.getAllDataElements();
-
-        Set<DataElement> targets = new HashSet<DataElement>();
-
-        for ( DataElement de : dataElements )
-        {
-            for ( DataElementGroupSet groupSet : groupSets )
-            {
-                if ( !CollectionUtils.containsAny( groupSet.getMembers(), de.getGroups() ) )
-                {
-                    targets.add( de );
-                }
-            }
-        }
-
-        return targets;
-
-    }
-
-    @Override
     public Collection<DataElement> getDataElementsViolatingExclusiveGroupSets()
     {
         Collection<DataElementGroupSet> groupSets = dataElementService.getAllDataElementGroupSets();
@@ -345,29 +321,6 @@
     }
 
     @Override
-    public Collection<Indicator> getIndicatorsViolatingCompulsoryGroupSets()
-    {
-        Collection<IndicatorGroupSet> groupSets = indicatorService.getAllIndicatorGroupSets();
-
-        Collection<Indicator> indicators = indicatorService.getAllIndicators();
-
-        Set<Indicator> targets = new HashSet<Indicator>();
-
-        for ( Indicator in :  indicators)
-        {
-            for ( IndicatorGroupSet groupSet : groupSets )
-            {
-                if ( !CollectionUtils.containsAny( groupSet.getMembers(), in.getGroups() ) )
-                {
-                    targets.add( in );
-                }
-            }
-        }
-
-        return targets;
-    }
-
-    @Override
     public Collection<Indicator> getIndicatorsViolatingExclusiveGroupSets()
     {
         Collection<IndicatorGroupSet> groupSets = indicatorService.getAllIndicatorGroupSets();

=== modified file 'dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataadmin/src/main/java/org/hisp/dhis/dataadmin/action/dataintegrity/GetDataIntegrityAction.java'
--- dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataadmin/src/main/java/org/hisp/dhis/dataadmin/action/dataintegrity/GetDataIntegrityAction.java	2011-01-20 11:39:27 +0000
+++ dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataadmin/src/main/java/org/hisp/dhis/dataadmin/action/dataintegrity/GetDataIntegrityAction.java	2011-03-13 05:02:23 +0000
@@ -91,13 +91,6 @@
         return dataElementsWithoutGroups;
     }
 
-    private List<DataElement> dataElementsViolatingCompulsoryGroupSets;
-
-    public List<DataElement> getDataElementsViolatingCompulsoryGroupSets()
-    {
-        return dataElementsViolatingCompulsoryGroupSets;
-    }
-
     private List<DataElement> dataElementsViolatingExclusiveGroupSets;
 
     public List<DataElement> getDataElementsViolatingExclusiveGroupSets()
@@ -154,13 +147,6 @@
         return invalidIndicatorDenominators;
     }
 
-    private List<Indicator> indicatorsViolatingCompulsoryGroupSets;
-
-    public List<Indicator> getIndicatorsViolatingCompulsoryGroupSets()
-    {
-        return indicatorsViolatingCompulsoryGroupSets;
-    }
-
     private List<Indicator> indicatorsViolatingExclusiveGroupSets;
 
     public List<Indicator> getIndicatorsViolatingExclusiveGroupSets()
@@ -240,7 +226,6 @@
         dataElementsWithoutDataSet = new ArrayList<DataElement>( dataIntegrityService.getDataElementsWithoutDataSet() );
         dataElementsWithoutGroups = new ArrayList<DataElement>( dataIntegrityService.getDataElementsWithoutGroups() );
         dataElementsAssignedToDataSetsWithDifferentPeriodTypes = dataIntegrityService.getDataElementsAssignedToDataSetsWithDifferentPeriodTypes();
-        dataElementsViolatingCompulsoryGroupSets = new ArrayList<DataElement>( dataIntegrityService.getDataElementsViolatingCompulsoryGroupSets() );
         dataElementsViolatingExclusiveGroupSets = new ArrayList<DataElement>( dataIntegrityService.getDataElementsViolatingExclusiveGroupSets() );
 
         log.info( "Checked data elements" );
@@ -254,7 +239,6 @@
         indicatorsWithoutGroups = new ArrayList<Indicator>( dataIntegrityService.getIndicatorsWithoutGroups() );
         invalidIndicatorNumerators = dataIntegrityService.getInvalidIndicatorNumerators();
         invalidIndicatorDenominators = dataIntegrityService.getInvalidIndicatorDenominators();
-        indicatorsViolatingCompulsoryGroupSets = new ArrayList<Indicator>( dataIntegrityService.getIndicatorsViolatingCompulsoryGroupSets() );
         indicatorsViolatingExclusiveGroupSets = new ArrayList<Indicator>( dataIntegrityService.getIndicatorsViolatingExclusiveGroupSets() );
 
         log.info( "Checked indicators" );
@@ -283,12 +267,10 @@
         
         Collections.sort( dataElementsWithoutDataSet, new DataElementNameComparator() );
         Collections.sort( dataElementsWithoutGroups, new DataElementNameComparator() );
-        Collections.sort( dataElementsViolatingCompulsoryGroupSets, new DataElementNameComparator() );
         Collections.sort( dataElementsViolatingExclusiveGroupSets, new DataElementNameComparator() );
         Collections.sort( dataSetsNotAssignedToOrganisationUnits, new DataSetNameComparator() );
         Collections.sort( sectionsWithInvalidCategoryCombinations, new SectionOrderComparator() );
         Collections.sort( indicatorsWithoutGroups, new IndicatorNameComparator() );
-        Collections.sort( indicatorsViolatingCompulsoryGroupSets, new IndicatorNameComparator() );
         Collections.sort( indicatorsViolatingExclusiveGroupSets, new IndicatorNameComparator() );
         Collections.sort( organisationUnitsWithCyclicReferences, new OrganisationUnitNameComparator() );
         Collections.sort( orphanedOrganisationUnits, new OrganisationUnitNameComparator() );

=== modified file 'dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataadmin/src/main/webapp/dhis-web-maintenance-dataadmin/dataIntegrityForm.vm'
--- dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataadmin/src/main/webapp/dhis-web-maintenance-dataadmin/dataIntegrityForm.vm	2010-11-29 17:16:41 +0000
+++ dhis-2/dhis-web/dhis-web-maintenance/dhis-web-maintenance-dataadmin/src/main/webapp/dhis-web-maintenance-dataadmin/dataIntegrityForm.vm	2011-03-13 05:02:23 +0000
@@ -14,7 +14,6 @@
 
 #integrityItem( $i18n.getString( "data_elements_without_data_set" ) "dataElementsWithoutDataSet" )
 #integrityItem( $i18n.getString( "data_elements_without_groups" ) "dataElementsWithoutGroups" )
-#integrityItem( $i18n.getString( "data_elements_violating_compulsory_group_sets" ) "dataElementsViolatingCompulsoryGroupSets" )
 #integrityItem( $i18n.getString( "data_elements_violating_exclusive_group_sets" ) "dataElementsViolatingExclusiveGroupSets" )
 #integrityItem( $i18n.getString( "data_elements_assigned_to_period_types_with_different_period_types" ) "dataElementsAssignedToDataSetsWithDifferentPeriodTypes" )
 #integrityItem( $i18n.getString( "data_sets_not_assigned_to_organisation_units" ) "dataSetsNotAssignedToOrganisationUnits" )
@@ -23,7 +22,6 @@
 #integrityItem( $i18n.getString( "indicators_without_groups" ) "indicatorsWithoutGroups" )
 #integrityItem( $i18n.getString( "invalid_indicator_numerators" ) "invalidIndicatorNumerators" )
 #integrityItem( $i18n.getString( "invalid_indicator_denominators" ) "invalidIndicatorDenominators" )
-#integrityItem( $i18n.getString( "indicators_violating_compulsory_group_sets" ) "indicatorsViolatingCompulsoryGroupSets" )
 #integrityItem( $i18n.getString( "indicators_violating_exclusive_group_sets" ) "indicatorsViolatingExclusiveGroupSets" )
 #integrityItem( $i18n.getString( "organisation_units_with_cyclic_references" ) "organisationUnitsWithCyclicReferences" )
 #integrityItem( $i18n.getString( "orphaned_organisation_units" ) "orphanedOrganisationUnits" )