Tapestry
  1. Tapestry
  2. TAPESTRY-2237

Grid should make its "currentPage" value available to containing components

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.11
    • Fix Version/s: 5.0.11
    • Component/s: Core Components
    • Labels:
      None
    • Environment:
      any

      Description

      With the introduction of the @GenerateAccessor annotation, tapestry's core components were stripped of their "getters" and "setters". But Grid's currentPage property should be accessible via java code, and not 100% internal to the component. There are many cases where a container may want or need to change the current page, outside of the standard "pager" links. Without the accessors, this is non-trivial (ie requires reflection).

        Activity

        Robert Zeigler created issue -
        Hide
        Robert Zeigler added a comment -

        patch adds accessors for currentPage property, and removes the @GenerateAccessors annotation from the same.

        Show
        Robert Zeigler added a comment - patch adds accessors for currentPage property, and removes the @GenerateAccessors annotation from the same.
        Robert Zeigler made changes -
        Field Original Value New Value
        Attachment grid_currentpage_accessors.patch [ 12377215 ]
        Howard M. Lewis Ship made changes -
        Resolution Fixed [ 1 ]
        Fix Version/s unspecified [ 10790 ]
        Fix Version/s 5.0.11 [ 12312968 ]
        Assignee Howard M. Lewis Ship [ hlship ]
        Status Open [ 1 ] Closed [ 6 ]
        Mark Thomas made changes -
        Workflow jira [ 12425182 ] Default workflow, editable Closed status [ 12568627 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12568627 ] jira [ 12591661 ]

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Robert Zeigler
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development