Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.1.3
    • Fix Version/s: 4.1.5
    • Component/s: Contrib
    • Labels:
      None
    • Environment:
      Websphere App Server 5.1

      Description

      One example that cause bugs on using String.replace is located on:

      org.apache.tapestry.contrib.table.model.simple.SimpleTableColumn.setDisplayName(String displayName) line 161:
      displayName = displayName.replace("_", ".");

      will cause this exception:
      java.lang.String: method replace(Ljava/lang/CharSequence;Ljava/lang/CharSequence;)Ljava/lang/String; not found
      because it uses double quotation so it is assumed as String instead of char.

      It should be displayName.replace('_', '.');
      with single quotation

        Activity

        Hide
        Joshua Partogi added a comment -

        Brent: On Websphere JDK 1.4 this doesn't work.

        Andreas: Thanks for fixing I already fetch the latest from SVN.

        Cheers,

        Show
        Joshua Partogi added a comment - Brent: On Websphere JDK 1.4 this doesn't work. Andreas: Thanks for fixing I already fetch the latest from SVN. Cheers,
        Hide
        Andreas Andreou added a comment -

        Both contrib and framework jars should be 1.4 compatible
        + I believe that's also what their problem was (having to use 1.4)

        Show
        Andreas Andreou added a comment - Both contrib and framework jars should be 1.4 compatible + I believe that's also what their problem was (having to use 1.4)
        Hide
        Brent Thomas added a comment -

        Can someone elaborate on the fix because I don't understand what was wrong here. A String is CharSequence in Java 1.5, so why would the original form not work?

        Show
        Brent Thomas added a comment - Can someone elaborate on the fix because I don't understand what was wrong here. A String is CharSequence in Java 1.5, so why would the original form not work?
        Hide
        Andreas Andreou added a comment -

        Thx for noticing, fixed

        Show
        Andreas Andreou added a comment - Thx for noticing, fixed

          People

          • Assignee:
            Andreas Andreou
            Reporter:
            Joshua Partogi
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development