DisplayTag
  1. DisplayTag
  2. DISPL-73

Unicode export with ResponseOverrideFilter fails because of improper content length setting

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0 RC1
    • Fix Version/s: 1.0 RC2
    • Component/s: Export
    • Labels:
      None
    • Application server:
      Resin 2.1.12

      Description

      org.displaytag.filter.ResponseOverride uses pageContent.length() to set content length of the response. If the content has characters that use more than one byte, this length is set to a too small value and the results get cut.

      I replaced the pageContent.length() with the actual byte count, calculated with the proper character encoding. CVS diff is inclued as an attachment.

      We encountered this bug while trying to csv export tables containing Japanese characters.

        Activity

        Hide
        Mikko Wilkman added a comment -
        cvs diff of the bugfix
        Show
        Mikko Wilkman added a comment - cvs diff of the bugfix
        Hide
        fabrizio giustina added a comment -
        Thanks, I will apply this patch before the rc2 release (as soon as other changes to the filter will be stable)
        Show
        fabrizio giustina added a comment - Thanks, I will apply this patch before the rc2 release (as soon as other changes to the filter will be stable)
        Hide
        fabrizio giustina added a comment -
        fix committed to cvs.
        Show
        fabrizio giustina added a comment - fix committed to cvs.
        Hide
        added a comment -
        http://www.jsvelho.org <a href="http://www.jsvelho.org">payday loan company</a> thanks
        Show
        added a comment - http://www.jsvelho.org <a href=" http://www.jsvelho.org ">payday loan company</a> thanks
        Hide
        added a comment -
        http://www.mylosoft.com/ <a href="http://www.mylosoft.com/">world series of poker</a> thanks for your site
        Show
        added a comment - http://www.mylosoft.com/ <a href=" http://www.mylosoft.com/ ">world series of poker</a> thanks for your site

          People

          • Reporter:
            Mikko Wilkman
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: