Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0022922Openbravo ERPB. User interfacepublic2013-01-29 13:072013-02-21 10:17
ReporterPebabr 
Assigned Toshankarb 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS10OS Version2003 Server
Product Version3.0MP18 
Target VersionFixed in Version3.0MP21 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Web browserGoogle Chrome
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0022922: Read-only fields with Text reference are allowing edition.
DescriptionEven if the field gets grayed out, its still allowing the user to choose it and edit.
Steps To ReproducePick a read-only(grayed out) text field.
Click on it.
Edit it.
Save.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
has duplicate defect 00236953.0MP24 closed AugustoMauch Read only does not work in fields associated to columns with reference Text 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-01-29 13:07PebabrNew Issue
2013-01-29 13:07PebabrAssigned To => dbaz
2013-01-29 13:07PebabrWeb browser => Google Chrome
2013-01-29 13:07PebabrModules => Core
2013-01-29 13:07PebabrTriggers an Emergency Pack => No
2013-01-29 13:09dbazAssigned Todbaz => shankarb
2013-02-14 13:19shankarbReview Assigned To => AugustoMauch
2013-02-14 13:19shankarbWeb browserGoogle Chrome => Google Chrome
2013-02-14 13:19shankarbStatusnew => scheduled
2013-02-14 13:19shankarbfix_in_branch => pi
2013-02-14 13:20shankarbIssue Monitored: AugustoMauch
2013-02-14 13:21hgbotCheckin
2013-02-14 13:21hgbotNote Added: 0056441
2013-02-14 13:21hgbotStatusscheduled => resolved
2013-02-14 13:21hgbotResolutionopen => fixed
2013-02-14 13:21hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/61beeb76c7711258e2c2d9eb78b76a22d2c129d1 [^]
2013-02-14 17:10hgbotCheckin
2013-02-14 17:10hgbotNote Added: 0056450
2013-02-14 17:12AugustoMauchNote Added: 0056451
2013-02-14 17:12AugustoMauchStatusresolved => new
2013-02-14 17:12AugustoMauchResolutionfixed => open
2013-02-17 09:00hgbotCheckin
2013-02-17 09:00hgbotNote Added: 0056498
2013-02-17 09:00hgbotStatusnew => resolved
2013-02-17 09:00hgbotResolutionopen => fixed
2013-02-17 09:00hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/61beeb76c7711258e2c2d9eb78b76a22d2c129d1 [^] => http://code.openbravo.com/erp/devel/pi/rev/5d1365a1b222b5fc7b32729b9eba30df3a711977 [^]
2013-02-17 09:04pramakrishnanNote Added: 0056499
2013-02-17 12:04shankarbNote Added: 0056500
2013-02-17 12:04shankarbStatusresolved => new
2013-02-17 12:04shankarbResolutionfixed => open
2013-02-17 12:04shankarbNote Deleted: 0056499
2013-02-17 12:04shankarbNote Added: 0056501
2013-02-17 12:05shankarbStatusnew => scheduled
2013-02-17 12:05shankarbStatusscheduled => resolved
2013-02-17 12:05shankarbResolutionopen => fixed
2013-02-20 04:50hudsonbotCheckin
2013-02-20 04:50hudsonbotNote Added: 0056602
2013-02-20 04:50hudsonbotCheckin
2013-02-20 04:50hudsonbotNote Added: 0056604
2013-02-20 04:50hudsonbotCheckin
2013-02-20 04:50hudsonbotNote Added: 0056612
2013-02-21 10:17AugustoMauchNote Added: 0056671
2013-02-21 10:17AugustoMauchStatusresolved => closed
2013-02-21 10:17AugustoMauchFixed in Version => 3.0MP21
2013-04-30 12:42jonalegriaesarteRelationship addedhas duplicate 0023695

Notes
(0056441)
hgbot   
2013-02-14 13:21   
Repository: erp/devel/pi
Changeset: 61beeb76c7711258e2c2d9eb78b76a22d2c129d1
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Thu Feb 14 17:48:27 2013 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/61beeb76c7711258e2c2d9eb78b76a22d2c129d1 [^]

Fixes Issue 0022922: Read-only fields with Text reference are allowing edition.

Set canEdit property to false to allow selection through click event.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-textarea.js
---
(0056450)
hgbot   
2013-02-14 17:10   
Repository: erp/devel/pi
Changeset: c518d176b5b5571195b1afe75139a37eb0e99d91
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Thu Feb 14 17:09:23 2013 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/c518d176b5b5571195b1afe75139a37eb0e99d91 [^]

Related to issue 22922: Backout of wrong fix

The changeset https://code.openbravo.com/erp/devel/pi/rev/61beeb76c771 [^] has been backed out because it made all text fields read only.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-textarea.js
---
(0056451)
AugustoMauch   
2013-02-14 17:12   
This issue has been reopened and backed out, because it made all text fields read only.
(0056498)
hgbot   
2013-02-17 09:00   
Repository: erp/devel/pi
Changeset: 5d1365a1b222b5fc7b32729b9eba30df3a711977
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Sun Feb 17 13:25:10 2013 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/5d1365a1b222b5fc7b32729b9eba30df3a711977 [^]

Fixes Issue 0022922: Read-only fields with Text reference are allowing edition.

Set canEdit property to false in case it is a read only text area field and true otherwise.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-textarea.js
---
(0056500)
shankarb   
2013-02-17 12:04   
Reopened to modify Test Plan
(0056501)
shankarb   
2013-02-17 12:04   
Test Plan:
1. Login as System Administrator.
2. In Windows and Tabs, go to Message Window -> Translation Tab.
3. In the fields tab, set the field 'hint' as read only.
4. Logout and Login.
5. Go to Message Window and add a new record to Translation tab.
6. Notice that Message text is editable, but hint is not editable or allows focus.
(0056602)
hudsonbot   
2013-02-20 04:50   
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/d828167a9e83 [^]

Maturity status: Test
(0056604)
hudsonbot   
2013-02-20 04:50   
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/d828167a9e83 [^]

Maturity status: Test
(0056612)
hudsonbot   
2013-02-20 04:50   
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/d828167a9e83 [^]

Maturity status: Test
(0056671)
AugustoMauch   
2013-02-21 10:17   
Code reviewed and verified in pi@9700d819690d