Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0019463 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] A. Platform | major | always | 2012-01-11 16:22 | 2012-02-01 08:58 | |||
| Reporter | jecharri | View Status | public | |||||
| Assigned To | jecharri | |||||||
| Priority | high | Resolution | fixed | Fixed in Version | 3.0MP8 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | 1c7cc06530d5 | ||||
| Projection | none | ETA | none | Target Version | 3.0MP8 | |||
| OS | Linux 32 bit | Database | PostgreSQL | Java version | ||||
| OS Version | Community Appliance | Database version | Ant version | |||||
| Product Version | pi | SCM revision | ||||||
| Review Assigned To | ||||||||
| Web browser | Google Chrome | |||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0019463: Read Only Logic for columns is applied incorrectly | |||||||
| Description | Currently, there are several ways to make some tabs/fields read only: - A role may have access but not write permission to a window. - A window can be of type "query" - A tab can have a UI Pattern of Read Only - A single field within a Tab can be Read Only - A column can be non updatable (meaning that it can only be edited when creating a record) - A column can have a Read Only Logic that will make the field read only dynamically, similar to Display Logics, but at a column level. Read Only Logic should only be validated / applied if the previous conditions are not met, e.g. the column should be already editable. Currently, Read Only Logic can make editable some fields which are not designed to be. | |||||||
| Steps To Reproduce | 1. Give an editable column a read only logic of 'false'. This means it should always be editable. 2. Make read only a field for this column 3. The field is editable even if it has been set to read only in this tab, since it is following the Read Only Logic rule instead of its field AD information. | |||||||
| Proposed Solution | Revise the whole read only functionality | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0044397) hgbot (developer) 2012-01-16 13:20 |
Repository: erp/devel/pi Changeset: 1c7cc06530d5faf57bf0aaafa43332d6c8293063 Author: Javier Etxarri <javier.echarri <at> openbravo.com> Date: Mon Jan 16 13:10:38 2012 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/1c7cc06530d5faf57bf0aaafa43332d6c8293063 [^] Fixes issue 17395, Fixes issue 19463: Read Only Logic for columns is applied incorrectly. Now the readonlylogic expression is not going to be applied until the application check if the field is defined as read only --- M modules/org.openbravo.client.application/src/org/openbravo/client/application/window/OBViewFieldHandler.java --- |
|
(0044432) hudsonbot (developer) 2012-01-18 14:22 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/0035a0985219 [^] Maturity status: Test |
|
(0044716) hgbot (developer) 2012-01-30 18:14 |
Repository: erp/devel/pi Changeset: a052b9bb86e5a76902755ba6c08da5fc02550923 Author: Javier Etxarri <javier.echarri <at> openbravo.com> Date: Mon Jan 30 18:13:28 2012 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/a052b9bb86e5a76902755ba6c08da5fc02550923 [^] issue 17395, issue 19463: Read Only Logic for columns is applied incorrectly. Now the read only logic is applied correctly for fields without column related (OBClientClassField) --- M modules/org.openbravo.client.application/src/org/openbravo/client/application/window/OBViewFieldHandler.java --- |
|
(0044721) AugustoMauch (administrator) 2012-01-31 09:16 |
Code reviewed and fix verified |
|
(0044752) hudsonbot (developer) 2012-02-01 08:58 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/933a9b7b9d89 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2012-01-11 16:22 | jecharri | New Issue | |
| 2012-01-11 16:22 | jecharri | Assigned To | => jecharri |
| 2012-01-11 16:22 | jecharri | Web browser | => Google Chrome |
| 2012-01-11 16:22 | jecharri | Modules | => Core |
| 2012-01-11 16:22 | jecharri | Issue generated from | 0017395 |
| 2012-01-11 16:22 | jecharri | Relationship added | related to 0017395 |
| 2012-01-11 16:33 | maria_perez | Issue Monitored: maria_perez | |
| 2012-01-16 13:20 | hgbot | Checkin | |
| 2012-01-16 13:20 | hgbot | Note Added: 0044397 | |
| 2012-01-16 13:20 | hgbot | Status | new => resolved |
| 2012-01-16 13:20 | hgbot | Resolution | open => fixed |
| 2012-01-16 13:20 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/1c7cc06530d5faf57bf0aaafa43332d6c8293063 [^] |
| 2012-01-18 14:22 | hudsonbot | Checkin | |
| 2012-01-18 14:22 | hudsonbot | Note Added: 0044432 | |
| 2012-01-30 18:14 | hgbot | Checkin | |
| 2012-01-30 18:14 | hgbot | Note Added: 0044716 | |
| 2012-01-31 09:16 | AugustoMauch | Note Added: 0044721 | |
| 2012-01-31 09:16 | AugustoMauch | Status | resolved => closed |
| 2012-01-31 09:16 | AugustoMauch | Fixed in Version | => 3.0MP8 |
| 2012-02-01 08:58 | hudsonbot | Checkin | |
| 2012-02-01 08:58 | hudsonbot | Note Added: 0044752 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |