------------------------------------------------------------
revno: 98
committer: Lars Helge Oeverland larshe...@gmail.com
branch nick: trunk
timestamp: Fri 2009-03-20 13:54:19 +0100
message:
  dhis-service-datamart: Added some comments.
modified:
  
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/AverageBoolAggregator.java
  
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/AverageIntAggregator.java
  
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/SumBoolAggregator.java
  
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/SumIntAggregator.java
  
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/dataelement/DefaultDataElementDataMart.java

=== modified file 
'dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/AverageBoolAggregator.java'
--- 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/AverageBoolAggregator.java
  2009-03-03 16:46:36 +0000
+++ 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/AverageBoolAggregator.java
  2009-03-20 12:54:19 +0000
@@ -127,20 +127,20 @@
                         value = 0.0;                        
                         relevantDays = 0.0;
                         
-                        if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentEndDate.compareTo( endDate ) <= 0 )
+                        if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentEndDate.compareTo( endDate ) <= 0 ) // Value is within period
                         {
                             relevantDays = getDays( currentEndDate ) - 
getDays( currentStartDate );
                         }
-                        else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( endDate ) >= 0 )
+                        else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( endDate ) >= 0 ) // Value spans whole period
                         {
                             relevantDays = getDays( endDate ) - getDays( 
startDate );
                         }
-                        else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( startDate ) >= 0
+                        else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( startDate ) >= 0 // Value spans period start
                             && currentEndDate.compareTo( endDate ) <= 0 )
                         {
                             relevantDays = getDays( currentEndDate ) - 
getDays( startDate );
                         }
-                        else if ( currentStartDate.compareTo( startDate ) >= 0 
&& currentStartDate.compareTo( endDate ) <= 0
+                        else if ( currentStartDate.compareTo( startDate ) >= 0 
&& currentStartDate.compareTo( endDate ) <= 0 // Value spans period end
                             && currentEndDate.compareTo( endDate ) >= 0 )
                         {
                             relevantDays = getDays( endDate ) - getDays( 
currentStartDate );

=== modified file 
'dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/AverageIntAggregator.java'
--- 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/AverageIntAggregator.java
   2009-03-03 16:46:36 +0000
+++ 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/AverageIntAggregator.java
   2009-03-20 12:54:19 +0000
@@ -144,21 +144,21 @@
                                         
                     relevantDays = 0.0;
                     
-                    if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentEndDate.compareTo( endDate ) <= 0 )
+                    if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentEndDate.compareTo( endDate ) <= 0 ) // Value is within period
                     {
                         relevantDays = getDays( currentEndDate ) - getDays( 
currentStartDate );
                     }
-                    else if ( currentStartDate.compareTo( startDate ) <= 0 && 
currentEndDate.compareTo( endDate ) >= 0 )
+                    else if ( currentStartDate.compareTo( startDate ) <= 0 && 
currentEndDate.compareTo( endDate ) >= 0 ) // Value spans whole period
                     {
                         relevantDays = getDays( endDate ) - getDays( startDate 
);
                     }
                     else if ( currentStartDate.compareTo( startDate ) <= 0 && 
currentEndDate.compareTo( startDate ) >= 0
-                        && currentEndDate.compareTo( endDate ) <= 0 )
+                        && currentEndDate.compareTo( endDate ) <= 0 ) // Value 
spans period start
                     {
                         relevantDays = getDays( currentEndDate ) - getDays( 
startDate );
                     }
                     else if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentStartDate.compareTo( endDate ) <= 0
-                        && currentEndDate.compareTo( endDate ) >= 0 )
+                        && currentEndDate.compareTo( endDate ) >= 0 ) // Value 
spans period end
                     {
                         relevantDays = getDays( endDate ) - getDays( 
currentStartDate );
                     }

=== modified file 
'dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/SumBoolAggregator.java'
--- 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/SumBoolAggregator.java
      2009-03-03 16:46:36 +0000
+++ 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/SumBoolAggregator.java
      2009-03-20 12:54:19 +0000
@@ -126,24 +126,24 @@
                         relevantDays = 0;
                         factor = 0;
 
-                        if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentEndDate.compareTo( endDate ) <= 0 )
+                        if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentEndDate.compareTo( endDate ) <= 0 ) // Value is within period
                         {
                             relevantDays = ( getDays( endDate ) - getDays( 
startDate ) );
                             factor = 1;
                         }
-                        else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( endDate ) >= 0 )
+                        else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( endDate ) >= 0 ) // Value spans whole period
                         {
                             relevantDays = ( getDays( endDate ) - getDays( 
startDate ) );
                             factor = relevantDays / duration;
                         }
                         else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( startDate ) >= 0
-                            && currentEndDate.compareTo( endDate ) <= 0 )
+                            && currentEndDate.compareTo( endDate ) <= 0 ) // 
Value spans period start
                         {
                             relevantDays = ( getDays( currentEndDate ) - 
getDays( startDate ) );
                             factor = relevantDays / duration;
                         }
                         else if ( currentStartDate.compareTo( startDate ) >= 0 
&& currentStartDate.compareTo( endDate ) <= 0
-                            && currentEndDate.compareTo( endDate ) >= 0 )
+                            && currentEndDate.compareTo( endDate ) >= 0 ) // 
Value spans period end
                         {
                             relevantDays = ( getDays( endDate ) - getDays( 
currentStartDate ) );
                             factor = relevantDays / duration;

=== modified file 
'dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/SumIntAggregator.java'
--- 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/SumIntAggregator.java
       2009-03-03 16:46:36 +0000
+++ 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/aggregation/dataelement/SumIntAggregator.java
       2009-03-20 12:54:19 +0000
@@ -140,29 +140,29 @@
                         relevantDays = 0;
                         factor = 0;
                         
-                        if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentEndDate.compareTo( endDate ) <= 0 )
+                        if ( currentStartDate.compareTo( startDate ) >= 0 && 
currentEndDate.compareTo( endDate ) <= 0 ) // Value is within period
                         {
                             relevantDays = getDays( endDate ) - getDays( 
startDate );
                             factor = 1;
                         }
-                        else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( endDate ) >= 0 )
+                        else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( endDate ) >= 0 ) // Value spans whole period
                         {
                             relevantDays = getDays( endDate ) - getDays( 
startDate );
                             factor = relevantDays / duration;
                         }
                         else if ( currentStartDate.compareTo( startDate ) <= 0 
&& currentEndDate.compareTo( startDate ) >= 0
-                            && currentEndDate.compareTo( endDate ) <= 0 )
+                            && currentEndDate.compareTo( endDate ) <= 0 ) // 
Value spans period start
                         {
                             relevantDays = getDays( currentEndDate ) - 
getDays( startDate );
                             factor = relevantDays / duration;
                         }
                         else if ( currentStartDate.compareTo( startDate ) >= 0 
&& currentStartDate.compareTo( endDate ) <= 0
-                            && currentEndDate.compareTo( endDate ) >= 0 )
+                            && currentEndDate.compareTo( endDate ) >= 0 ) // 
Value spans period end
                         {
                             relevantDays = getDays( endDate ) - getDays( 
currentStartDate );
                             factor = relevantDays / duration;
                         }
-
+                        
                         value = value * factor;
                         
                         existingValue = totalSums.containsKey( entry.getKey() 
) ? totalSums.get( entry.getKey() )[ 0 ] : 0;

=== modified file 
'dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/dataelement/DefaultDataElementDataMart.java'
--- 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/dataelement/DefaultDataElementDataMart.java
 2009-03-03 16:46:36 +0000
+++ 
dhis-2/dhis-services/dhis-service-datamart-default/src/main/java/org/hisp/dhis/datamart/dataelement/DefaultDataElementDataMart.java
 2009-03-20 12:54:19 +0000
@@ -129,10 +129,8 @@
                 {
                     value.clear();
                     
-                    Operand operand = entry.getKey();
-                    
-                    value.setDataElementId( operand.getDataElementId() );
-                    value.setCategoryOptionComboId( operand.getOptionComboId() 
);
+                    value.setDataElementId( entry.getKey().getDataElementId() 
);
+                    value.setCategoryOptionComboId( 
entry.getKey().getOptionComboId() );
                     value.setPeriodId( period.getId() );
                     value.setPeriodTypeId( periodType.getId() );
                     value.setOrganisationUnitId( unit.getId() );
@@ -155,7 +153,7 @@
     // Id-to-object methods
     // 
-------------------------------------------------------------------------
 
-    private Collection<Period> getPeriods( Collection<Integer> periodIds )
+    private Collection<Period> getPeriods( final Collection<Integer> periodIds 
)
     {
         final Set<Period> periods = new HashSet<Period>( periodIds.size() );
         
@@ -167,7 +165,7 @@
         return periods;
     }
     
-    private Collection<OrganisationUnit> getOrganisationUnits( 
Collection<Integer> organisationUnitIds )
+    private Collection<OrganisationUnit> getOrganisationUnits( final 
Collection<Integer> organisationUnitIds )
     {
         final Set<OrganisationUnit> organisationUnits = new 
HashSet<OrganisationUnit>( organisationUnitIds.size() );
         



--

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.

_______________________________________________
Mailing list: https://launchpad.net/~dhis2-devs
Post to     : dhis2-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~dhis2-devs
More help   : https://help.launchpad.net/ListHelp

Reply via email to