Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0045039Retail ModulesWeb POSpublic2020-09-14 10:012020-09-17 12:41
Reportergorka_gil 
Assigned Togorka_gil 
PriorityurgentSeverityminorReproducibilityN/A
StatusclosedResolutionno change required 
PlatformOS5OS Version
Product Version 
Target VersionRR20Q4Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0045039: OB.App.DAL.remoteSearch and OB.App.DAL.remoteGet shouldn't be in OB.App
DescriptionOB.App is the new code and is moved to the new pos.

But these two functions are wrappers of old API (OB.Dal.find and OB.Dal.get) that are not present in the new code.

This code should not be in /app or in OB.App
Steps To Reproduce-
Proposed SolutionMove to OB.Dal u other place, but out of /app and OB.App
Additional Information
TagsNo tags attached.
Relationships
depends on backport 0045046RR20Q3.1 closed gorka_gil OB.App.DAL.remoteSearch and OB.App.DAL.remoteGet shouldn't be in OB.App 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2020-09-14 10:01gorka_gilNew Issue
2020-09-14 10:01gorka_gilAssigned To => gorka_gil
2020-09-14 10:01gorka_gilTriggers an Emergency Pack => No
2020-09-14 19:14gorka_gilStatusnew => scheduled
2020-09-17 12:41gorka_gilNote Added: 0123150
2020-09-17 12:41gorka_gilStatusscheduled => closed
2020-09-17 12:41gorka_gilResolutionopen => no change required

Notes
(0123150)
gorka_gil   
2020-09-17 12:41   
Rejecting the issue since this is a wrapper that its implementation will be different in old pos and the new one.

In old code will call to ob.dal.find, but in new pos its implementation will change.

So it is consistent with the idea of not copy old code to the new pos.