Uploaded image for project: 'Sling'
  1. Sling
  2. SLING-5812

Add option to include attributes in request scope for Sightly data-sly-resource and data-sly-include

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • Scripting Sightly Engine 1.0.18
    • Scripting HTL Engine 1.0.20
    • Scripting
    • None

    Description

      A common pattern for sending information between scripts/components is setting specific attributes in request scope before including another resource or script. At the moment this cannot be done nicely in Sightly.

      It would be very helpful to set request attributes as in following examples:

      <sly data-sly-include="${'script.html' @ requestAttributes=helper.attributesMap}"/>

      or:

      <sly data-sly-resource="${'some/path' @ resourceType='some/type', requestAttributes=helper.attributesMap}"/>

      where attributesMap is a Map<String, Object>

      The attributes would be set before the actual script/resource inclusion and reset/unset back afterwards.

      Attachments

        Issue Links

          breaks

          Bug - A problem which impairs or prevents the functions of the product. SLING-7538 Request attributes not correctly reset after using data-sly-resource or data-sly-include with requestAttributes

          • Major - Major loss of function.
          • Resolved
          is related to

          Improvement - An improvement or enhancement to an existing feature or task. SLING-7515 ResourceRuntimeExtension: Allow to pass arbitrary options a request attributes via data-sly-resource

          • Major - Major loss of function.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. SLING-6636 Add request-attributes to SlingHttpServletRequest

          • Major - Major loss of function.
          • Resolved
          links to

          Web Link GitHub Pull Request #149

          Activity

            People

              radu Radu Cotescu
              vladb Vlad Bailescu
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: