As quoted from "Steven M. Ottens" <[EMAIL PROTECTED]>:
> Bug   MAP-175         WidgetBase.js incorrectly mentions stylesheet
> This bug is still valid, GertJan mentions that JSDoc and config.xsd  
> should be updated

I'm picking this one up.  There's many a file that incorrect or
imcomplete JSDoc w.r.t. inheritance/extensions (i.e. missing or wrong
@base tag) and sometimes even does a loadScript() of the wrong file.
I'll fix all these as well.

BTW, does any of the old-timers know why the coding pattern for loading
the JS file dependencies that is followed in lib/tool/ and lib/widget/
like so:

  // Ensure this object's dependancies are loaded.
  mapbuilder.loadScript(baseDir+"/widget/WidgetBaseXSL.js");

is not at all followed in lib/model/?  None of the files (well, except
one at the moment, Transaction.js) there call:

  // Ensure this object's dependancies are loaded.
  mapbuilder.loadScript(baseDir+"/widget/ModelBase.js");

Now I know that ModelBase.js is loaded by Mapbuilder.js, but for
orthogonality's sake, shouldn't we do it in lib/model/ as well?

-- 
-- Gertjan van Oosten, [EMAIL PROTECTED], West Consulting B.V., +31 15 2191 600

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
mapbuilder-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mapbuilder-devel

Reply via email to