Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015943Openbravo ERP07. Sales managementpublic2011-02-10 17:122011-05-03 11:21
Reporteregoitz 
Assigned Toadrianromero 
PriorityimmediateSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version2.50MP25 
Target Version2.50MP28Fixed in Version2.50MP29 
Merge Request Status
Review Assigned To
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0015943: The date of documents created by a process includes the time, but when creating manually they don't include time
DescriptionThe date of documents created by a process includes the time, but when creating manually they don't include time.
This is a problem in some triggers because when updating a trigger, although the data is not modified, the triggers identify the data as modified.

This is beceuse the new.date is taken without time, and the old.date with time,
so if you compare them they looks different.

Steps To Reproduce-Create an standar sales order with a line.
-complete it
-Go to generate shipment from orders, and select the created order.
-The invoice for the order is created, completed and processed.
-Go to the new invoice.
-Try to modify the description , only the description.
-You get a message saying the the process is completed/posted.
*This message should not appear because the trigger allows modifying the description, but in this case, the triggers identifies that the date has changed, becuase is comparing a date with time , with a date wihout time.
Proposed SolutionThe processes that creates documents, such as m_inout_create or c_invoice_create
should insert the date formated to not use time.
Additional Information
TagsNo tags attached.
Relationships
depends on defect 0016952 closed dalsasua The date of documents created by a process includes the time, but when creating manually they don't include time 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-02-10 17:12egoitzNew Issue
2011-02-10 17:12egoitzAssigned To => adrianromero
2011-02-10 17:12egoitzModules => Core
2011-02-10 17:12egoitzOBNetwork customer => No
2011-02-10 17:27egoitzIssue Monitored: networkb
2011-02-10 17:43egoitzNote Added: 0034139
2011-04-29 10:45egoitzOBNetwork customerNo => Yes
2011-04-29 12:18adrianromeroPriorityhigh => immediate
2011-04-29 12:18adrianromeroStatusnew => scheduled
2011-04-29 12:18adrianromeroTypedefect => backport
2011-05-02 10:15adrianromeroIssue cloned0016952
2011-05-02 10:15adrianromeroRelationship addeddepends on 0016952
2011-05-02 10:16hgbotCheckin
2011-05-02 10:16hgbotNote Added: 0036238
2011-05-02 10:16hgbotStatusscheduled => resolved
2011-05-02 10:16hgbotResolutionopen => fixed
2011-05-02 10:16hgbotFixed in SCM revision => http://code.openbravo.com/erp/stable/2.50/rev/b89aedcaf3d264d15ce214507a4ca2e3466b28c1 [^]
2011-05-02 10:23adrianromeroNote Added: 0036239
2011-05-03 11:21egoitzNote Added: 0036303
2011-05-03 11:21egoitzStatusresolved => closed
2011-05-03 11:21egoitzFixed in Version => 2.50MP29

Notes
(0034139)
egoitz   
2011-02-10 17:43   
The database is postgresql 8.4. ticket updated
(0036238)
hgbot   
2011-05-02 10:16   
Repository: erp/stable/2.50
Changeset: b89aedcaf3d264d15ce214507a4ca2e3466b28c1
Author: Adrián Romero <adrianromero <at> openbravo.com>
Date: Mon May 02 10:14:57 2011 +0200
URL: http://code.openbravo.com/erp/stable/2.50/rev/b89aedcaf3d264d15ce214507a4ca2e3466b28c1 [^]

Fixes issue 0015943: The date of documents created by a process includes the time, but when creating manually they don't include time
It has been removed the time in the fields movementdate and dateacct when creating a new shipment in the function M_INOUT_CREATE

---
M src-db/database/model/functions/M_INOUT_CREATE.xml
---
(0036239)
adrianromero   
2011-05-02 10:23   
* Testing the issue

Follow the steps to test and verify that now the description can be modified

* Other areas affected

No other areas affected.
(0036303)
egoitz   
2011-05-03 11:21   
Verified. The problem has not been reproduced witht the steps.