Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012705Openbravo ERP04. Warehouse managementpublic2010-03-17 10:032010-05-20 11:15
Reporternetworkb 
Assigned Toadrianromero 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version2.40MP12 
Target Version2.40MP13Fixed in Version 
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
Summary0012705: Phisical Inventory processed twice
DescriptionNavigating back with the browser (instead of with the application button) you are able to process an inventory two times.
Steps To Reproduce1. Warehouse Management || Transactions || Physical Inventory || Header : create new record
2. Warehouse Management || Transactions || Physical Inventory || Header >> Lines : add one line
3. Back to header and process inventory
4. Press "Go back" button of Firefox
5. Try to process the inventory again. You will obtain an error.
6. Try to process the inventory again. You will be able to process it and you can check that a second line has been inserted in m_transaction
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00127112.40MP13 closed adrianromero Phisical Inventory processed twice 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-03-17 10:03networkbNew Issue
2010-03-17 10:03networkbAssigned To => adrianromero
2010-03-17 17:39adrianromeroIssue cloned0012711
2010-03-17 17:39adrianromeroRelationship addeddepends on 0012711
2010-03-17 17:40adrianromeroStatusnew => scheduled
2010-03-17 17:40adrianromerofix_in_branch => pi
2010-05-13 17:41adrianromeroNote Added: 0027210
2010-05-13 17:41adrianromeroStatusscheduled => closed
2010-05-13 17:41adrianromeroResolutionopen => unable to reproduce
2010-05-13 18:03adrianromeroStatusclosed => new
2010-05-13 18:03adrianromeroResolutionunable to reproduce => open
2010-05-13 18:04adrianromeroNote Added: 0027211
2010-05-13 18:04adrianromeroStatusnew => scheduled
2010-05-17 17:20hgbotCheckin
2010-05-17 17:20hgbotNote Added: 0027378
2010-05-17 17:20hgbotStatusscheduled => resolved
2010-05-17 17:20hgbotResolutionopen => fixed
2010-05-17 17:20hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3086c7323bbf21a0df43d7a6ea5f1dacc8857e61 [^]
2010-05-17 17:20adrianromeroNote Added: 0027379
2010-05-17 20:07sureshbabuNote Added: 0027395
2010-05-17 20:07sureshbabuStatusresolved => closed
2010-05-18 00:00anonymoussf_bug_id0 => 3002949
2010-05-20 11:15hudsonbotCheckin
2010-05-20 11:15hudsonbotNote Added: 0027587

Notes
(0027210)
adrianromero   
2010-05-13 17:41   
This issue cannot be reproduced in 2.50. It works properly. When pressing the button back it asks to resend the information and if the user presses yes the page is reloaded with the correct status.
(0027211)
adrianromero   
2010-05-13 18:04   
It was a false test result.
This issue can be reproduced in 2.50 too.
(0027378)
hgbot   
2010-05-17 17:20   
Repository: erp/devel/pi
Changeset: 3086c7323bbf21a0df43d7a6ea5f1dacc8857e61
Author: Adrián Romero <adrianromero <at> openbravo.com>
Date: Mon May 17 17:19:20 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/3086c7323bbf21a0df43d7a6ea5f1dacc8857e61 [^]

Fixes issue 0012705: Phisical Inventory processed twice
It has been modified the M_INVENTORY trigger to avoid to set as unprocessed a physycal inventory record that has already been processed. This avoids to process the same record twice

---
M src-db/database/model/triggers/M_INVENTORY_TRG.xml
---
(0027379)
adrianromero   
2010-05-17 17:20   
* Steps to test
Just follow the steps to test and notice that a physical inventory cannot be processed twice.

* Other areas affected by the changeset
Only it is affected the Physical Inventory window. No other areas affected.
(0027395)
sureshbabu   
2010-05-17 20:07   
verified. system not allowing the user to process the Physical inventory twice on clicking the Back button
(0027587)
hudsonbot   
2010-05-20 11:15   
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/3086c7323bbf [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/a7901e097e1a [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17407.obx [^]