Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0019741Openbravo ERPA. Platformpublic2012-02-13 13:272013-08-16 20:31
Reporteriperdomo 
Assigned Toalostale 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS30OS Version3.2.1-2-ARCH
Product Versionpi 
Target VersionFixed in Versionpi 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browserGoogle Chrome
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0019741: Wrong behavior when trying to edit part of the contents in a text field
DescriptionThe user is not able to edit part of a text field. When selecting part of the text box contents, after the first keypress, the whole content of the field gets selected, and the user overwrites everything.
Steps To Reproduce* Using the System Administrator role
* Go to Tables and Columns
* Pick any Table
* Open any Column in form view
* Select part of the contents in the "DB Column Name" field
* Make more than 2 keystrokes
* You'll find that after the first keystroke the whole contents get selected, and overwritten with the content with the second (and following) keys

See the attached video
Proposed Solution
Additional Information
TagsRegression QA
Relationships
related to defect 00195583.0MP10 closed alostale The automatic sequence does not work in projects 
related to design defect 0019844 closed dbaz Text field in display/readonly logic expressions are difficult to edit 
Attached Files? out.ogv (4,450,346) 2012-02-13 13:27
https://issues.openbravo.com/file_download.php?file_id=4992&type=bug
Issue History
Date ModifiedUsernameFieldChange
2012-02-13 13:27iperdomoNew Issue
2012-02-13 13:27iperdomoAssigned To => alostale
2012-02-13 13:27iperdomoFile Added: out.ogv
2012-02-13 13:27iperdomoWeb browser => Google Chrome
2012-02-13 13:27iperdomoModules => Core
2012-02-14 10:55alostaleRelationship addedrelated to 0019558
2012-02-14 10:55alostaleTag Attached: Regression QA
2012-02-14 11:18alostaleNote Added: 0045086
2012-02-14 13:22alostaleNote Added: 0045103
2012-02-14 16:10alostaleNote Added: 0045114
2012-02-22 16:54alostaleRelationship addedrelated to 0019844
2012-02-22 18:10hgbotCheckin
2012-02-22 18:10hgbotNote Added: 0045382
2012-02-22 18:10hgbotStatusnew => resolved
2012-02-22 18:10hgbotResolutionopen => fixed
2012-02-22 18:10hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/c7f713708ab9dfd40da31aa5842ec1857984357e [^]
2012-02-22 18:12alostaleNote Added: 0045383
2012-02-23 20:11guilleaerNote Added: 0045424
2012-02-23 20:12guilleaerNote Added: 0045425
2012-02-23 20:12guilleaerStatusresolved => closed
2012-02-23 20:12guilleaerFixed in Version => pi
2012-03-02 15:38hudsonbotCheckin
2012-03-02 15:38hudsonbotNote Added: 0045926
2013-08-02 21:43hgbotCheckin
2013-08-02 21:43hgbotNote Added: 0060484
2013-08-16 20:31hudsonbotCheckin
2013-08-16 20:31hudsonbotNote Added: 0060616

Notes
(0045086)
alostale   
2012-02-14 11:18   
This bug was revealed by fix for 0019558

Even fix for 0019558 is correct, it makes this issue to arise. The problem is in fields of String reference participating in a dynamic expression (read only logic or display logic), in this case whenever the field changes it is marked for redraw forcing it to be redrawn and in this event the existent text is selected.
(0045103)
alostale   
2012-02-14 13:22   
The problem is caused by selectOnFocus, when redraw is done focus is set back causing it to select the whole text.
(0045114)
alostale   
2012-02-14 16:10   
See http://forums.smartclient.com/showthread.php?t=20913 [^]
(0045382)
hgbot   
2012-02-22 18:10   
Repository: erp/devel/pi
Changeset: c7f713708ab9dfd40da31aa5842ec1857984357e
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Wed Feb 22 18:09:54 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/c7f713708ab9dfd40da31aa5842ec1857984357e [^]

fixed bug 19741, related to issue 19558, related to issue 19844

  Reverting display logic in automatic sequence field added by 0019558
  because it was causing 0019741. This display logic should be set back
  once 0019844 is fixed.

---
M src-db/database/sourcedata/AD_FIELD.xml
---
(0045383)
alostale   
2012-02-22 18:12   
See notes in the commit message.

No regression risk.
(0045424)
guilleaer   
2012-02-23 20:11   
code review and tested on pi@4940d1f96b76
(0045425)
guilleaer   
2012-02-23 20:12   
code review and tested on pi@cbf12eaccb56
(0045926)
hudsonbot   
2012-03-02 15:38   
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/544d64e0c159 [^]

Maturity status: Test
(0060484)
hgbot   
2013-08-02 21:43   
Repository: erp/devel/pi
Changeset: aae683fdb69eaf0a28b472f5b34ec58225ad2461
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Fri Aug 02 21:43:09 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/aae683fdb69eaf0a28b472f5b34ec58225ad2461 [^]

Related to issue 19741, related to issue 19558, related to issue 19844

Added again display logic due that with Smartclient 2012-05-26
issue 19844 is fixed

---
M src-db/database/sourcedata/AD_FIELD.xml
---
(0060616)
hudsonbot   
2013-08-16 20:31   
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/59a1180e7f4f [^]

Maturity status: Test