Tapestry
  1. Tapestry
  2. TAPESTRY-1219

The order of loading bodyscripts and initializationscripts is non deterministic

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Later
    • Affects Version/s: 4.1.1
    • Fix Version/s: 4.1.6
    • Component/s: XHR/dhtml/Ajax
    • Labels:
      None
    • Environment:
      FF 2.0

      Description

      A comment in core.js reads 'load body scripts before initialization'...
      Due to the way loadScriptContent has to be implemented (to circumvent IE issues) this doesn't always hold true.

      So, a call to taperstry.widget.synchronizeWidgetState (in an initialization script) can sometimes get executed BEFORE a
      dojo.require('tapestry.widget.Widget') (typically in body scripts) resulting in
      'taperstry.widget.synchronizeWidgetState not a function' javascript errors.

      Perhaps somehow block until all body scripts get processed?

        Activity

        Hide
        Jesse Kuhnert added a comment -

        Dunno ....Do you have an example of where this has happened? Ie is it worth the effort?

        Show
        Jesse Kuhnert added a comment - Dunno ....Do you have an example of where this has happened? Ie is it worth the effort?
        Hide
        Andreas Andreou added a comment -

        i think i got around this by adding those dojo.requires directly to the page,
        even when the component that needs them isn't rendering - of course it's a hack.

        So, due to the hack, this has become a 2nd priority for now, but i'll soon create a test case for this
        in order to investigate more.

        Show
        Andreas Andreou added a comment - i think i got around this by adding those dojo.requires directly to the page, even when the component that needs them isn't rendering - of course it's a hack. So, due to the hack, this has become a 2nd priority for now, but i'll soon create a test case for this in order to investigate more.
        Hide
        Jesse Kuhnert added a comment -

        Ok..Changing status and owner for now then.

        Show
        Jesse Kuhnert added a comment - Ok..Changing status and owner for now then.
        Hide
        Andreas Andreou added a comment -

        We will reopen if this indeed affects someone

        Show
        Andreas Andreou added a comment - We will reopen if this indeed affects someone

          People

          • Assignee:
            Andreas Andreou
            Reporter:
            Andreas Andreou
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development