Changes between Version 2 and Version 3 of Ticket #13769, comment 27


Ignore:
Timestamp:
Apr 6, 2012, 12:36:30 AM (7 years ago)
Author:
bill
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #13769, comment 27

    v2 v3  
    11Adam, can you provide a test case (less than 750K)?    As I said above, circular dependency is not a bug, so if something isn't working then that indicates a bug outside the DOM code.   Perhaps in the sync loader.   @rbackhouse's suggestion should not be necessary, as he himself said above in comment {{{#18}}}.
     2
     3I'm not necessarily opposed to removing the circular dependency, but it would be good to solve the actual bug too, rather than sweeping it under the rug.
    24
    35Note: After zynevich's patch, the dependencies between the dom* modules are:
     
    1517}}}
    1618
    17 So it does resolve the circular dependency.
     19So it does resolve the circular dependency.   The patch looks OK to me, although I had some trouble applying it so I'll upload an updated version.