glassfish
  1. glassfish
  2. GLASSFISH-298

Verifier is not scanning JSF managed beans

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 9.0pe
    • Fix Version/s: 9.0pe
    • Component/s: verifier
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      298

      Description

      While discussing an issue in GlassFish forum
      http://forums.java.net/jive/post!reply.jspa?messageID=85667
      I came across this bug.
      WebArchiveClassesLoadable test is not scanning JSF managed beans. See the
      attached ear tpcc-ea.ear looks like this:

      lib/tpcc-persist.jar
      tpcc-session.jar
      tpcc-web.war

      tpcc-web.war has WEB-INF/faces-config.xml which defines two managed beans:
      tpcc.jsf.model.WarehouseFormBean and tpcc.jsf.model.DistrictFormBean

      If you look at the source code, these beans are clients to the EJBs defined
      inside tpcc-session.jar. The ejb interface classes are not packaged in
      tpcc-web.war, yet, when I run verifier in portability mode (-p option), it does
      not report them as missing classes.

      To build the test case yourself:
      Unzip jsfAndWebArchiveClassesLoadable.zip.
      modify build.properties in each subdirectory to set appropriate glassfish.home.
      Run ant verify.
      This test case is obtained from Dibyendu.

        Activity

        Hide
        Sanjeeb Sahoo added a comment -

        Created an attachment (id=91)
        test case. Contains source code and build.xml

        Show
        Sanjeeb Sahoo added a comment - Created an attachment (id=91) test case. Contains source code and build.xml
        Hide
        Bhavanishankar added a comment -

        Code changes:
        =============

        RCS file:
        /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/web/FacesConfigDescriptor.java,v
        done
        Checking in web/FacesConfigDescriptor.java;
        /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/web/FacesConfigDescriptor.java,v
        <-- FacesConfigDescriptor.java
        initial revision: 1.1
        done
        Checking in web/WebCheckMgrImpl.java;
        /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/web/WebCheckMgrImpl.java,v
        <-- WebCheckMgrImpl.java
        new revision: 1.11; previous revision: 1.10
        done
        Checking in Context.java;
        /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/Context.java,v
        <-- Context.java
        new revision: 1.10; previous revision: 1.9
        done
        Checking in tests/web/WebArchiveClassesLoadable.java;
        /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/tests/web/WebArchiveClassesLoadable.java,v
        <-- WebArchiveClassesLoadable.java
        new revision: 1.12; previous revision: 1.11
        done
        Checking in LocalStrings.properties;
        /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/LocalStrings.properties,v
        <-- LocalStrings.properties
        new revision: 1.68; previous revision: 1.67
        done

        Show
        Bhavanishankar added a comment - Code changes: ============= RCS file: /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/web/FacesConfigDescriptor.java,v done Checking in web/FacesConfigDescriptor.java; /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/web/FacesConfigDescriptor.java,v <-- FacesConfigDescriptor.java initial revision: 1.1 done Checking in web/WebCheckMgrImpl.java; /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/web/WebCheckMgrImpl.java,v <-- WebCheckMgrImpl.java new revision: 1.11; previous revision: 1.10 done Checking in Context.java; /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/Context.java,v <-- Context.java new revision: 1.10; previous revision: 1.9 done Checking in tests/web/WebArchiveClassesLoadable.java; /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/tests/web/WebArchiveClassesLoadable.java,v <-- WebArchiveClassesLoadable.java new revision: 1.12; previous revision: 1.11 done Checking in LocalStrings.properties; /cvs/glassfish/avk/src/tools/com/sun/enterprise/tools/verifier/LocalStrings.properties,v <-- LocalStrings.properties new revision: 1.68; previous revision: 1.67 done

          People

          • Assignee:
            Bhavanishankar
            Reporter:
            Sanjeeb Sahoo
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: