Openbravo Issue Tracking System - Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0049555ModulesCopy Entity Processpublic2022-06-14 18:322022-06-15 10:05
Reportercaristu 
Assigned Toalostale 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Regression date2022-05-31
Regression introduced by commit
Regression levelPre packaging ( pi )
Review Assigned To
Support ticket
OBNetwork customer
Regression introduced in release
Summary0049555: Warning when executing a WeldBaseTest having the Copy Entity Process module installed
DescriptionHaving the Copy Entity Process module installed, the following warning is raised when executing a WeldBaseTest:

WARN org.jboss.weld.Bootstrap - WELD-000167: Class org.openbravo.copyprocess.process.fileimport.CopyProcessFileColumnHandler is annotated with @RequestScoped but it does not declare an appropriate constructor therefore is not registered as a bean!
Steps To Reproduce1) Install the Copy Entity Process module
2) Execute any test extending WeldBaseTest. The warning appears during the test initialization
Proposed SolutionThe CopyProcessFileColumnHandler[1] is an interface, so annotating it as @RequestScoped has no effect on the classes implementing it. We have to review it and decide one of these two options:

a) Just remove the annotation from the CopyProcessFileColumnHandler
b) Remove the annotation from the CopyProcessFileColumnHandler and add the annotation into the classes implementing it

[1] https://gitlab.com/openbravo/product/pmods/org.openbravo.copyprocess/-/blob/master/src/org/openbravo/copyprocess/process/fileimport/CopyProcessFileColumnHandler.java [^]
Additional Information
TagsNo tags attached.
Relationships
caused by feature request 0049298 closed njimenez Retail Modules Generalize Copy OB Object From Copy Store 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2022-06-14 18:32caristuNew Issue
2022-06-14 18:32caristuAssigned To => Triage Platform Base
2022-06-14 18:39caristuProposed Solution updated
2022-06-15 08:18alostaleRelationship addedcaused by 0049298
2022-06-15 08:19alostaleRegression date => 2022-05-31
2022-06-15 08:19alostaleRegression level => Pre packaging ( pi )
2022-06-15 10:00alostaleAssigned ToTriage Platform Base => alostale
2022-06-15 10:02hgbotNote Added: 0138359
2022-06-15 10:05hgbotResolutionopen => fixed
2022-06-15 10:05hgbotStatusnew => closed
2022-06-15 10:05hgbotNote Added: 0138360
2022-06-15 10:05hgbotNote Added: 0138361

Notes
(0138359)
hgbot   
2022-06-15 10:02   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.copyprocess/-/merge_requests/5 [^]
(0138360)
hgbot   
2022-06-15 10:05   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.copyprocess [^]
Changeset: a1cd51e77141a2979c602fa35529c7894746c71a
Author: Asier Lostalé <asier.lostale@openbravo.com>
Date: 15-06-2022 10:00:50
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.copyprocess/-/commit/a1cd51e77141a2979c602fa35529c7894746c71a [^]

fixes BUG-49555: useless CDI scope definition in interface

CSDI scopes in interfaces are not taken into account. In this case
removing the scope (leaving default @Dependent scope) for classes
implementing it is fine.

---
M src/org/openbravo/copyprocess/process/fileimport/CopyProcessFileColumnHandler.java
---
(0138361)
hgbot   
2022-06-15 10:05   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.copyprocess/-/merge_requests/5 [^]