Openbravo Issue Tracking System - Localization Pack: Spain
View Issue Details
0044801Localization Pack: SpainCIF and NIF validatorpublic2020-08-12 16:482020-09-11 12:09
shuehner 
Mery Anelo 
normalminorhave not tried
closedfixed 
5
 
 
vmromanos
0044801: SE_CIFNIF_Validator is an old-style callout and should be refactored to use SimpleCallout
This callout is one of the few ones left over not not refactored to be SimpleCallout (issue 36067 refactored every supported module except a few one).
product/mods/org.openbravo.module.cifnifvalidator/src/org/openbravo/module/cifnifvalidator/erpCommon/ad_callouts/SE_CIFNIF_Validator.java: resultado.append("var respuesta = new Array( \n");
No tags attached.
blocks design defect 0044795 new Triage Omni OMS Openbravo ERP Tracking issue to refactor remaining old callouts to extend SimpleCallout 
Issue History
2020-08-12 16:48shuehnerNew Issue
2020-08-12 16:48shuehnerAssigned To => Jorge Bravo
2020-08-12 16:48shuehnerRelationship addedblocks 0044795
2020-08-14 14:28Jorge BravoAssigned ToJorge Bravo => Mery Anelo
2020-08-17 12:36shuehnerNote Added: 0122096
2020-08-17 12:40shuehnerNote Edited: 0122096bug_revision_view_page.php?bugnote_id=0122096#r21495
2020-09-05 03:11Mery AneloNote Added: 0122698
2020-09-05 03:11Mery AneloStatusnew => scheduled
2020-09-10 16:42hgbotNote Added: 0122928
2020-09-11 12:08hgbotResolutionopen => fixed
2020-09-11 12:08hgbotStatusscheduled => resolved
2020-09-11 12:08hgbotNote Added: 0122956
2020-09-11 12:08hgbotNote Added: 0122957
2020-09-11 12:08hgbotNote Added: 0122958
2020-09-11 12:08hgbotNote Added: 0122959
2020-09-11 12:09vmromanosReview Assigned To => vmromanos
2020-09-11 12:09vmromanosStatusresolved => closed

Notes
(0122096)
shuehner   
2020-08-17 12:36   
(edited on: 2020-08-17 12:40)
EXECUTE action of callouts is obsolete and does not work for many years. That part of the callout should be reviewed and fixed while working on this also.

Note that this is in codepath trying to support old 2.50 style UI. So probably that could be just removed as no longer required.

(0122698)
Mery Anelo   
2020-09-05 03:11   
Test plan:
- Login as System Admin
- Manually associated the SE_CIFNIF_Validator callout to the TaxID column of the C_BPartner table.
- Compile the Business Partner window and redeploy the application.
- Login as F&B Internation Group Admin
- Set values to the field "Tax ID" in "Business Partner" windows and check that you have a message when this value is wrong.
(0122928)
hgbot   
2020-09-10 16:42   
Merge Request created: https://gitlab.com/openbravo/product/mods/org.openbravo.module.cifnifvalidator/-/merge_requests/1 [^]
(0122956)
hgbot   
2020-09-11 12:08   
Repository: https://gitlab.com/openbravo/product/mods/org.openbravo.module.cifnifvalidator [^]
Changeset: 6c4910cbd769d95ccefc420be5cd25f876606da5
Author: Mery Anelo <mery.anelo@smfconsulting.es>
Date: 2020-09-10T11:39:59-03:00
URL: https://gitlab.com/openbravo/product/mods/org.openbravo.module.cifnifvalidator/-/commit/6c4910cbd769d95ccefc420be5cd25f876606da5 [^]

Fixed BUG-44801: CIFNIF validations to SimpleCallout

Format java.

---
M src/org/openbravo/module/cifnifvalidator/erpCommon/ad_callouts/SE_CIFNIF_Validator.java
---
(0122957)
hgbot   
2020-09-11 12:08   
Repository: https://gitlab.com/openbravo/product/mods/org.openbravo.module.cifnifvalidator [^]
Changeset: 17c8944083f66028aa873307efdd11ecba0921f3
Author: Mery Anelo <mery.anelo@smfconsulting.es>
Date: 2020-09-10T11:41:25-03:00
URL: https://gitlab.com/openbravo/product/mods/org.openbravo.module.cifnifvalidator/-/commit/17c8944083f66028aa873307efdd11ecba0921f3 [^]

Fixed BUG-44801: CIFNIF validations to SimpleCallout

SE_CIFNIF_Validator transformed from old-style callout to SimpleCallout.

---
M src/org/openbravo/module/cifnifvalidator/erpCommon/ad_callouts/SE_CIFNIF_Validator.java
---
(0122958)
hgbot   
2020-09-11 12:08   
Repository: https://gitlab.com/openbravo/product/mods/org.openbravo.module.cifnifvalidator [^]
Changeset: 382dadf3b510fbe2ca4e4a1273d2907ac8593b55
Author: Víctor Martínez Romanos <victor.martinez@openbravo.com>
Date: 2020-09-11T12:00:48+02:00
URL: https://gitlab.com/openbravo/product/mods/org.openbravo.module.cifnifvalidator/-/commit/382dadf3b510fbe2ca4e4a1273d2907ac8593b55 [^]

Related to ISSUE-44801: Simplified code

Removed useless check for Openbravo3.
Use standard OBMessageUtils.messageBD() method.

---
M src/org/openbravo/module/cifnifvalidator/erpCommon/ad_callouts/SE_CIFNIF_Validator.java
---
(0122959)
hgbot   
2020-09-11 12:08   
Merge request merged: https://gitlab.com/openbravo/product/mods/org.openbravo.module.cifnifvalidator/-/merge_requests/1 [^]