Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0055021 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 02. Master data management | minor | always | 2024-03-20 20:57 | 2024-05-13 08:56 | |||
| Reporter | sdossantos | View Status | public | |||||
| Assigned To | gonzalo_fernandez | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | PR24Q3 | |||
| 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 | OBPS | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | 92349 | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0055021: Error with computed image column | |||||||
| Description | If I create a computed image column, an error is generated. When a modification is performed, the RemoveImagesEventHandler event is triggered and it tries to recover a property that does not exist in the database, returning an error. The problem is that the ModelProvider is uploading all the ADImage properties without taking into account whether they are computed or not. | |||||||
| Steps To Reproduce | 1- Create a computed column in c_orderline to see the product image. Example: (select m_product.ad_image_id from m_product where m_product.m_product_id = M_product_ID) Use it creating a field for the Sales Order window and opening it (you would see the error). | |||||||
| Proposed Solution | --- a/src/org/openbravo/base/model/ModelProvider.java 2022-03-01 14:20:59.000000000 +0100 +++ b/src/org/openbravo/base/model/ModelProvider.java 2024-03-20 11:53:19.183266000 +0100 @@ -338,8 +338,10 @@ if (p.getReferencedProperty() != null) { Entity referencedEntity = p.getReferencedProperty().getEntity(); if ("ADImage".equals(referencedEntity.getName())) { - entitiesWithImage.computeIfAbsent(p.getEntity(), k -> new ArrayList<>()) - .add(p.getName()); + if (!p.isComputedColumn()) { + entitiesWithImage.computeIfAbsent(p.getEntity(), k -> new ArrayList<>()) + .add(p.getName()); + } } else if ("OBPRF_FILE".equals(referencedEntity.getName())) { entitiesWithFile.computeIfAbsent(p.getEntity(), k -> new ArrayList<>()) .add(p.getName()); | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0163008) AugustoMauch (administrator) 2024-04-10 09:15 |
Hello, could you rephrase a little the steps to reproduce? "And use it - you can see the error" -> I would like to know what do you mean by "use it", and it would be useful if you could paste the error message |
|
(0163658) hgbot (developer) 2024-04-24 13:38 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/1214 [^] |
|
(0164466) hgbot (developer) 2024-05-13 08:56 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: f0236164ac15b8eceff1f9a7aa11a41b03735827 Author: Gonzalo Fernández García <gonzalo.fernandez@openbravo.com> Date: 13-05-2024 06:55:41 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/f0236164ac15b8eceff1f9a7aa11a41b03735827 [^] Fixes ISSUE-55021: Fixes error when working with computed image columns --- M src/org/openbravo/base/model/ModelProvider.java --- |
|
(0164467) hgbot (developer) 2024-05-13 08:56 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/1214 [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2024-03-20 20:57 | sdossantos | New Issue | |
| 2024-03-20 20:57 | sdossantos | Assigned To | => Triage Omni OMS |
| 2024-03-20 20:57 | sdossantos | OBNetwork customer | => OBPS |
| 2024-03-20 20:57 | sdossantos | Modules | => Core |
| 2024-03-20 20:57 | sdossantos | Support ticket | => 92349 |
| 2024-03-20 20:57 | sdossantos | Triggers an Emergency Pack | => No |
| 2024-03-20 21:03 | Practics | Issue Monitored: Practics | |
| 2024-03-21 10:56 | aferraz | Assigned To | Triage Omni OMS => Triage Platform Base |
| 2024-04-10 09:15 | AugustoMauch | Note Added: 0163008 | |
| 2024-04-10 09:15 | AugustoMauch | Status | new => feedback |
| 2024-04-24 09:58 | AugustoMauch | Assigned To | Triage Platform Base => gonzalo_fernandez |
| 2024-04-24 13:38 | hgbot | Merge Request Status | => open |
| 2024-04-24 13:38 | hgbot | Note Added: 0163658 | |
| 2024-04-24 14:04 | gonzalo_fernandez | Status | feedback => scheduled |
| 2024-04-24 14:04 | gonzalo_fernandez | Summary | Error with computed image field => Error with computed image column |
| 2024-04-24 14:04 | gonzalo_fernandez | Description Updated | View Revisions |
| 2024-04-24 14:04 | gonzalo_fernandez | Steps to Reproduce Updated | View Revisions |
| 2024-05-13 08:55 | hgbot | Merge Request Status | open => approved |
| 2024-05-13 08:56 | hgbot | Resolution | open => fixed |
| 2024-05-13 08:56 | hgbot | Status | scheduled => closed |
| 2024-05-13 08:56 | hgbot | Fixed in Version | => PR24Q3 |
| 2024-05-13 08:56 | hgbot | Note Added: 0164466 | |
| 2024-05-13 08:56 | hgbot | Note Added: 0164467 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |