Anonymous | Login

Project:

News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ]

[ Issue History ] [ Print ]

ID

0050308

Type

Category

Severity

Reproducibility

Date Submitted

Last Update

defect

[Modules] Magento Connector - Mappings

minor

have not tried

2022-09-22 10:36

2022-09-22 11:06

Reporter

caristu

View Status

public

Assigned To

caristu

Priority

high

Resolution

fixed

Fixed in Version

Status

closed

Fix in branch

Fixed in SCM revision

Projection

none

ETA

none

Target Version

OS

Any

Database

Any

Java version

OS Version

Database version

Ant version

Product Version

SCM revision

Regression date

Regression introduced by commit

Regression level

Review Assigned To

Regression introduced in release

Summary

0050308: Saving characteristic with DAL fails if not in admin mode

Description

Having the Magento Connector Mappings module installed is not possible to save a Characteristic with DAL being not in admin mode.

In that case, the following exception is thrown:

org.openbravo.base.exception.OBSecurityException: Entity ADList is not readable by the user A530AAE22C864702B7E1C22D58E7B17B
    at org.openbravo.dal.security.EntityAccessChecker.checkReadable(EntityAccessChecker.java:579)
    at org.openbravo.dal.service.OBDal.checkReadAccess(OBDal.java:741)
    at org.openbravo.dal.service.OBDal.checkReadAccess(OBDal.java:728)
    at org.openbravo.dal.service.OBDal.createQuery(OBDal.java:485)
    at org.openbravo.dal.service.OBDal.createQuery(OBDal.java:442)
    at org.openbravo.service.integration.magento.mappings.utils.MagentoUtils.isSystemCharacteristic(MagentoUtils.java:58)
    at org.openbravo.service.integration.magento.mappings.outbound.characteristics.CharacteristicEventHandler.onSave(CharacteristicEventHandler.java:46)
...

Steps To Reproduce

0) Install the Magento Connector Mappings module
1) Execute any of these test cases:

- org.openbravo.event.CharacteristicEventHandlerTest
- org.openbravo.client.kernel.reference.plm.RelevantCharacteristicQueryHookTest

Note that they fail with the mentioned error.

Proposed Solution

MagentoUtils.isSystemCharacteristic should execute its query to the ADList entity in admin mode

Tags

No tags attached.

Relationships [ Relation Graph ] [ Dependency Graph ]

Notes
(0141297) hgbot (developer) 2022-09-22 10:51	Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.integration.magento.mappings/-/merge_requests/7 [^]

(0141299) hgbot (developer) 2022-09-22 11:06	Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.integration.magento.mappings [^] Changeset: 41eb1f8b964644a662aa6d0166aebc0e99f0a29c Author: Carlos Aristu <carlos.aristu@openbravo.com> Date: 22-09-2022 10:48:38 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.integration.magento.mappings/-/commit/41eb1f8b964644a662aa6d0166aebc0e99f0a29c [^] fixes BUG-50308: use admin mode in isSystemCharacteristic method --- M src/org/openbravo/service/integration/magento/mappings/utils/MagentoUtils.java ---

(0141300) hgbot (developer) 2022-09-22 11:06	Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.service.integration.magento.mappings/-/merge_requests/7 [^]

Issue History
Date Modified	Username	Field	Change
2022-09-22 10:36	caristu	New Issue
2022-09-22 10:36	caristu	Assigned To	=> caristu
2022-09-22 10:51	hgbot	Note Added: 0141297
2022-09-22 11:06	hgbot	Resolution	open => fixed
2022-09-22 11:06	hgbot	Status	new => closed
2022-09-22 11:06	hgbot	Note Added: 0141299
2022-09-22 11:06	hgbot	Note Added: 0141300

Copyright © 2000 - 2009 MantisBT Group