Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0056395
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[POS2] POSmajoralways2024-09-06 08:482024-09-10 07:53
ReporterjorgewederagoView Statuspublic 
Assigned Tosreehari 
PriorityhighResolutionfixedFixed in Version24Q3.1
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version24Q3.1
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commithttps://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/e90bcd8437481fb6f341e920574df7f56364e741 [^]
Triggers an Emergency PackNo
Summary

0056395: It is not possible to send OTF coupon by email

DescriptionSince the fix related to https://issues.openbravo.com/view.php?id=55626 [^] was merged, anytime a Coupon is going to be sent an error is generated as import entry which ends in the Coupon not being sent
Steps To Reproduce1.- Have a configuration for send email
2.- Create a Coupon OTF Rule
3.- In POS do a transaction that will trigger the OTF Rule
4.- In the ticket delivery choose send by email
5.- See that the email is not received and there is an Import Entry related to the email in error
Proposed SolutionDuring the fix for that issue now is mandatory to have the "ticket" object, however this is not true when other things that are not ticket are being sent by email.

It is needed to add code to avoid this kind of scenarios

Take into account that the fix has been backported to other versions so it is needed to do the same
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks defect 0056393 closedsreehari It is not possible to send OTF coupon by email 

-  Notes
(0168906)
hgbot (developer)
2024-09-09 07:52

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/2910 [^]
(0169000)
hgbot (developer)
2024-09-10 07:53

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/2910 [^]
(0169001)
hgbot (developer)
2024-09-10 07:53

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 269a22568c534046bde096ca302ed43e27f32698
Author: Sreehari Venkataraman <s.venkataraman@external.orisha.com>
Date: 10-09-2024 05:53:23
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/269a22568c534046bde096ca302ed43e27f32698 [^]

Fixes ISSUE-56395: Error in Send email in case other files than tickets were sent
Added case to check ticket available in param, now when param doesn't have ticket file is directly added

---
M src/org/openbravo/pos2/mailing/SendEmailLoader.java
---

- Issue History
Date Modified Username Field Change
2024-09-06 10:40 sreehari Type defect => backport
2024-09-06 10:40 sreehari Target Version => 24Q3.1
2024-09-09 07:52 hgbot Note Added: 0168906
2024-09-10 07:53 hgbot Resolution open => fixed
2024-09-10 07:53 hgbot Status scheduled => closed
2024-09-10 07:53 hgbot Note Added: 0169000
2024-09-10 07:53 hgbot Fixed in Version => 24Q3.1
2024-09-10 07:53 hgbot Note Added: 0169001


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker