← Back to team overview

dhis2-devs team mailing list archive

Re: [Branch ~dhis2-devs-core/dhis2/trunk] Rev 1749: Doing it this way should help (but don't like these untyped apis requiring unsafe casting)

 

The problem seems to have come about with the addition of the
MinMaxValuesGeneratingService which also wants to implement the
DataAnalysisService but doesn't have the same return type to the
analyse() method.  So Tran made the return type of the interface
generic to accomodate both.  I'm not sure if this was a good idea.

Tran is there a good reason for  MinMaxValuesGeneratingService to
implement DataAnalysisService?  It looks wrong to me.  And it makes Jo
not feel very well.

Regards
Bob

On 7 April 2010 21:58,  <noreply@xxxxxxxxxxxxx> wrote:
> ------------------------------------------------------------
> revno: 1749
> committer: Jo Størset <storset@xxxxxxxxx>
> branch nick: trunk
> timestamp: Wed 2010-04-07 22:55:29 +0200
> message:
>  Doing it this way should help (but don't like these untyped apis requiring unsafe casting)
> modified:
>  dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataanalysis/DataAnalysisService.java
>  dhis-2/dhis-web/dhis-web-validationrule/src/main/java/org/hisp/dhis/validationrule/action/dataanalysis/GetAnalysisAction.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/dataanalysis/DataAnalysisService.java'
> --- dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataanalysis/DataAnalysisService.java   2010-04-07 20:33:09 +0000
> +++ dhis-2/dhis-api/src/main/java/org/hisp/dhis/dataanalysis/DataAnalysisService.java   2010-04-07 20:55:29 +0000
> @@ -30,7 +30,6 @@
>  import java.util.Collection;
>
>  import org.hisp.dhis.dataelement.DataElement;
> -import org.hisp.dhis.datavalue.DeflatedDataValue;
>  import org.hisp.dhis.organisationunit.OrganisationUnit;
>  import org.hisp.dhis.period.Period;
>
> @@ -43,6 +42,6 @@
>  {
>     String ID = DataAnalysisService.class.getName();
>
> -    Collection<DeflatedDataValue> analyse( OrganisationUnit organisationUnit, Collection<DataElement> dataElements,
> +    Collection<?> analyse( OrganisationUnit organisationUnit, Collection<DataElement> dataElements,
>         Collection<Period> periods, Double stdDevFactor );
>  }
>
> === modified file 'dhis-2/dhis-web/dhis-web-validationrule/src/main/java/org/hisp/dhis/validationrule/action/dataanalysis/GetAnalysisAction.java'
> --- dhis-2/dhis-web/dhis-web-validationrule/src/main/java/org/hisp/dhis/validationrule/action/dataanalysis/GetAnalysisAction.java       2010-02-02 13:35:59 +0000
> +++ dhis-2/dhis-web/dhis-web-validationrule/src/main/java/org/hisp/dhis/validationrule/action/dataanalysis/GetAnalysisAction.java       2010-04-07 20:55:29 +0000
> @@ -220,7 +220,7 @@
>         {
>             log.info( "Data analysis type: " + key );
>
> -            dataValues = service.analyse( organisationUnit, dataElements, periods, standardDeviation );
> +            dataValues = (Collection<DeflatedDataValue>) service.analyse( organisationUnit, dataElements, periods, standardDeviation );
>         }
>
>         return SUCCESS;
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~dhis2-devs
> Post to     : dhis2-devs@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~dhis2-devs
> More help   : https://help.launchpad.net/ListHelp
>
>



Follow ups

References