Torque
  1. Torque
  2. TORQUE-31

complete generic type compatible API

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: 3.3
    • Component/s: Generator
    • Labels:
      None

      Description

      If the generator creates Java5 compatible methods then there are some methods from BasePeer that are not wrapped. This includes for example the execute*() methods. If we wrap them in the generated classes the API looks a little bit better.

      Currently there is just one easy solution in my mind - wrap this methods in the generated Base classes. For example:

      public class BaseTablePeer extends BasePeer {
      ....
      public static List<Record> executeQuery(String queryString) throws TorqueException

      { return (List<Record>)super.executeQuery(queryString); }

      ....
      }

      This means, that every public method in BasePeer should be wrapped in the generated class. I know, this is a lot and results in bigger classes and some synchronization problems, if the API of BasePeer is changed. But I think, it's easy to create.

      I'll create a patch for that next week.

      1. TORQUE-31.patch
        10 kB
        Thoralf Rickert

        Activity

        Thoralf Rickert created issue -
        Thoralf Rickert made changes -
        Field Original Value New Value
        Attachment TORQUE-31.patch [ 12335759 ]
        Thomas Fox made changes -
        Assignee Thomas Fischer [ tfischer ]
        Fix Version/s 3.2.1 [ 12310991 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Thomas Vandahl made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Thomas Fox
            Reporter:
            Thoralf Rickert
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development