Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035106Openbravo ERPB. User interfacepublic2017-01-31 13:352017-03-15 20:19
Reporteregoitz 
Assigned Tocaristu 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR17Q2 
Merge Request Status
Review Assigned Toalostale
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket45220
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035106: The prompt text is not shown on the multiple selector grids
DescriptionWhen the mouse is moved over the grid a prompt text (tip text) is shown with the text of the cell.
That does not work on the grid of the multiple selector.

Steps To Reproduce1- Install an use the module
https://code.openbravo.com/erp/mods/org.openbravo.platform.features/ [^]

2- Open the 'Product Simple Report'

3- Click on the magnifying-glass of the product category selector in order to show the selector pop-up

4- Move the mouse over the cells of the selector grid and see that the tip text is now shown
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0035215 closed caristu The showHover property is not configured properly for some UI references 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-01-31 13:35egoitzNew Issue
2017-01-31 13:35egoitzAssigned To => platform
2017-01-31 13:35egoitzOBNetwork customer => Yes
2017-01-31 13:35egoitzModules => Core
2017-01-31 13:35egoitzSupport ticket => 45220
2017-01-31 13:35egoitzTriggers an Emergency Pack => No
2017-02-01 08:41caristuSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=14410#r14410
2017-02-01 09:17caristuSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=14411#r14411
2017-02-01 09:18caristuSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=14412#r14412
2017-02-01 09:58caristuAssigned Toplatform => caristu
2017-02-01 09:58caristuStatusnew => scheduled
2017-02-01 16:03caristuNote Added: 0093990
2017-02-09 17:41hgbotCheckin
2017-02-09 17:41hgbotNote Added: 0094186
2017-02-09 17:41hgbotStatusscheduled => resolved
2017-02-09 17:41hgbotResolutionopen => fixed
2017-02-09 17:41hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/cd0eba2a0539519a2bfbbf8f48bb498bfd148d64 [^]
2017-02-09 18:02caristuRelationship addedrelated to 0035215
2017-02-10 08:06alostaleRelationship replacedblocks 0035215
2017-02-10 08:25caristuReview Assigned To => alostale
2017-02-10 08:46hgbotCheckin
2017-02-10 08:46hgbotNote Added: 0094205
2017-02-10 08:47alostaleNote Added: 0094206
2017-02-10 08:47alostaleStatusresolved => closed
2017-02-10 08:47alostaleFixed in Version => 3.0PR17Q2
2017-03-15 20:19hudsonbotCheckin
2017-03-15 20:19hudsonbotNote Added: 0095173
2017-03-15 20:19hudsonbotCheckin
2017-03-15 20:19hudsonbotNote Added: 0095175

Notes
(0093990)
caristu   
2017-02-01 16:03   
There are two problems:

- The prompt text is not shown in selector/multiselector
- In standard grids, there are some type of fields whose prompt is not displayed

Here is the summary: https://docs.google.com/spreadsheets/d/18QRAxwfp-if919RbGiKz4sv6mmQyxg8kpJV9Z2XpRag/edit#gid=0 [^]
(0094186)
hgbot   
2017-02-09 17:41   
Repository: erp/devel/pi
Changeset: cd0eba2a0539519a2bfbbf8f48bb498bfd148d64
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Thu Feb 09 17:40:55 2017 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/cd0eba2a0539519a2bfbbf8f48bb498bfd148d64 [^]

fixes issue 35106: The prompt text is not shown on the multiple selector grids

The selector component was not configuring the showHover property. Now it is configured in the selector fields by checking the UIDefinition of each one. The UIDefinition has a new showHover method which defines if a reference supports hover. It returns true by default but it can be overriden by any subclass in case hover texts are not desired, like in the case of PasswordUIDefinition.

Together with this, the prompt for the checkbox fields of the multi-selector grid has been enabled.

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/PasswordUIDefinition.java
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/UIDefinition.java
M modules/org.openbravo.userinterface.selector/src/org/openbravo/userinterface/selector/SelectorComponent.java
M modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-multi-selector-item.js
---
(0094205)
hgbot   
2017-02-10 08:46   
Repository: erp/devel/pi
Changeset: 8453cd54ca4ff7969fdaa7f620d3cdd3780c9816
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Fri Feb 10 08:45:30 2017 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/8453cd54ca4ff7969fdaa7f620d3cdd3780c9816 [^]

related to issue 35106: added missing javadoc for showHover method

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/UIDefinition.java
---
(0094206)
alostale   
2017-02-10 08:47   
code reviewed + tested
(0095173)
hudsonbot   
2017-03-15 20:19   
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/54e102bef53e [^]
Maturity status: Test
(0095175)
hudsonbot   
2017-03-15 20:19   
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/54e102bef53e [^]
Maturity status: Test