Hello,

I think you caught another bug with
    else if (nbFeatureWithin == 1) appearing two times
but
    else if (nbFeatureEqualAndWithin == 1)
is different from
    else if (nbFeatureEqualOrWithin == 1)

I'm close to get a write access to CVS
Hope next time, i'll fix it myself

Michaël


Stefan Steiniger a écrit :

>Hei Michael,
>
>thanx! i fixed it. But i wonder why the following lines appear two times
>
>else if (nbFeatureWithin == 1){
>       mapping.transferAttributes(fEqual, aFeature, feature);
>       feature.setGeometry((Geometry) aFeature.getGeometry().clone());
>                       fcRecup.add(feature);
>       }
>
>same for
>else if (nbFeatureEqualAndWithin == 1){
>...
>
>best greetings to Vincennes
>stefan
>
>@all:
>sorry for being currently not that active, but my agenda has some other 
>priorities. I will also try to have a look on SkyJUMP these days.
>BTW: the changes for the translation of the imagery framework are done 
>and commited.. But i have not yet commited the ECW dlls - also because 
>of the email by  Jan corresponding the GPL and the ecw license terms
>
>Michaël Michaud schrieb:
>  
>
>>Hi Sunburned :
>>
>>There is probably a "copy/paste" bug line 173 of SpatialJoinPlugIn class :
>>
>>else if (methodName.equals(METHOD_WITHIN)) {
>>    if (aFeature.getGeometry().within(bFeature.getGeometry())) {
>>    nbFeatureWithin++;
>>    nbFeature++;
>>    fEqual = bFeature;
>>}
>>
>>It should be fWithin instead of fEqual
>>
>>I did not try to recompile to check if it solves the problem
>>
>>Michaël
>>
>>Sunburned Surveyor a écrit :
>>
>>    
>>
>>>Sorry,
>>>
>>>I meant to label this as Bug #1546889, but I forgot to put it in the 
>>>subject line.
>>>
>>>SS
>>>
>>>On 1/18/07, *Sunburned Surveyor* < [EMAIL PROTECTED] 
>>><mailto:[EMAIL PROTECTED]>> wrote:
>>>
>>>    I'm working on my second bug for the 01.00.02 OpenJUMP release.
>>>    This bug was reported by Johannes Metzler on 2006-08-25. His error
>>>    involved the Sigle-Geoprocessing "Transfer Attributes" command. He
>>>    got a NullPointerException while using this command, and I did as
>>>    well. (I did get it to work one of the times I attempted to use
>>>    it, however.)
>>>
>>>    Johannes wrote this in his bug report: "The new function
>>>    "Geoprocessing > two layers > Transfer attributes" doesnt seems to
>>>    work (or I use it in the wrong way, a description would be
>>>    helpfull)..."
>>>
>>>    I agree with this comment. Would it be possible to get a
>>>    description of this command from the SIGLE team with some brief
>>>    instructions on how it can and can't be used. If this can be done
>>>    I will take a closer look at the bug Johannes reported.
>>>
>>>    Some sample data to test the command with would also be really
>>>    great, but not absolutely necessary.
>>>
>>>    I would like to thank the SIGLE team in advance for their help
>>>    with Bug 2 of 3...
>>>
>>>    The Sunburned Surveyor
>>>
>>>
>>>------------------------------------------------------------------------
>>>
>>>-------------------------------------------------------------------------
>>>Take Surveys. Earn Cash. Influence the Future of IT
>>>Join SourceForge.net's Techsay panel and you'll get the chance to share your
>>>opinions on IT & business topics through brief surveys - and earn cash
>>>http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
>>>
>>>------------------------------------------------------------------------
>>>
>>>_______________________________________________
>>>Jump-pilot-devel mailing list
>>>Jump-pilot-devel@lists.sourceforge.net
>>>https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>> 
>>>
>>>      
>>>
>>-------------------------------------------------------------------------
>>Take Surveys. Earn Cash. Influence the Future of IT
>>Join SourceForge.net's Techsay panel and you'll get the chance to share your
>>opinions on IT & business topics through brief surveys - and earn cash
>>http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
>>_______________________________________________
>>Jump-pilot-devel mailing list
>>Jump-pilot-devel@lists.sourceforge.net
>>https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>
>>
>>    
>>
>
>-------------------------------------------------------------------------
>Take Surveys. Earn Cash. Influence the Future of IT
>Join SourceForge.net's Techsay panel and you'll get the chance to share your
>opinions on IT & business topics through brief surveys - and earn cash
>http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
>_______________________________________________
>Jump-pilot-devel mailing list
>Jump-pilot-devel@lists.sourceforge.net
>https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
>
>  
>


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to