Details

    • Type: Bug Bug
    • Status: Open Open
    • Priority: Critical Critical
    • Resolution: Unresolved
    • Affects Version/s: 1.1
    • Fix Version/s: None
    • Component/s: Paging/Sorting
    • Labels:
      None

      Activity

      Hide
      Rob MacGrogan added a comment -
      Please fix this. I see this behavior in the 1.1.1 release from August of 2007. My sortName attribute is ignored. The generated sort link shows something like http://mypage?sort=page=2.
      Show
      Rob MacGrogan added a comment - Please fix this. I see this behavior in the 1.1.1 release from August of 2007. My sortName attribute is ignored. The generated sort link shows something like http://mypage?sort=page=2 .
      Hide
      Rob MacGrogan added a comment -
      I found a workaround to this issue that works OK for me. Basically, what you have to do is set the property attribute on the column. I was not using the property attribute because I was using EL to display all my values, which was working fine for me. Anyway, if you set the property attribute, then this attribute will be used for the sort. The sortName is completely ignored either way, so this is still a bug.
      Show
      Rob MacGrogan added a comment - I found a workaround to this issue that works OK for me. Basically, what you have to do is set the property attribute on the column. I was not using the property attribute because I was using EL to display all my values, which was working fine for me. Anyway, if you set the property attribute, then this attribute will be used for the sort. The sortName is completely ignored either way, so this is still a bug.
      Hide
      Jamie Taylor added a comment -
      The attached patch file was generated by svn diff from the displaytag/src directory of code I checked out from the repository earlier today. It modifies the parts of HtmlTableWriter, TableModel, and TableTag that look at the sortProperty or sortBeanPropertyName properties of HeaderCell to first check the sortName property.

      It also fixes the issue described in DISPL-327, since that got in the way when I was clicking around testing my changes.

      I hereby release the modifications contained in this patch file under whatever license is necessary for them to be included in displaytag as it is currently licensed.
      Show
      Jamie Taylor added a comment - The attached patch file was generated by svn diff from the displaytag/src directory of code I checked out from the repository earlier today. It modifies the parts of HtmlTableWriter, TableModel, and TableTag that look at the sortProperty or sortBeanPropertyName properties of HeaderCell to first check the sortName property. It also fixes the issue described in DISPL-327 , since that got in the way when I was clicking around testing my changes. I hereby release the modifications contained in this patch file under whatever license is necessary for them to be included in displaytag as it is currently licensed.

        People

        • Reporter:
          Tony Johnson
        • Votes:
          6 Vote for this issue
          Watchers:
          3 Start watching this issue

          Dates

          • Created:
            Updated: