Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0023479Openbravo ERPA. Platformpublic2013-04-03 15:462013-04-07 01:03
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PriorityimmediateSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0MP22Fixed in Version3.0MP22 
Merge Request Status
Review Assigned Toshankarb
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelPre packaging ( pi )
Regression date2013-03-17
Regression introduced in release
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/5def5186a07f8394ad89bc4aee171a8de3fe186d [^]
Triggers an Emergency PackNo
Summary0023479: Retrieval of EmailServerConfiguration does not work if several clients have configured their email
DescriptionDue to a missing restriction in the OBCriteria used to retrieve the EmailServerConfiguration, the retrieval does not work if several clients have configured their email.
Steps To ReproduceConfigure the email in one client. (see http://wiki.openbravo.com/wiki/Client#Email_Configuration [^]).
Configure the email in another client.
Try to send an email:
- Open Sales Invoice
- Select an invoice
- Push the email button in the toolbar. The email will not be sent
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to feature request 00231983.0MP22 closed shankarb Centralize and improve the retrieval of EmailServerConfiguration 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-04-03 15:46AugustoMauchNew Issue
2013-04-03 15:46AugustoMauchAssigned To => AugustoMauch
2013-04-03 15:46AugustoMauchModules => Core
2013-04-03 15:46AugustoMauchTriggers an Emergency Pack => No
2013-04-03 15:48AugustoMauchRegression level => Pre packaging ( pi )
2013-04-03 15:48AugustoMauchRegression date => 2013-03-17
2013-04-03 15:48AugustoMauchRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/5def5186a07f8394ad89bc4aee171a8de3fe186d [^]
2013-04-03 15:48AugustoMauchPrioritynormal => immediate
2013-04-03 16:31AugustoMauchRelationship addedrelated to 0023198
2013-04-03 16:31AugustoMauchIssue Monitored: shankarb
2013-04-03 16:32AugustoMauchReview Assigned To => shankarb
2013-04-03 16:34hgbotCheckin
2013-04-03 16:34hgbotNote Added: 0057709
2013-04-03 16:34hgbotStatusnew => resolved
2013-04-03 16:34hgbotResolutionopen => fixed
2013-04-03 16:34hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3ba5d370c1b96fe06fd1ebf0a2e586c3335cb28f [^]
2013-04-04 05:11shankarbNote Added: 0057720
2013-04-04 05:11shankarbStatusresolved => closed
2013-04-04 05:11shankarbFixed in Version => 3.0MP22
2013-04-07 01:03hudsonbotCheckin
2013-04-07 01:03hudsonbotNote Added: 0057761

Notes
(0057709)
hgbot   
2013-04-03 16:34   
Repository: erp/devel/pi
Changeset: 3ba5d370c1b96fe06fd1ebf0a2e586c3335cb28f
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Wed Apr 03 16:21:37 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/3ba5d370c1b96fe06fd1ebf0a2e586c3335cb28f [^]

Fixes issue 23479: Sending email works having several email configurations

It was not possible to send an email having defined several email configurations. It did not work if a client had two configurations defined, or even it two different clients had one configuration defined.

To fix this a client filter has been set (so the email configuration of another client is not fetched), and the results are not fetched using uniqueResult, just in case a client has defined several email configurations for the same organization.

---
M src/org/openbravo/email/EmailUtils.java
---
(0057720)
shankarb   
2013-04-04 05:11   
Code reviewed and tested in pi changeset 4a36ca94ba63
(0057761)
hudsonbot   
2013-04-07 01:03   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/b03a336bd23b [^]

Maturity status: Test