[OpenLayers-Dev] duplicate attribution

[Christopher Schmidt]

On Thu, Oct 08, 2009 at 10:06:25AM -0600, Tim Schaub wrote:
> Hey-
> 
> Marc Jansen wrote:
> > Hi Tim,
> > 
> > thanks for the review. Renaming seems reasonable, I guess there is no 
> > wrong or right.
> > 
> > As for the default: I chose not to break the behaviour of the control 
> > when being called without the parameter "eliminateDuplicates" set. On 
> > the other hand: I can hardly guess of any situation where you wanted 
> > duplicate atttribution texts.
> > 
> 
> Yeah, I would suggest that it is a bug to show the same attribution 
> twice.  In that light, changing the behavior would be fixing something 
> instead of breaking something.
> 
> Anybody else think that displaying the same attribution text multiple 
> times makes sense?
> 
> I'm guessing there is not a strong opinion one way or another.  I do 
> like the idea that at some point we can streamline the API, instead of 
> continually adding to it (particularly in cases where we are adding 
> things to work around somewhat odd behavior).

REviewed it, though honestly, I'd be willing to drop the option, since it
seems silly to maintain buggy behavior, and I can't imagine a case where
this behavior makes sense.

-- Chris

> http://trac.openlayers.org/ticket/2300
> 
> Tim
> 
> > Just my two cents,
> > 
> > Regards,
> > Marc
> > 
> > 
> > Tim Schaub schrieb:
> >> Thanks for the excellent contribution Marc.
> >>
> >> http://trac.openlayers.org/changeset/9720
> >>
> >> I think it would be worth making eliminateDuplicates true by default 
> >> (and maybe changing it to simply "duplicates" and making it false).
> >>
> >> Would anyone object?
> >>
> >> Tim
> >>
> >> Marc Jansen wrote:
> >>  
> >>> Hi Bart,
> >>>
> >>> thanks for the warm welcome :-)
> >>>
> >>> Bart van den Eijnden (OSGIS) schrieb:
> >>>    
> >>>> Hi Marc,
> >>>>
> >>>> welcome to the community, you're almost there (opening a ticket with 
> >>>> a patch is the right way), a few pointers:
> >>>>
> >>>> -it is wise to set the state of your ticket to review, also if you 
> >>>> want it to get attention, make sure the milestone is the next 
> >>>> possible release (right now 2.9)
> >>>>       
> >>> OK. I'll change that
> >>>
> >>>    
> >>>> -a patch with a testcase is better than a patch without a testcase, 
> >>>> so if you could look into how to write a testcase that would be great
> >>>>       
> >>> OK. I will try to do that as well.
> >>>
> >>>    
> >>>> -a patch should ideally work against trunk
> >>>>
> >>>>       
> >>> Both patches work against the latest version of the trunk... or am I 
> >>> misinterpreing something,
> >>>
> >>> Regards,
> >>> Marc
> >>>
> >>>    
> >>>> Best regards,
> >>>> Bart
> >>>>
> >>>> Marc Jansen wrote:
> >>>>      
> >>>>> Hi devs,
> >>>>>
> >>>>> I recently opened two tickets ([1], [2]) for new / enhanced 
> >>>>> functionality of OpenLayers components. Both of them have (more or 
> >>>>> less trivial) patches attached to provide the actual functionality.
> >>>>>
> >>>>> I would love to have a short note/review about the code in question 
> >>>>> by someone with deeper insights in the project.
> >>>>>
> >>>>> I am not quite sure if I did everything the "OpenLayers-Way". Is 
> >>>>> the process of opening a ticket and submitting a patch the desired 
> >>>>> way? Should I have started a discussion on this list prior to 
> >>>>> opening the ticket? Please help me since I would love to 
> >>>>> participate in the project.
> >>>>>
> >>>>> BTW: I have not handed in a CLA up until now. Maybe we can fix this 
> >>>>> in Sydney, as I guess many from the project will meet up there.
> >>>>>
> >>>>> Regards,
> >>>>> Marc
> >>>>>
> >>>>> [1] http://trac.openlayers.org/ticket/2266
> >>>>> [2] http://trac.openlayers.org/ticket/2288
> >>>>> _______________________________________________
> >>>>> Dev mailing list
> >>>>> Dev at openlayers.org
> >>>>> http://openlayers.org/mailman/listinfo/dev
> >>>>>
> >>>>>
> >>>>>           
> >>> _______________________________________________
> >>> Dev mailing list
> >>> Dev at openlayers.org
> >>> http://openlayers.org/mailman/listinfo/dev
> >>>     
> >>
> >>
> >>   
> > 
> 
> 
> -- 
> Tim Schaub
> OpenGeo - http://opengeo.org
> Expert service straight from the developers.
> _______________________________________________
> Dev mailing list
> Dev at openlayers.org
> http://openlayers.org/mailman/listinfo/dev

-- 
Christopher Schmidt
MetaCarta