Tapestry 5
  1. Tapestry 5
  2. TAP5-714

Incorrect encoding of single quotes for Ajax requests

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.1.0.5
    • Fix Version/s: 5.2.0, 5.1.0.6, 5.1.0.7
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      I've encountered what I think is a bug in the rendering of an ajax response.
      Basically, we're returning a block that contains a textfield component. The
      value of that textfield component is something with a single quote in it,
      for example "o'reilly". The response markup for that textfield is:

      <input value='o'reilly' id='addressLine1-1215dd60f22'
      name='addressLine1' type='text'><\/input>

      which renders as "o'reilly" in Firefox, but in IE the value renders as
      "o'reilly". According to
      http://cssvault.com/blog/2007/10/17/internet-explorer-apos-feature/ , it
      should be rendering out ' instead.

      I've looked into where the ' is coming from and its in the
      AbstractMarkupModel class, in the encodeQuoted() method (around line 132).

      1. TAP-714.patch
        1 kB
        Ciaran Wood
      2. TAP5-714-5.1.txt
        1 kB
        Ulrich Stärk

        Activity

        Ciaran Wood created issue -
        Ciaran Wood made changes -
        Field Original Value New Value
        Attachment TAP-714.patch [ 12409149 ]
        Thiago H. de Paula Figueiredo made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Thiago H. de Paula Figueiredo [ thiagohp ]
        Fix Version/s 5.2.0.0 [ 12314122 ]
        Resolution Fixed [ 1 ]
        Ulrich Stärk made changes -
        Attachment TAP5-714-5.1.txt [ 12425470 ]
        Igor Drobiazko made changes -
        Fix Version/s 5.1.0.6 [ 12314399 ]
        Igor Drobiazko made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Igor Drobiazko made changes -
        Fix Version/s 5.1.0.7 [ 12314494 ]

          People

          • Assignee:
            Thiago H. de Paula Figueiredo
            Reporter:
            Ciaran Wood
          • Votes:
            4 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development