svn commit: r1353422 - /ofbiz/trunk/framework/images/webapp/images/fieldlookup.js

[sascharodekamp]

Author: sascharodekamp
Date: Mon Jun 25 08:12:17 2012
New Revision: 1353422

URL: http://svn.apache.org/viewvc?rev=1353422&view=rev
Log:
Fix a small bug with the lookupDescription: An exception was thrown when a 
wrong parameter string was passed.

Modified:
    ofbiz/trunk/framework/images/webapp/images/fieldlookup.js

Modified: ofbiz/trunk/framework/images/webapp/images/fieldlookup.js
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/fieldlookup.js?rev=1353422&r1=1353421&r2=1353422&view=diff
==============================================================================
--- ofbiz/trunk/framework/images/webapp/images/fieldlookup.js (original)
+++ ofbiz/trunk/framework/images/webapp/images/fieldlookup.js Mon Jun 25 
08:12:17 2012
@@ -173,7 +173,7 @@ function initiallyCollapseDelayed() {
 
/*******************************************************************************
  * Lookup Object
  
******************************************************************************/
-var Lookup = function (options) {
+var Lookup = function(options) {
        var _newInputBoxId, _lookupId, _inputBox, _lookupContainer, 
_backgroundCloseClickEvent;
 
        options = {
@@ -404,7 +404,7 @@ var Lookup = function (options) {
 
/*******************************************************************************
  * Lookup Counter Object
  
******************************************************************************/
-var FieldLookupCounter = function () {
+var FieldLookupCounter = function() {
        this.refArr = {};
 
        this.setReference = function(key, ref) {
@@ -454,7 +454,7 @@ var GLOBAL_LOOKUP_REF = new FieldLookupC
 
/*******************************************************************************
  * Button Modifier Object
  
******************************************************************************/
-var ButtonModifier = function (lookupDiv) {
+var ButtonModifier = function(lookupDiv) {
 
        function _modifySubmitButton() {
                if (!lookupDiv) {
@@ -845,7 +845,7 @@ function closeLookup() {
  * Lookup Description Helper
  
******************************************************************************/
 // load description for lookup fields
-var lookupDescriptionLoaded = function (fieldId, url, params, formName) {
+var lookupDescriptionLoaded = function(fieldId, url, params, formName) {
        this.init(fieldId, url, params, formName);
 }
 lookupDescriptionLoaded.prototype.init = function(fieldId, url, params, 
formName) {
@@ -859,8 +859,14 @@ lookupDescriptionLoaded.prototype.update
        if (tooltipElement.length) {// first remove current description
                tooltipElement.remove();
        }
+
+       var indexOf = this.params.indexOf("searchValueFieldName");
+       if (indexOf == -1) {
+               return;
+       }
+
        // actual server call
-       var fieldName = 
this.params.substring(this.params.indexOf("searchValueFieldName"));
+       var fieldName = this.params.substring(indexOf);
        fieldName = fieldName.substring(fieldName.indexOf("=") + 1);
        if (jQuery("input[name=" + fieldName + "]").val()) {
                var fieldSerialized = jQuery("input[name=" + fieldName + "]", 
jQuery("form[name=" + this.formName + "]")).serialize();