Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.1
    • Component/s: Uncategorized
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      323
    • Status Whiteboard:
      Hide

      EGTop5 cat2

      Show
      EGTop5 cat2

      Description

      in com.sun.el.lang.ELSupport.coerceToString()
      I noticed that a in case of a null String, the method returns "";

      The way down.
      In the converter:
      if (patternExpression != null)

      { pattern = (String) ELUtils.evaluateValueExpression(patternExpression, elContext); }

      in

      com.sun.faces.el.ELUtils.evaluateValueExpression()
      (expectedType == String... for the pattern)

      if (this.expectedType != null)

      { return ELSupport.coerceToType(value, this.expectedType); }

      coerceToType CALLs the coerceToString()

      public final static String coerceToString(final Object obj) {
      if (obj == null) {
      return "";
      ...
      }

      That is because:
      Unified EL Spec says:

      If A is String: return A
      Otherwise, if A is null: return ""

      Isn't that crazy ?

      1. changebundle.txt
        22 kB
        Ryan Lubke
      2. UIInput.html
        113 kB
        Ed Burns

        Activity

        Hide
        Ed Burns added a comment -

        Move these to unscheduled because we need to target them correctly. 2.next isn't
        specific enough.

        Show
        Ed Burns added a comment - Move these to unscheduled because we need to target them correctly. 2.next isn't specific enough.
        Hide
        rogerk added a comment -

        cat2

        Show
        rogerk added a comment - cat2
        Hide
        Ed Burns added a comment -
        Show
        Ed Burns added a comment - Closed, moved to https://uel.dev.java.net/issues/show_bug.cgi?id=13
        Hide
        Ed Burns added a comment -

        These are targeted at 2.1.

        Show
        Ed Burns added a comment - These are targeted at 2.1.
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

          • Assignee:
            Ed Burns
            Reporter:
            mwessendorf
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: