dhis2-devs team mailing list archive
-
dhis2-devs team
-
Mailing list archive
-
Message #05337
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)
On Wed, Apr 7, 2010 at 11:34 PM, Bob Jolliffe <bobjolliffe@xxxxxxxxx> wrote:
> 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.
>
>
Argh I had the release branch source code in eclipse and was compiling trunk
:-(
> 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
> >
> >
>
> _______________________________________________
> 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
>
References