Uploaded image for project: 'Torque'
  1. Torque
  2. TORQUE-31

complete generic type compatible API

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Fixed
    • 3.3
    • 3.3
    • Generator
    • 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.

      Attachments

        1. TORQUE-31.patch
          10 kB
          Thoralf Rickert

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            tfischer Thomas Fox
            trickert Thoralf Rickert
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment