Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0041333
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 02. Master data managementmajoralways2019-07-10 15:292019-08-22 14:45
ReporterfranciscoView Statuspublic 
Assigned Tocollazoandy4 
PriorityhighResolutionfixedFixed in Version3.0PR19Q4
StatusclosedFix in branchFixed in SCM revision1e75513925e8
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Todmiguelez
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0041333: Duplicate values when create new Product characteristics values

DescriptionDuplicate values when create new Product characteristics values
Steps To ReproduceTested in Pi:
1.- Go to the back-end and create a Product characteristic:
    - Org: F&B España - Región Norte
    - Name: Test
    - Variant = N
    - Active = Y

2.- Go to values tab for "Test"
    - Name = Test1
    - SummaryLevel = N
    - Active = Y

3.- Go to the back-end and insert again the same characteristic value
 
It is allowing to have duplicates values
Proposed Solution- Add a trigger in m_characteristic to check the uniqueness of the values when the variant flag is enabled.

- Disallow to modify the variant flag once the characteristic has values
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0040547 closednonofrancisco m_ch_value_trg2 is not taking into account the new created register 
related to defect 0041485 closeddmiguelez API Change Request 

-  Notes
(0113705)
dmiguelez (developer)
2019-07-30 13:05

Test Plan:

Log in as F&B Admin

1.Go to Product Characteristic Window and create a new record with:
    - Org: F&B España - Región Norte
    - Name: Test
    - Variant = N
    - Active = Y
Go to Values tab and create a new record with
    - Name = Test1
    - SummaryLevel = N
    - Active = Y
Try to create a new record with the same value, a message is shown:
Saving failed. There is already a Characteristic Value with the same (Characteristic, Name). (Characteristic, Name) must be unique. You must change the values entered.

2. Create another Product Characteristic named test2 with a Value named Test. It is possible to do so

3.Go to Product Characteristic Window and create a new record with:
    - Org: F&B España - Región Norte
    - Name: Test
    - Variant = Y
    - Active = Y
Go to Values tab and create a new record with
    - Name = Test1
    - SummaryLevel = N
    - Active = Y
Try to create a new record with the same value, a message is shown:
Saving failed. There is already a Characteristic Value with the same (Characteristic, Name). (Characteristic, Name) must be unique. You must change the values entered.
(0113707)
hgbot (developer)
2019-07-30 13:06

Repository: erp/devel/pi
Changeset: 1e75513925e887bb617dbba6977183c992178823
Author: Armaignac <collazoandy4 <at> gmail.com>
Date: Tue Jul 30 12:46:59 2019 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/1e75513925e887bb617dbba6977183c992178823 [^]

Fixes issue 41333:Duplicate values when create Product characteristics values

If a process create product characteristic with insert sql query a duplicate product
characteristics value can be added to the database because the validation in the
Event Observer doesn't take into account while inserting.

Now a unique constraint was added to the characteristic_id and name columns and
a new build validation was created to check duplicated product characteristic values

---
M src-db/database/model/tables/M_CH_VALUE.xml
M src/org/openbravo/event/CharacteristicValueEventHandler.java
A src-util/buildvalidation/build/classes/org/openbravo/buildvalidation/DuplicatedCharacteristicName.class
A src-util/buildvalidation/build/classes/org/openbravo/buildvalidation/DuplicatedCharacteristicNameData.class
A src-util/buildvalidation/src/org/openbravo/buildvalidation/DuplicatedCharacteristicName.java
A src-util/buildvalidation/src/org/openbravo/buildvalidation/DuplicatedCharacteristicName_data.xsql
---
(0113713)
hgbot (developer)
2019-07-30 16:20

Repository: erp/pmods/org.openbravo.retail.sampledata
Changeset: a897052f511426110ce64da03844ba6c1af7d563
Author: Gorka Gil <gorka.gil <at> openbravo.com>
Date: Tue Jul 30 16:20:01 2019 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sampledata/rev/a897052f511426110ce64da03844ba6c1af7d563 [^]

Related to issue 41333: fixes wrong data in retail sampledata

---
M referencedata/sampledata/The_White_Valley_Group/AD_TREENODE.xml
M referencedata/sampledata/The_White_Valley_Group/M_CH_VALUE.xml
M referencedata/sampledata/The_White_Valley_Group/M_PRODUCT_CH_CONF.xml
---
(0113719)
dmiguelez (developer)
2019-07-30 17:12

Code Review + Testing Ok
(0113723)
hgbot (developer)
2019-07-31 09:47

Repository: erp/devel/pi
Changeset: e52c191231ee7a4e491cd53febde31170d798036
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Wed Jul 31 09:32:55 2019 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/e52c191231ee7a4e491cd53febde31170d798036 [^]

Related to Issue 41333. Modifies final Core version till which the buildvalidation
should be executed. It uses the changeset number to ensure it will work in
upgrades for Q4

---
M src-util/buildvalidation/build/classes/org/openbravo/buildvalidation/DuplicatedCharacteristicName.class
M src-util/buildvalidation/src/org/openbravo/buildvalidation/DuplicatedCharacteristicName.java
---
(0113727)
hgbot (developer)
2019-07-31 11:22

Repository: tools/automation/pi-mobile
Changeset: f732305db94a2e3c8d993e4642e394ac4e0fbb8e
Author: Gorka Gil <gorka.gil <at> openbravo.com>
Date: Wed Jul 31 11:22:18 2019 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/f732305db94a2e3c8d993e4642e394ac4e0fbb8e [^]

Related to issue 41333: update test of retail api with the new values

---
M src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/retailapi/POSProductChAndConfRequest.java
---
(0113875)
dmiguelez (developer)
2019-08-07 12:28

Commit related to Issue:
https://code.openbravo.com/erp/devel/pi/rev/f89790531335 [^]

Fixes problem in Build Validation when updating from an OB version in which m_ch_value table does not exist.
(0114184)
hudsonbot (developer)
2019-08-22 14:45

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/ad3efd3bd07c [^]
Maturity status: Test
(0114185)
hudsonbot (developer)
2019-08-22 14:45

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/ad3efd3bd07c [^]
Maturity status: Test
(0114213)
hudsonbot (developer)
2019-08-22 14:45

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/ad3efd3bd07c [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2019-07-10 15:29 francisco New Issue
2019-07-10 15:29 francisco Assigned To => Triage Finance
2019-07-10 15:29 francisco Modules => Core
2019-07-10 15:29 francisco Resolution time => 1564524000
2019-07-10 15:29 francisco Triggers an Emergency Pack => No
2019-07-10 15:54 Sandrahuguet Relationship added related to 0040547
2019-07-10 15:55 Sandrahuguet Assigned To Triage Finance => collazoandy4
2019-07-19 15:17 collazoandy4 Status new => scheduled
2019-07-30 12:40 dmiguelez Relationship added related to 0041485
2019-07-30 13:05 dmiguelez Note Added: 0113705
2019-07-30 13:06 hgbot Checkin
2019-07-30 13:06 hgbot Note Added: 0113707
2019-07-30 13:06 hgbot Status scheduled => resolved
2019-07-30 13:06 hgbot Resolution open => fixed
2019-07-30 13:06 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/1e75513925e887bb617dbba6977183c992178823 [^]
2019-07-30 16:20 hgbot Checkin
2019-07-30 16:20 hgbot Note Added: 0113713
2019-07-30 17:12 dmiguelez Review Assigned To => dmiguelez
2019-07-30 17:12 dmiguelez Note Added: 0113719
2019-07-30 17:12 dmiguelez Status resolved => closed
2019-07-30 17:12 dmiguelez Fixed in Version => 3.0PR19Q4
2019-07-31 09:47 hgbot Checkin
2019-07-31 09:47 hgbot Note Added: 0113723
2019-07-31 11:22 hgbot Checkin
2019-07-31 11:22 hgbot Note Added: 0113727
2019-08-07 12:28 dmiguelez Note Added: 0113875
2019-08-22 14:45 hudsonbot Checkin
2019-08-22 14:45 hudsonbot Note Added: 0114184
2019-08-22 14:45 hudsonbot Checkin
2019-08-22 14:45 hudsonbot Note Added: 0114185
2019-08-22 14:45 hudsonbot Checkin
2019-08-22 14:45 hudsonbot Note Added: 0114213


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker