Geronimo
  1. Geronimo
  2. GERONIMO-3880

PersistenceUnitInfo.getJarFileUrls() can return null

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.2, 2.1
    • Fix Version/s: 2.0.3, 2.1.1, 2.2
    • Component/s: persistence
    • Security Level: public (Regular issues)
    • Labels:
      None

      Description

      Hibernate doesn't like null from PersistenceUnitInfo.getJarFileUrls(), see patch proposed in http://cwiki.apache.org/GMOxSAMPLES/running-jboss-seam-200ga-on-geronimo-21.html
      While we can argue hibernate should check for null, we shouldn't be supplying null.

        Activity

        Hide
        David Jencks added a comment -

        Fixed in trunk rev 631810, 2.1 branch 631811 and 2.0 branch rev 631816

        Show
        David Jencks added a comment - Fixed in trunk rev 631810, 2.1 branch 631811 and 2.0 branch rev 631816
        Hide
        David Jencks added a comment -

        I added a dumb simple test in trunk rev 631819.

        Show
        David Jencks added a comment - I added a dumb simple test in trunk rev 631819.
        Hide
        David Jencks added a comment -

        Due to the same original refactoring we are also often supplying a null persistenceUnitRoot url.

        Show
        David Jencks added a comment - Due to the same original refactoring we are also often supplying a null persistenceUnitRoot url.
        Hide
        David Jencks added a comment -

        trunk 632359
        2.1 632360
        2.0 632361

        This also added the test to 2.1 and 2.0.

        Show
        David Jencks added a comment - trunk 632359 2.1 632360 2.0 632361 This also added the test to 2.1 and 2.0.
        Hide
        David Jencks added a comment -

        similar null managedClassNames that toplink essentials complains about.

        Show
        David Jencks added a comment - similar null managedClassNames that toplink essentials complains about.
        Hide
        David Jencks added a comment -

        trunk rev 633225
        2.1 rev 633227
        2.0 rev 633229

        Show
        David Jencks added a comment - trunk rev 633225 2.1 rev 633227 2.0 rev 633229

          People

          • Assignee:
            David Jencks
            Reporter:
            David Jencks
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development