Torque
  1. Torque
  2. TORQUE-107

calling getRelatedObjects(Criteria) can result in NullPointerException if the collection has not be initialized (with a Criteria)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3-RC1
    • Fix Version/s: 4.0-beta1
    • Component/s: Generator
    • Labels:
      None
    • Environment:
      RHEL 5, Java 6, Oracle 10g

      Description

      The Object.vm template and generator produce object classes with

      public List<SomeClass> getSomeClasss(Criteria criteria) throws TorqueException

      If torque.objectIsCaching is set, this method uses a "last used Criteria" comparison to determine whether or not to re-fetch the Collection of <SomeClass>. The way the template (Object.vm) is written, we wind up with

      if (!lastSomeClassCriteria.equals(criteria))
      ...

      which can throw NullPointerException if this is the first time this method is called (lastSomeClassCriteria will be null).

      This could be fixed by changing the comparison to

      if (!criteria.equals(last$

      {relCol}Criteria))

      or

      if (last${relCol}

      Criteria == null || !last$

      {relCol}

      Criteria.equals(criteria))

      Granted, there are other ways to achieve these results (create the appropriate Criteria and call SomeClassPeer.doSelect(Criteria)), but the "getRelatedObjects(Criteria)" method is a handy abstraction that is already generated for each class.

        Activity

        Brendan Miller created issue -
        Thomas Fox made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Thomas Fox [ tfischer ]
        Fix Version/s 4.0 [ 12312102 ]
        Resolution Fixed [ 1 ]
        Thomas Fox made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Thomas Fox
            Reporter:
            Brendan Miller
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development