Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012265Openbravo ERPA. Platformpublic2010-02-15 10:242010-03-10 18:30
Reportersureshbabu 
Assigned Toshuehner 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version2.50MP12Fixed in Version2.50MP12 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0012265: Broken link in Audit Trail - Deleted Records window
DescriptionBroken link in Audit Trail - Deleted Records window
Steps To ReproduceLogin as system Admin

Then Navigate to Application Dictionary || Tables and Columns || Table
And check the "Fully audited checkbox" for all the below tables
C_order,
C_Order_Discount
C_Order_Header_V
C_OrderLine,
C_Orderline_Offer,
C_OrderLineTax,
C_OrderLineTax_V,
C_order_Open,
C_orderTax


Then start the "Update Audit Trail Infrastructure" process


Navigate to Master Data Management || Pricing || Price Adjustments || Adjustments
click new
Enter Name: testing 2
Starting Date: 01-01-2010
Discount: 10%
Include Business Partner: All Selected
Include Products: All Selected


Move to product tab;
Click new
Select : any valid product (Product 1)
Click new
Select : any valid product (Product 2)
Click new
Select : any valid product (Product 3)

Move to Business partner tab
Click new
Select : any valid BP (BP1)



Navigate to Sales Order window
Click new to create a new sales order
And enter the required details
Document Type: Standard order
BP: Any valid Customer (BP1)
Click Save
Move to lines tab
Click new to create a new line item:10
Select any product (Product 1)
Then Click new to create a new line item:20
Select any product (Product 2)
Then Delete line item:20

"One Row Deleted" message appears

From Line Item:10

Click "Show Audit Trial" icon
Then click "View Deleted Records" link
verify the line 20 appears in the table.

Then click the "Price Adjustments" link at the bottom of the window..

Price adjustment details will appear in the screen

Then click "Back to History" link

Actual Result: No Records appears in the screen
Proposed Solution
Additional Information
TagsAudit trial
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-02-15 10:24sureshbabuNew Issue
2010-02-15 10:24sureshbabuAssigned To => alostale
2010-02-15 10:25sureshbabuTag Attached: Audit trial
2010-02-15 11:29alostaleAssigned Toalostale => shuehner
2010-02-15 11:29alostaleStatusnew => scheduled
2010-02-15 11:29alostalefix_in_branch => pi
2010-02-15 15:53psarobePriorityhigh => urgent
2010-02-15 15:53psarobeTarget Version => 2.50MP12
2010-02-15 15:53psarobefix_in_branchpi =>
2010-02-15 16:29shuehnerNote Added: 0024278
2010-02-16 15:00hgbotCheckin
2010-02-16 15:00hgbotNote Added: 0024413
2010-02-16 15:00hgbotStatusscheduled => resolved
2010-02-16 15:00hgbotResolutionopen => fixed
2010-02-16 15:00hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/195dda7f410b7aec6ed34084a3b0fab775f2d629 [^]
2010-02-16 15:58alostaleStatusresolved => closed
2010-02-16 15:58alostaleFixed in Version => 2.50MP12
2010-02-17 00:01anonymoussf_bug_id0 =>
2010-02-18 00:31hudsonbotCheckin
2010-02-18 00:31hudsonbotNote Added: 0024539
2010-03-10 18:30anonymoussf_bug_id => 2968067

Notes
(0024278)
shuehner   
2010-02-15 16:29   
Reproduced. Problem is that the behaviour definition of the 'Back to history' link is unclear in the case where you followed any link to a child (the ones below the grid).

Initial design of the 'Back to history' link was to give a Back... option after you switched to deleted records view.

However the 'Go back' option is not clearly defined if you followed into a child tab, as the problematic point is to what record to go back.

Possible solution to avoid confusion is to hide the 'Go back' link in the cases where no clear 'back..' target exists (when you followed into a child tab).
(0024413)
hgbot   
2010-02-16 15:00   
Repository: erp/devel/pi
Changeset: 195dda7f410b7aec6ed34084a3b0fab775f2d629
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Tue Feb 16 15:00:50 2010 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/195dda7f410b7aec6ed34084a3b0fab775f2d629 [^]

Fixed 12286, Fixed 12265, fixed to audit trail popup
- don't assume ad_element.columnname is unique
- hide go back link when navigated into a child tab
- format modification time without milliseconds
- small ui fixes

---
M src-db/database/sourcedata/AD_TEXTINTERFACES.xml
M src/org/openbravo/erpCommon/businessUtility/AuditTrailPopup.java
M src/org/openbravo/erpCommon/businessUtility/AuditTrailPopupDeleted.html
M src/org/openbravo/erpCommon/businessUtility/AuditTrailPopupHistory.html
---
(0024539)
hudsonbot   
2010-02-18 00:31   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/195dda7f410b [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/7c1b489fb0a0 [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.16459.obx [^]