Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-6698

sendCommEventAsEmail does not filter CommEventContentAssoc

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 14.12.01, 16.11.01
    • Component/s: party
    • Labels:
      None

      Description

      I only found this whilst testing another piece of code. The emails that were being sent contained attachments that were expired.

      It's unlikely that attachments for new CommunicationEvents will be expired in the time it took to create the records and send the email but it might be worth adding.

      1. OFBIZ-6698.patch
        1.0 kB
        Rohit Koushal

        Activity

        Hide
        toashishvijay Ashish Vijaywargiya added a comment -

        Thanks Gareth for reporting the issue and Thanks Rohit for providing the patch. Changes has been committed in trunk at r1715475 and in R14.12 at r1715476.

        Show
        toashishvijay Ashish Vijaywargiya added a comment - Thanks Gareth for reporting the issue and Thanks Rohit for providing the patch. Changes has been committed in trunk at r1715475 and in R14.12 at r1715476.
        Hide
        rohit.koushal Rohit Koushal added a comment - - edited

        Here is the patch which fix the reported problem on trunk.

        Show
        rohit.koushal Rohit Koushal added a comment - - edited Here is the patch which fix the reported problem on trunk.
        Hide
        gareth.carter Gareth Carter added a comment -

        Would it be worth changing to use EntityQuery rather than getRelated? EntityQuery provides EntityCondition date filter rather than EntityUtil.filterByDate

        Show
        gareth.carter Gareth Carter added a comment - Would it be worth changing to use EntityQuery rather than getRelated? EntityQuery provides EntityCondition date filter rather than EntityUtil.filterByDate

          People

          • Assignee:
            toashishvijay Ashish Vijaywargiya
            Reporter:
            gareth.carter Gareth Carter
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development