0030403: Unexpected behavior in grid view with OBUISEL_Selector reference when the Display Field is set

Notice:
- _extraProperties -> 'searchKey'
- gridProps.displayField -> 'businessPartner$searchKey' 
Selector Definition:
init: function() {
        this.optionDataSource =     OB.Datasource.create(
    {
        createClassName: '',
        dataURL:'/charly/org.openbravo.service.datasource/BusinessPartner',
        requestProperties : { params : {
                    targetProperty : 'businessPartner',     
                    adTabId : '186',     
                    IsSelectorItem : 'true',     
                    Constants_FIELDSEPARATOR : '$',     
                    columnName : 'C_BPartner_ID',     
                    Constants_IDENTIFIER : '_identifier',     
                    _extraProperties : 'searchKey,businessPartnerCategory$name,vendor,name,customer'     
                }
        }, fields:[
    {name: 'id', type: '_id_13' ,primaryKey: true },
    {name: 'client', type: '_id_19' } , {name: 'client$_identifier' },
    {name: 'organization', type: '_id_19' } , {name: 'organization$_identifier' },
    {name: 'active', type: '_id_20' },
    {name: 'creationDate', type: '_id_16' },
    {name: 'createdBy', type: '_id_30' } , {name: 'createdBy$_identifier' },
    {name: 'updated', type: '_id_16' },
    {name: 'updatedBy', type: '_id_30' } , {name: 'updatedBy$_identifier' },
    {name: 'searchKey', type: '_id_10' },
    {name: 'name', type: '_id_10' },
    {name: 'name2', type: '_id_10' },
    {name: 'description', type: '_id_10' },
    {name: 'summaryLevel', type: '_id_20' },
    {name: 'businessPartnerCategory', type: '_id_19' } , {name: 'businessPartnerCategory$_identifier' },
    {name: 'oneTimeTransaction', type: '_id_20' },
    {name: 'potentialCustomer', type: '_id_20' },
    {name: 'vendor', type: '_id_20' },
    {name: 'customer', type: '_id_20' },
    {name: 'employee', type: '_id_20' },
    {name: 'isSalesRepresentative', type: '_id_20' },
    {name: 'referenceNo', type: '_id_10' },
    {name: 'dUNS', type: '_id_10' },
    {name: 'uRL', type: '_id_800101' },
    {name: 'language', type: '_id_106' } , {name: 'language$_identifier' },
    {name: 'taxID', type: '_id_10' },
    {name: 'taxExempt', type: '_id_20' },
    {name: 'invoiceSchedule', type: '_id_19' } , {name: 'invoiceSchedule$_identifier' },
    {name: 'valuation', type: '_id_10' },
    {name: 'volumeOfSales', type: '_id_12' },
    {name: 'noOfEmployees', type: '_id_11' },
    {name: 'nAICSSIC', type: '_id_10' },
    {name: 'dateOfFirstSale', type: '_id_15' },
    {name: 'acquisitionCost', type: '_id_12' },
    {name: 'expectedLifetimeRevenue', type: '_id_12' },
    {name: 'lifetimeRevenueToDate', type: '_id_12' },
    {name: 'share', type: '_id_11' },
    {name: 'formOfPayment', type: '_id_195' , valueMap: { '5': 'Bank Deposit', 'R': 'Bank Remittance', 'B': 'Cash', 'C': 'Cash on Delivery', '2': 'Check', 'K': 'Credit Card', '4': 'Money Order', 'P': 'On Credit', '3': 'Promissory Note', '1': 'Wire Transfer', 'W': 'Withholding' } },
    {name: 'creditLimit', type: '_id_12' },
    {name: 'creditUsed', type: '_id_12' },
    {name: 'paymentTerms', type: '_id_19' } , {name: 'paymentTerms$_identifier' },
    {name: 'priceList', type: '_id_166' } , {name: 'priceList$_identifier' },
    {name: 'printDiscount', type: '_id_20' },
    {name: 'orderDescription', type: '_id_10' },
    {name: 'orderReference', type: '_id_10' },
    {name: 'pOFormOfPayment', type: '_id_195' , valueMap: { '5': 'Bank Deposit', 'R': 'Bank Remittance', 'B': 'Cash', 'C': 'Cash on Delivery', '2': 'Check', 'K': 'Credit Card', '4': 'Money Order', 'P': 'On Credit', '3': 'Promissory Note', '1': 'Wire Transfer', 'W': 'Withholding' } },
    {name: 'purchasePricelist', type: '_id_800031' } , {name: 'purchasePricelist$_identifier' },
    {name: 'pOPaymentTerms', type: '_id_227' } , {name: 'pOPaymentTerms$_identifier' },
    {name: 'numberOfCopies', type: '_id_11' },
    {name: 'greeting', type: '_id_19' } , {name: 'greeting$_identifier' },
    {name: 'invoiceTerms', type: '_id_150' , valueMap: { 'D': 'After Delivery', 'O': 'After Order Delivered', 'S': 'Customer Schedule After Delivery', 'N': 'Do Not Invoice', 'I': 'Immediate' } },
    {name: 'deliveryTerms', type: '_id_151' , valueMap: { 'R': 'After Receipt', 'A': 'Availability', 'L': 'Complete Line', 'O': 'Complete Order' } },
    {name: 'deliveryMethod', type: '_id_152' , valueMap: { 'D': 'Delivery', 'P': 'Pickup', 'S': 'Shipper' } },
    {name: 'salesRepresentative', type: '_id_187' } , {name: 'salesRepresentative$_identifier' },
    {name: 'partnerParent', type: '_id_13' },
    {name: 'creditStatus', type: '_id_289' , valueMap: { 'H': 'Credit Hold', 'O': 'Credit OK', 'S': 'Credit Stop', 'W': 'Credit Watch', 'X': 'No Credit Check' } },
    {name: 'forcedOrg', type: '_id_276' } , {name: 'forcedOrg$_identifier' },
    {name: 'pricesShownInOrder', type: '_id_20' },
    {name: 'invoiceGrouping', type: '_id_800026' , valueMap: { '000000000000000': 'By customer', '000000000010000': 'By project', '000010000000000': 'By ship location' } },
    {name: 'maturityDate1', type: '_id_11' },
    {name: 'maturityDate2', type: '_id_11' },
    {name: 'maturityDate3', type: '_id_11' },
    {name: 'operator', type: '_id_20' },
    {name: 'uPCEAN', type: '_id_10' },
    {name: 'salaryCategory', type: '_id_19' } , {name: 'salaryCategory$_identifier' },
    {name: 'invoicePrintformat', type: '_id_800007' , valueMap: { '1': 'Detail', '3': 'Product detail', '2': 'Total' } },
    {name: 'consumptionDays', type: '_id_11' },
    {name: 'bankAccount', type: '_id_800046' } , {name: 'bankAccount$_identifier' },
    {name: 'taxCategory', type: '_id_800090' } , {name: 'taxCategory$_identifier' },
    {name: 'pOMaturityDate1', type: '_id_11' },
    {name: 'pOMaturityDate2', type: '_id_11' },
    {name: 'pOMaturityDate3', type: '_id_11' },
    {name: 'transactionalBankAccount', type: '_id_800046' } , {name: 'transactionalBankAccount$_identifier' },
    {name: 'sOBPTaxCategory', type: '_id_800090' } , {name: 'sOBPTaxCategory$_identifier' },
    {name: 'fiscalcode', type: '_id_10' },
    {name: 'isofiscalcode', type: '_id_10' },
    {name: 'incotermsPO', type: '_id_D2D4CC1F12434BB69733166A9423F2C6' } , {name: 'incotermsPO$_identifier' },
    {name: 'incotermsSO', type: '_id_D2D4CC1F12434BB69733166A9423F2C6' } , {name: 'incotermsSO$_identifier' },
    {name: 'paymentMethod', type: '_id_EED0EF97D4A7421687F3B365D009E7A6' } , {name: 'paymentMethod$_identifier' },
    {name: 'pOPaymentMethod', type: '_id_EED0EF97D4A7421687F3B365D009E7A6' } , {name: 'pOPaymentMethod$_identifier' },
    {name: 'account', type: '_id_DF1CEA94B3564A33AFDB37C07E1CE353' } , {name: 'account$_identifier' },
    {name: 'pOFinancialAccount', type: '_id_DF1CEA94B3564A33AFDB37C07E1CE353' } , {name: 'pOFinancialAccount$_identifier' },
    {name: 'customerBlocking', type: '_id_20' },
    {name: 'vendorBlocking', type: '_id_20' },
    {name: 'paymentIn', type: '_id_20' },
    {name: 'paymentOut', type: '_id_20' },
    {name: 'salesInvoice', type: '_id_20' },
    {name: 'purchaseInvoice', type: '_id_20' },
    {name: 'salesOrder', type: '_id_20' },
    {name: 'purchaseOrder', type: '_id_20' },
    {name: 'goodsShipment', type: '_id_20' },
    {name: 'goodsReceipt', type: '_id_20' },
    {name: 'cashVAT', type: '_id_20' },
    {name: 'currency', type: '_id_112' } , {name: 'currency$_identifier' },
    {name: 'setNewCurrency', type: '_id_28' },
    {name: 'searchKey', type: '_id_10' , additional: true },
    {name: 'businessPartnerCategory$name', type: '_id_10' , additional: true },
    {name: 'vendor', type: '_id_20' , additional: true },
    {name: 'name', type: '_id_10' , additional: true },
    {name: 'customer', type: '_id_20' , additional: true }
    ]}
    )

;
        this.Super('init', arguments);
    },
        outHiddenInputPrefix: 'inpcBpartnerId',
gridProps: { sort: 5 , autoExpand: true , displayField: 'businessPartner$searchKey', displaylength:44,fkField: true, showHover: true },    type: '_id_56DEFF37A33F46D1AC918C97C4447EAF'
}
,

# HG changeset patch
# User Carlos Aristu <carlos.aristu@openbravo.com>
# Date 1437577370 -7200
#      Wed Jul 22 17:02:50 2015 +0200
# Node ID cd1ddf017185991943a616cae587cd15f35d3a5a
# Parent  dcf8b6e81b7b01c3c54a37e2780bd43f7720e6a5
Fixes issue 30403: selectors not working in grid view when display field is set

When setting the display field for a selector with FKSelectorUIDefinition as its UI Definition class, if the display field was referencing to a property of the main entity of the selector, the display field name returned as part of the gridProps in the view definition was 'entity'.
The _extraProperties in the datasource used by the selector does not contain the entity name if they are referencing to the main entity. That is the root of the problem: when trying to set the selected value of the selector picklist in grid view, the display field name was never found as we were using 'entity' instead of looking directly for 'property'.
This problem does not affect when setting as display field a field that references a property which does not belong to the main entity of the selector.
To solve this problem, now in FKSelectorUIDefinition we return 'property' as the display field name instead of 'entity' when the property belongs to the main entity.

diff --git a/modules/org.openbravo.userinterface.selector/src/org/openbravo/userinterface/selector/reference/FKSelectorUIDefinition.java b/modules/org.openbravo.userinterface.selector/src/org/openbravo/userinterface/selector/reference/FKSelectorUIDefinition.java
--- a/modules/org.openbravo.userinterface.selector/src/org/openbravo/userinterface/selector/reference/FKSelectorUIDefinition.java
+++ b/modules/org.openbravo.userinterface.selector/src/org/openbravo/userinterface/selector/reference/FKSelectorUIDefinition.java
@@ -106,23 +106,11 @@
     String displayFieldName = JsonConstants.IDENTIFIER;
     if (displayField != null && displayField.getProperty() != null) {
       displayFieldName = displayField.getProperty();
+      return displayFieldName.replace(".", DalUtil.FIELDSEPARATOR);
     } else {
       // fallback to the default
       return null;
     }
-
-    if (!prop.getReferencedProperty().getEntity().hasProperty(getFirstProperty(displayFieldName))) {
-      // If the first property of the display field name does not belong to the referenced entity,
-      // return the displayFieldName
-      // Otherwise trying to append the displayFieldName to the referenced property would later
-      // result in an error
-      return displayFieldName.replace(".", DalUtil.FIELDSEPARATOR);
-    } else {
-      final String result = (prop.getName() + DalUtil.FIELDSEPARATOR + displayFieldName).replace(
-          ".", DalUtil.FIELDSEPARATOR);
-      return result;
-    }
-
   }
 
   private String getFirstProperty(String displayFieldName) {