Openbravo Issue Tracking System - Openbravo ERP | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0019344 | Openbravo ERP | A. Platform | public | 2011-12-22 13:15 | 2012-01-12 10:26 |
Reporter | azabaleta | ||||
Assigned To | dbaz | ||||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Platform | OS | 5 | OS Version | ||
Product Version | 3.0MP6 | ||||
Target Version | Fixed in Version | ||||
Merge Request Status | |||||
Review Assigned To | |||||
OBNetwork customer | |||||
Web browser | |||||
Modules | Core | ||||
Support ticket | |||||
Regression level | |||||
Regression date | |||||
Regression introduced in release | |||||
Regression introduced by commit | |||||
Triggers an Emergency Pack | No | ||||
Summary | 0019344: Add 2 columns to the email sent audit | ||||
Description | Hi, now, when openbravo sends an email from a document via PrintController, at the end, it makes a insert in an audit table called c_poc_email . I suggest you add 2 more columns to that table so the admin can track which table sent the email and which is the document id that was sent. I attach diff for the 2 sources that need to be modified and here I add the 2 colums needed in the database. ALTER TABLE c_poc_email ADD COLUMN tablename character varying(60); ALTER TABLE c_poc_email ADD COLUMN document_id character varying(32); Thanks | ||||
Steps To Reproduce | diff -r f330b414cd5a src/org/openbravo/erpCommon/utility/reporting/printing/Email_data.xsql --- a/src/org/openbravo/erpCommon/utility/reporting/printing/Email_data.xsql Tue Nov 30 08:37:26 2010 +0100 +++ b/src/org/openbravo/erpCommon/utility/reporting/printing/Email_data.xsql Thu Dec 22 13:11:15 2011 +0100 @@ -78,7 +78,9 @@ DATEOFEMAIL, SUBJECT , BODY , - C_BPARTNER_ID + C_BPARTNER_ID, + TABLENAME, + DOCUMENT_ID ) VALUES ( @@ -99,6 +101,8 @@ TO_DATE(?, 'YYYYMMDDHH24MISS'), ?, ?, + ?, + ?, ? ) ]]></Sql> @@ -117,6 +121,8 @@ <Parameter name="subject"/> <Parameter name="body"/> <Parameter name="cBpartnerId"/> + <Parameter name="tableName"/> + <Parameter name="documentId"/> </SqlMethod> <SqlMethod name="insertEmailUser" type="preparedStatement" connection="true" return="rowCount"> diff -r f330b414cd5a src/org/openbravo/erpCommon/utility/reporting/printing/PrintController.java --- a/src/org/openbravo/erpCommon/utility/reporting/printing/PrintController.java Tue Nov 30 08:37:26 2010 +0100 +++ b/src/org/openbravo/erpCommon/utility/reporting/printing/PrintController.java Thu Dec 22 13:11:15 2011 +0100 @@ -747,7 +747,7 @@ EmailData.insertEmail(conn, this, newEmailId, clientId, organizationId, userId, EmailType.OUTGOING.getStringValue(), from, to, cc, bcc, dateOfEmail, subject, body, - bPartnerId); + bPartnerId, report.getDocumentType().getTableName(), documentData.documentId); releaseCommitConnection(conn); } catch (final NoConnectionAvailableException exception) { | ||||
Proposed Solution | |||||
Additional Information | |||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | |||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2011-12-22 13:15 | azabaleta | New Issue | |||
2011-12-22 13:15 | azabaleta | Assigned To | => alostale | ||
2011-12-22 13:15 | azabaleta | Modules | => Core | ||
2011-12-27 17:33 | iperdomo | Assigned To | alostale => dbaz | ||
2012-01-02 13:27 | dbaz | Note Added: 0044093 | |||
2012-01-03 13:31 | iperdomo | Note Added: 0044100 | |||
2012-01-03 13:35 | iperdomo | Status | new => scheduled | ||
2012-01-03 13:35 | iperdomo | fix_in_branch | => pi | ||
2012-01-03 21:31 | hgbot | Checkin | |||
2012-01-03 21:31 | hgbot | Note Added: 0044104 | |||
2012-01-03 21:31 | hgbot | Status | scheduled => resolved | ||
2012-01-03 21:31 | hgbot | Resolution | open => fixed | ||
2012-01-03 21:31 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/da9ac3c4fcbe9932536213a7db25360391adb02b [^] | ||
2012-01-04 11:12 | iperdomo | Note Added: 0044110 | |||
2012-01-04 11:12 | iperdomo | Status | resolved => closed | ||
2012-01-12 10:26 | hudsonbot | Checkin | |||
2012-01-12 10:26 | hudsonbot | Note Added: 0044286 |
Notes | |||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|