MyFaces Core
  1. MyFaces Core
  2. MYFACES-1656

Wrong coercing in #{} for <h:inputText>

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.2.0-SNAPSHOT
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Environment:
      geronimo 2.0 - M6, winxp

      Description

      I tried to run GuessNumber application on the latest SNAPSHOT of 1.2.0
      Looks like is cannot coerce #

      {integer-value}

      for string. A week ago this code worked. Now it prints:
      /greeting.jsp(44,3) '#

      {UserNumberBean.userNumber}' java.lang.IllegalArgumentException: argument type mismatch
      snip from bean:
      public void setUserNumber(Integer user_number) { userNumber = user_number; System.out.println("Set userNumber " + userNumber); }


      public Integer getUserNumber() { System.out.println("get userNumber " + userNumber); return userNumber; }
      snip from jsp:
      <h:inputText id="userNo" value="#{UserNumberBean.userNumber}

      "
      validator="#

      {UserNumberBean.validate}

      "/>
      you may download war from GERONIMO-3167
      see also screen shot

      1. Bad coercing.jpg
        68 kB
        Alexander Zynevich

        Activity

        Hide
        Bruno Aranda added a comment -

        I have modified the test-webapp of myfaces-1.2 in order to reproduce this issue. When executing the app with the jetty maven plugin it works ok. If I deploy it in geronimo (+tomcat) I can reproduce the issue. Any ideas?

        Show
        Bruno Aranda added a comment - I have modified the test-webapp of myfaces-1.2 in order to reproduce this issue. When executing the app with the jetty maven plugin it works ok. If I deploy it in geronimo (+tomcat) I can reproduce the issue. Any ideas?
        Hide
        Mike Kienenberger added a comment -

        Different versions of el-api.jar and el-ri.jar?

        I'd think Geronimo would come with it prebundled.

        No idea what Jetty would use.

        Show
        Mike Kienenberger added a comment - Different versions of el-api.jar and el-ri.jar? I'd think Geronimo would come with it prebundled. No idea what Jetty would use.
        Hide
        Paul McMahan added a comment -

        From a visual inspection of jetty it looks like it uses the jsp-2.1 RI from glassfish, which includes the el api. Geronimo uses tomcat's el api, see:
        http://svn.apache.org/repos/asf/tomcat/tc6.0.x/tags/TOMCAT_6_0_13/java/javax/el/

        (Actually geronimo uses a copy of tomcat's el api, but will soon switch back over to tomcat's original version now that they are published to maven central repo)

        If we can narrow this problem down to a bug in EL and provide a patch to tomcat then geronimo will be able to pick up that change.

        Show
        Paul McMahan added a comment - From a visual inspection of jetty it looks like it uses the jsp-2.1 RI from glassfish, which includes the el api. Geronimo uses tomcat's el api, see: http://svn.apache.org/repos/asf/tomcat/tc6.0.x/tags/TOMCAT_6_0_13/java/javax/el/ (Actually geronimo uses a copy of tomcat's el api, but will soon switch back over to tomcat's original version now that they are published to maven central repo) If we can narrow this problem down to a bug in EL and provide a patch to tomcat then geronimo will be able to pick up that change.
        Hide
        Bruno Aranda added a comment -

        Definitely it seems to be a bug in tomcat's el api, as the test-webapp runs well also in glassfish

        Show
        Bruno Aranda added a comment - Definitely it seems to be a bug in tomcat's el api, as the test-webapp runs well also in glassfish

          People

          • Assignee:
            Unassigned
            Reporter:
            Alexander Zynevich
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development