Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0029561Openbravo ERPB. User interfacepublic2015-04-10 13:162015-05-16 05:21
Reportercaristu 
Assigned Todbaz 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR15Q3 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0029561: Time Fields can't be set as read only
DescriptionTime Fields can't be set as read only
Steps To Reproduce1) Create a new column with time reference
2) Create a field for column in 1). Set it as read only
3) Recompile and restart tomcat
4) Access to the window with the new field. Notice that it is possible to write characters on the field.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00276703.0PR15Q1 closed alostale Time fields do not have the correct style in read only mode 
related to defect 0029878 acknowledged Triage Platform Base blur from time field detects a change 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-04-10 13:16caristuNew Issue
2015-04-10 13:16caristuAssigned To => platform
2015-04-10 13:16caristuModules => Core
2015-04-10 13:16caristuTriggers an Emergency Pack => No
2015-04-10 13:16caristuRelationship addedrelated to 0027670
2015-04-10 13:17caristuIssue Monitored: networkb
2015-04-17 08:35alostalePriorityhigh => urgent
2015-04-17 08:35alostaleStatusnew => acknowledged
2015-05-12 18:24egoitzPriorityurgent => immediate
2015-05-12 18:24egoitzSeverityminor => major
2015-05-12 19:07egoitzIssue Monitored: egoitz
2015-05-13 08:16alostaleStatusacknowledged => scheduled
2015-05-13 08:16alostaleAssigned Toplatform => dbaz
2015-05-13 15:53dbazReview Assigned To => alostale
2015-05-13 15:56hgbotCheckin
2015-05-13 15:56hgbotNote Added: 0077387
2015-05-13 15:56hgbotStatusscheduled => resolved
2015-05-13 15:56hgbotResolutionopen => fixed
2015-05-13 15:56hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/8fc80e4bb1fb17bbddc541a782f873756089e704 [^]
2015-05-14 10:04alostaleRelationship addedrelated to 0029878
2015-05-14 10:07alostaleNote Added: 0077424
2015-05-14 10:07alostaleStatusresolved => new
2015-05-14 10:07alostaleResolutionfixed => open
2015-05-14 12:15hgbotCheckin
2015-05-14 12:15hgbotNote Added: 0077436
2015-05-14 12:15hgbotStatusnew => resolved
2015-05-14 12:15hgbotResolutionopen => fixed
2015-05-14 12:15hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/8fc80e4bb1fb17bbddc541a782f873756089e704 [^] => http://code.openbravo.com/erp/devel/pi/rev/5a7da4d16d1d04f1b75649bcdcc91d835a39ccb4 [^]
2015-05-14 16:44alostaleNote Added: 0077453
2015-05-14 16:44alostaleStatusresolved => closed
2015-05-14 16:44alostaleFixed in Version => 3.0PR15Q3
2015-05-14 17:48hudsonbotCheckin
2015-05-14 17:48hudsonbotNote Added: 0077457
2015-05-16 05:21hudsonbotCheckin
2015-05-16 05:21hudsonbotNote Added: 0077506

Notes
(0077387)
hgbot   
2015-05-13 15:56   
Repository: erp/devel/pi
Changeset: 8fc80e4bb1fb17bbddc541a782f873756089e704
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Wed May 13 15:56:05 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/8fc80e4bb1fb17bbddc541a782f873756089e704 [^]

Fixed issue 29561: 'Time' fields now can be set as readonly via AD_FIELD

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-time.js
---
(0077424)
alostale   
2015-05-14 10:07   
There is still a case where the behavior is not correct: no uneatable column with editable field. In this case the expected behavior is the field to be editable only when creating a new record but no editable afterwards. In this case it behaves correctly in terms of styling but just after saving it is possible to edit the value though the style looks like read only even after saving the record the value is restored to the original one.

See all cases that have been tested: https://docs.google.com/spreadsheets/d/1fi4VRn4uhlwMcMYs6ZWFRdNi085zQDl-zaYvcuv2cDY/edit#gid=0 [^]
(0077436)
hgbot   
2015-05-14 12:15   
Repository: erp/devel/pi
Changeset: 5a7da4d16d1d04f1b75649bcdcc91d835a39ccb4
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Thu May 14 12:15:15 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/5a7da4d16d1d04f1b75649bcdcc91d835a39ccb4 [^]

Fixed issue 29561: 'Time' fields now can be set as no updateable via AD_COLUMN

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-time.js
---
(0077453)
alostale   
2015-05-14 16:44   
code reviewed

tested for the cases defined in this sheet: https://docs.google.com/spreadsheets/d/1fi4VRn4uhlwMcMYs6ZWFRdNi085zQDl-zaYvcuv2cDY/edit#gid=0 [^]
(0077457)
hudsonbot   
2015-05-14 17:48   
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/3375171dd310 [^]
Maturity status: Test
(0077506)
hudsonbot   
2015-05-16 05:21   
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/33b0ebaa36ae [^]
Maturity status: Test