Changes between Initial Version and Version 1 of Ticket #11262, comment 10


Ignore:
Timestamp:
Dec 7, 2012, 11:15:19 AM (8 years ago)
Author:
bill
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #11262, comment 10

    initial v1  
    77(2) could be "fixed" by changing _WidgetBase.js (nee _Widget.js) to just use a {{{for(var p in params)}}} loop rather than dojo.mixin().     The only fallout would be if someone specified a parameter with a funny name, like toString(), which is possible but unlikely.
    88
     9However, you might soon hit a problem somewhere else due to dojo.mixin() not handling targets from other frames.
     10
    911Also, I wouldn't say that this ticket is the same as #9409, but rather that it's fallout from #9409, which fixed one problem but caused this other one.
    1012