Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0049500 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] A. Platform | major | always | 2022-06-07 17:42 | 2022-07-12 09:03 | |||
| Reporter | vmromanos | View Status | public | |||||
| Assigned To | ablasco | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | PR22Q4 | |||
| 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 | |||||||
| Merge Request Status | approved | |||||||
| Review Assigned To | ||||||||
| OBNetwork customer | No | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0049500: Preference visibility in other clients | |||||||
| Description | In an instance with several clients, the preferences defined in one client can affected other clients. | |||||||
| Steps To Reproduce | Use any instance with several clients, like https://livebuilds.openbravo.com/retail_modules_oracle_pi [^] Login in backoffice and go to WhiteValley client Create the following preference: Client: The White Valley Group Org: * Property List: Y Property: Allow Goods Transaction outside AWO for Inventories Value: Y Leave empty all the fields in Visibility field group Information: with this preference we are allowing to manually process physical inventories outside AWO. Now change the role to AWO-QA Admin (in AWO client, which is different from White Valley) Go to Physical Inventory and create a new record: Org: US West Coast Warehouse: US West Coast Enter a new line: Product: ISSalesAWOTest0001 Bin: ISSalesAWOTest0001 Quantity Count: 99 Process Inventory Count. Verify the record is processed successfully, however an exception should be thrown instead! Go back to WhiteValley client and remove the previous preference. Go back to AWO-QA client and create the same Physical Inventory, but setting Quantity Count: 100. Process Inventory Count. Verify an exception is thrown. This is the expected behavior. So the preference in the WhiteValley client is impacting the AWO-QA client. | |||||||
| Proposed Solution | The problem is with the Visible At Client field. When you leave it empty in the preference definition, the AD_GET_PREFERENCE_VALUE is bypassing this field: and (visibleat_client_id = v_ad_client_id or coalesce(visibleat_client_id, '0') = '0') I honestly think this Visible At Client field has no sense and it should be removed. Instead we should do the following logic: * You are able to see the preferences defined at your client AND the ones defined at System level. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||
|
|||||||||
 Relationships |
|
Notes |
|
|
(0138126) hgbot (developer) 2022-06-07 19:09 |
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations.api [^] Changeset: dea4408d031cf92ad4698eef69a63e8b5419d8ec Author: Víctor Martínez Romanos <victor.martinez@openbravo.com> Date: 07-06-2022 18:58:59 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations.api/-/commit/dea4408d031cf92ad4698eef69a63e8b5419d8ec [^] Related to ISSUE-49500: Force to set the visible at client to workaround issue --- M src/org/openbravo/warehouse/advancedwarehouseoperations/api/hooks/AWOAPIHookUtils.java --- |
|
(0138127) hgbot (developer) 2022-06-07 19:09 |
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations [^] Changeset: 99dda422c1c5a94be5444e095456eb2ac84d4c90 Author: Víctor Martínez Romanos <victor.martinez@openbravo.com> Date: 07-06-2022 19:09:41 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.warehouse.advancedwarehouseoperations/-/commit/99dda422c1c5a94be5444e095456eb2ac84d4c90 [^] Related to ISSUE-49500: enable tests again as there is a workaround in org.openbravo.warehouse.advancedwarehouseoperations.api that sets the visible at client property when creating the preference --- M src-test/org/openbravo/warehouse/advancedwarehouseoperations/test/AWOb_BasicFunctionalFlowsWithAWOPreference.java --- |
|
(0138781) hgbot (developer) 2022-06-27 12:19 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/630 [^] |
|
(0139260) hgbot (developer) 2022-07-12 09:03 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: 6ddeb15c5cc57bb33d3ff3f55df9a7468b22a97e Author: Adrián Blasco <adrian.blasco@openbravo.com> Date: 12-07-2022 07:03:11 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/6ddeb15c5cc57bb33d3ff3f55df9a7468b22a97e [^] Fixes ISSUE-49500: Update preference visibility based on ad_user_id on several preference xml files - This also updates Preferences.java to include use preference visibility at client level --- M src-db/database/model/functions/AD_GET_PREFERENCE_VALUE.xml M src-db/database/model/functions/C_BP_SOCREDITUSED_REFRESH.xml M src-db/database/model/functions/C_GET_PRODUCT_TAXCATEGORY.xml M src-db/database/model/functions/C_INVOICE_POST.xml M src-db/database/model/functions/C_ORDER_POST1.xml M src-db/database/model/functions/C_TAXPAYMENT_POST.xml M src-db/database/model/functions/M_INOUT_CREATE.xml M src-db/database/model/functions/M_PRICELIST_CREATE.xml M src-db/database/model/triggers/M_RESERVATION_TRG.xml M src/org/openbravo/erpCommon/businessUtility/Preferences.java --- |
|
(0139261) hgbot (developer) 2022-07-12 09:03 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/630 [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2022-06-07 17:42 | vmromanos | New Issue | |
| 2022-06-07 17:42 | vmromanos | Assigned To | => Triage Platform Base |
| 2022-06-07 17:42 | vmromanos | OBNetwork customer | => No |
| 2022-06-07 17:42 | vmromanos | Modules | => Core |
| 2022-06-07 17:42 | vmromanos | Triggers an Emergency Pack | => No |
| 2022-06-07 19:09 | hgbot | Note Added: 0138126 | |
| 2022-06-07 19:09 | hgbot | Note Added: 0138127 | |
| 2022-06-23 22:24 | AugustoMauch | Assigned To | Triage Platform Base => ablasco |
| 2022-06-23 22:24 | AugustoMauch | Status | new => scheduled |
| 2022-06-27 12:19 | hgbot | Merge Request Status | => open |
| 2022-06-27 12:19 | hgbot | Note Added: 0138781 | |
| 2022-07-08 15:11 | cberner | Relationship added | related to 0049768 |
| 2022-07-12 09:00 | hgbot | Merge Request Status | open => approved |
| 2022-07-12 09:03 | hgbot | Resolution | open => fixed |
| 2022-07-12 09:03 | hgbot | Status | scheduled => closed |
| 2022-07-12 09:03 | hgbot | Fixed in Version | => PR22Q4 |
| 2022-07-12 09:03 | hgbot | Note Added: 0139260 | |
| 2022-07-12 09:03 | hgbot | Note Added: 0139261 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |