Tiles
  1. Tiles
  2. TILES-65

Split <tiles:insert> into specialized tags

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.0
    • Component/s: tiles-core
    • Labels:
      None
    • Environment:

      N/A (design issue)

      Description

      The <tiles:insert> tag can lead to confusion because it can insert different things: pages, strings, definitions and attribute.
      There is a particular confusion for the attribute, since some people could place it as they place definitions and pages, not remembering that attributes need to be filled.

      <tiles:insert> could be split into:

      <tiles:insertDefinition>
      <tiles:insertTemplate>
      <tiles:insertString> (not sure about it).
      <tiles:attribute>: it has a different name pattern because attributes need to be "defined" and not "inserted".

      Relevant thread:
      http://www.mail-archive.com/dev%40struts.apache.org/msg24891.html

      And the message:
      http://www.mail-archive.com/user%40shale.apache.org/msg00663.html

        Issue Links

          Activity

          Hide
          Antonio Petrelli added a comment -

          InsertTag has been splitted into three tags:
          InsertDefinitionTag, to insert definitions;
          InsertTemplateTag, to insert templates;
          AttributeTag, to insert (I should say "define") attributes.
          There was no need to create a <tiles:insertString> tag, since there are better tag libraries to put strings.
          Modified tiles-core.tld accordingly and added a FAQ entry for the change.

          Show
          Antonio Petrelli added a comment - InsertTag has been splitted into three tags: InsertDefinitionTag, to insert definitions; InsertTemplateTag, to insert templates; AttributeTag, to insert (I should say "define") attributes. There was no need to create a <tiles:insertString> tag, since there are better tag libraries to put strings. Modified tiles-core.tld accordingly and added a FAQ entry for the change.

            People

            • Assignee:
              Antonio Petrelli
              Reporter:
              Antonio Petrelli
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development