Uploaded image for project: 'Fluid Infusion'
  1. Fluid Infusion
  2. FLUID-4537

Failure to expand contextual EL references within prototrees that are not UIBound elements

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: 1.5
    • Component/s: Renderer
    • Labels:
      None

      Description

      Problem reported by the Jen Bourey: prototree material like the following:

      protoTree: {
      expander: {
      type: "fluid.renderer.repeat",
      repeatID: "item",
      controlledBy: "feeds",
      pathAs: "item",
      tree: {
      title: "${{item}.title}",
      link: { target: "${{item}.link}", decorators: { attrs: { title: "${{item}.description}" } } },
      description: "${{item}.description}"
      }
      }
      }

      has UIBound members like title, link, description expanded properly, whereas material sent to the attribute decorator via the "expandLight" pathway ends up destroyed.

      This is as a result of a fault in the prototree expander - the custom "EL concatentation pathway" operated by code handing via "fetchEL" within makeProtoExpander is only operated from "expandBound". Any other material is handed to "expandLight" which just uses the default environmental fetcher (this was actually code hacked out of the IoC system as a kind of "generalisation" of the common requirements of the protoTree expander and the old historical "half-IoC" system in fluid Engage - but is not in fact used directly by any part of Fluid IoC itself despite being defined in that file).

      The "default environmental fetcher" needs to be modified to allow the special strategy used in protoTree expansion to transform "parsed" context objects, consistently with the "fetchEL" strategy.

      In the long term this pathway needs to be moved out of FluidIoC entirely and further rationalisation provided with context parsing expression.s

      In the even longer term the protoTree expansion pathway needs to be "burned to the ground" entirely and replaced with pure IoC definitions.

        Attachments

          Activity

            People

            Assignee:
            antranig Antranig Basman
            Reporter:
            antranig Antranig Basman
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: