[ka-Map-users] code changes in cvs

[Brent Pedersen]

a forgotten prototype!
that does it, thanks.


On 9/27/05, Paul Spencer <pspencer at dmsolutions.ca> wrote:
>
> Brent,
>
> thanks for picking this up. I could have sworn that was working when
> I committed it, now it isn't. Well, actually it is, but I just fixed
> it and committed the change to cvs. Please give it a try :)
>
> Cheers
>
> Paul
>
> On 27-Sep-05, at 5:18 PM, Brent Pedersen wrote:
>
> > hi paul, thanks for all the great work.
> > i get some errors with the new cvs version:
> >
> > 1. in my javascript console: kaTool_onmouseout is not defined
> > (kaTool.js line 227)
> >
> > 2. also in js console 'myKaQuery.activate is not a function'
> > and the query tool doesn't work. i can fix that by changing to
> > myKaMap.activateTool(myKaQuery)
> >
> > this happens with the index.html that you distribute.
> > i couldn't see how to fix them. any ideas?
> > -brent
> >
> > On 9/27/05, Paul Spencer <pspencer at dmsolutions.ca> wrote: All,
> >
> > once again, I've made some fairly substantial changes in code
> > structure to accommodate some of the work I am doing. This time, I
> > have changed how objects are declared in the javascript source.
> >
> > Originally, I had code that declared objects and used
> > var kaMap()
> > {
> > this.aFunction = kaMap_aFunction;
> > }
> >
> > function kaMap_aFunction()
> > {
> > }
> >
> > As Ben Nolan pointed out, this pollutes the global namespace in
> > javascript. Rather than take the full plunge into using prototype.js
> > (), I've changed the references to:
> >
> > var kaMap()
> > {
> > }
> >
> > kaMap.prototype.aFunction = function()
> > {
> > }
> >
> > I have also moved some of the core DHTMLapi functions into the kaMap
> > class and modified (hopefully) all of the references to these
> > functions appropriately for the internal use of these files.
> >
> > I've still left DHTMLapi.js in the index.html file as its functions
> > are used there to layout the page (and I don't think it is
> > appropriate to move all that code into kaMap)
> >
> > If anyone has built a kaTool-based tool, you'll need to add a little
> > loop at the bottom of your constructor to copy the prototype
> > properties (and you should probably switch to using prototype at this
> > time). See kaTool and kaQuery for examples.
> >
> > There is no substantial change to functionality, and everything
> > appears to work. However, it should be considered somewhat unstable
> > at this point so test heavily and let me know :)
> >
> > Cheers
> >
> > Paul
> >
> > +-----------------------------------------------------------------+
> > |Paul Spencer pspencer at dmsolutions.ca |
> > +-----------------------------------------------------------------+
> > |Applications & Software Development |
> > |DM Solutions Group Inc http://www.dmsolutions.ca/|
> > +-----------------------------------------------------------------+
> >
> >
> >
> >
> > _______________________________________________
> > ka-Map-users mailing list
> > ka-Map-users at lists.maptools.org
> > http://lists.maptools.org/mailman/listinfo/ka-map-users
> >
> > _______________________________________________
> > ka-Map-users mailing list
> > ka-Map-users at lists.maptools.org
> > http://lists.maptools.org/mailman/listinfo/ka-map-users
> >
>
> +-----------------------------------------------------------------+
> |Paul Spencer pspencer at dmsolutions.ca |
> +-----------------------------------------------------------------+
> |Applications & Software Development |
> |DM Solutions Group Inc http://www.dmsolutions.ca/|
> +-----------------------------------------------------------------+
>
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.maptools.org/pipermail/ka-map-users/attachments/20050927/2a8b4418/attachment.html