Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034180Openbravo ERPA. Platformpublic2016-10-10 11:032016-12-16 18:38
ReporterdaniOpenbravo 
Assigned Toinigosanchez 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS30OS Version14.04
Product Version 
Target VersionFixed in Version3.0PR17Q1 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034180: Missing code to copy some tables from AD dataset which makes impossible importing translations
DescriptionSome code is missing to copy several tables from AD Dataset. That makes impossible to import translations for those tables (that pl is in charge to populate _TRL tables from base table, and then xml import just does update sql's).
We noticed that on table M_OFFER_TYPE which is part of dataset AD. However, cross-checking ad_dataset tables having matching trl_tables with "inserts in ad_create_languages" shows that maybe more are missing.
Phidias has reported for BUT the same problem in these tables:
- AD_MESSAGE_TRL_fr_FR
- AD_REF_LIST_TRL_fr_FR
- AD_TEXTINTERFACES_TRL_fr_FR
- AD_FIELD_TRL_fr_FR
- OBUIAPP_PROCESS_TRL_fr_FR
- OBUISEL_SELECTOR_FIELD_TRL_fr_FR
Steps To ReproduceIn backend:
- Add a translation for some M_OFFER_TYPE record.
- realice that those translations are not applied correctly. In BUT customer we had to export those one to their template.
Proposed SolutionSHU proposed (email: "Missing AD tables in ad_create_languages"):
1. Small bugfix to add at least missing m_offer_type to the pl.
2. Maybe add some code to cross-check ad_tables with that pl function (note need to take into account some ad_module_id<>0 is handled via extra pl's in extension point.
idea: unit or ci test to run above pl for a language -> after that verify that numRows in new trl-table filtered for that language == numRows in base-table.
Ideally make sure every table in AD has at least 1 row somewhere (at least for that test-case).


Additional Information
TagsSER-QA
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-10-10 11:03daniOpenbravoNew Issue
2016-10-10 11:03daniOpenbravoAssigned To => alostale
2016-10-10 11:03daniOpenbravoModules => Core
2016-10-10 11:03daniOpenbravoTriggers an Emergency Pack => No
2016-10-10 11:04daniOpenbravoTag Attached: SER-QA
2016-10-10 11:04daniOpenbravoIssue Monitored: daniOpenbravo
2016-10-17 13:32daniOpenbravoResolution time => 1477260000
2016-10-18 14:32inigosanchezStatusnew => scheduled
2016-10-18 14:32inigosanchezAssigned Toalostale => inigosanchez
2016-10-20 13:01inigosanchezReview Assigned To => caristu
2016-10-20 13:09hgbotCheckin
2016-10-20 13:09hgbotNote Added: 0090720
2016-10-20 13:09hgbotStatusscheduled => resolved
2016-10-20 13:09hgbotResolutionopen => fixed
2016-10-20 13:09hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/ae110fab1bff7f5b5370e769cb7899b67f096ab3 [^]
2016-10-21 13:00caristuNote Added: 0090764
2016-10-21 13:00caristuStatusresolved => new
2016-10-21 13:00caristuResolutionfixed => open
2016-10-21 14:20hgbotCheckin
2016-10-21 14:20hgbotNote Added: 0090774
2016-10-21 14:20inigosanchezStatusnew => scheduled
2016-10-21 14:20inigosanchezStatusscheduled => resolved
2016-10-21 14:20inigosanchezResolutionopen => fixed
2016-10-21 14:21inigosanchezFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/ae110fab1bff7f5b5370e769cb7899b67f096ab3 [^] => http://code.openbravo.com/erp/devel/pi/rev/8f17df764d0f43196fc331890e4b1ec83d812d8d [^]
2016-10-21 15:20caristuNote Added: 0090776
2016-10-21 15:20caristuStatusresolved => closed
2016-10-21 15:20caristuFixed in Version => 3.0PR17Q1
2016-12-16 18:37hudsonbotCheckin
2016-12-16 18:37hudsonbotNote Added: 0092538
2016-12-16 18:38hudsonbotCheckin
2016-12-16 18:38hudsonbotNote Added: 0092543

Notes
(0090720)
hgbot   
2016-10-20 13:09   
Repository: erp/devel/pi
Changeset: ae110fab1bff7f5b5370e769cb7899b67f096ab3
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Wed Oct 19 15:41:26 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/ae110fab1bff7f5b5370e769cb7899b67f096ab3 [^]

Fixed issue 34180: A missing code in AD_LANGUAGE_CREATE function.

The problem was the missing code in AD_LANGUAGE_CREATE function
related with M_Offer_Type table. A translation problem in
"Discounts and Promotions Types" is caused by this missing code.

The problem has resolved by adding the missing code related with
M_Offer_Type. Now M_Offer_Type is managed properly in
AD_LANGUAGE_CREATE function.

---
M src-db/database/model/functions/AD_LANGUAGE_CREATE.xml
---
(0090764)
caristu   
2016-10-21 13:00   
Reopened: https://docs.google.com/spreadsheets/d/18l2biKWeG6iGDIPqO6VuYQ_TtP_6YxyQU4KSrPvQlJ8/edit#gid=0 [^]

Description column of M_Offer_Type_Trl is not being populated by the procedure.
(0090774)
hgbot   
2016-10-21 14:20   
Repository: erp/devel/pi
Changeset: 8f17df764d0f43196fc331890e4b1ec83d812d8d
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Fri Oct 21 14:16:10 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/8f17df764d0f43196fc331890e4b1ec83d812d8d [^]

Related with issue 34180: M_Offer_Type_Trl.Description column is not populated

---
M src-db/database/model/functions/AD_LANGUAGE_CREATE.xml
---
(0090776)
caristu   
2016-10-21 15:20   
Code review + tested OK: using the verify languages process, the records are created properly in the M_Offer_Type_Trl table.
(0092538)
hudsonbot   
2016-12-16 18:37   
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/dc8bf00badd0 [^]
Maturity status: Test
(0092543)
hudsonbot   
2016-12-16 18:38   
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/dc8bf00badd0 [^]
Maturity status: Test