Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: SVN trunk
    • Fix Version/s: SVN trunk
    • Component/s: framework
    • Labels:
      None
    • Environment:

      WinXP

      Description

      This issue is part of Key-Work Ofbiz Contribution Proposal.
      http://docs.ofbiz.org/display/OFBIZ/Key-Work+Ofbiz+Contribution+Proposal

      Based on Release: 540035
      Includes: -
      Required: -
      Recommended: -
      Classes: WrappingException.java, WrappingRuntimeException,
      WrappingGenericEntityException
      Description: These classes were introduced to have a standard way of
      exception tunneling.

      1. patch9648.patch
        7 kB
        Karl Eilebrecht

        Activity

        Hide
        David E. Jones added a comment -

        Please review this carefully before applying. OFBiz already has generic "tunneling" exception classes, namely the GeneralException plus one in more of the components, like the GenericEntityException and the GenericServiceException.

        Whatever is done should be based on and extend that pattern, rather than having two sets of classes that fill this need.

        Show
        David E. Jones added a comment - Please review this carefully before applying. OFBiz already has generic "tunneling" exception classes, namely the GeneralException plus one in more of the components, like the GenericEntityException and the GenericServiceException. Whatever is done should be based on and extend that pattern, rather than having two sets of classes that fill this need.
        Hide
        Si Chen added a comment -

        I was interested in OFBIZ-1029 which depends on this. This seems just to be a few new classes.

        Show
        Si Chen added a comment - I was interested in OFBIZ-1029 which depends on this. This seems just to be a few new classes.

          People

          • Assignee:
            Si Chen
            Reporter:
            Karl Eilebrecht
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development