midsuit
/
core-jgi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix [ 2053140 ] Mandatory lookup fields autofilled (badly)

This commit is contained in:
Carlos Ruiz 2008-08-15 17:30:00 +00:00
parent dd231952a6
commit 5d75884ae7
1 changed files with 19 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
base/src/org/compiere/model/Lookup.java
View File

@ -1,5 +1,5 @@
/******************************************************************************
* Product: Adempiere ERP & CRM Smart Business Solution *
* Product: Adempiere ERP & CRM Smart Business Solution *
* Copyright (C) 1999-2006 ComPiere, Inc. All Rights Reserved. *
* This program is free software; you can redistribute it and/or modify it *
* under the terms version 2 of the GNU General Public License as published *
@ -16,10 +16,16 @@
*****************************************************************************/
package org.compiere.model;
import java.io.*;
import java.util.*;
import javax.swing.*;
import org.compiere.util.*;
import java.io.Serializable;
import java.util.ArrayList;
import javax.swing.AbstractListModel;
import javax.swing.MutableComboBoxModel;
import org.compiere.util.CLogger;
import org.compiere.util.KeyNamePair;
import org.compiere.util.NamePair;
import org.compiere.util.ValueNamePair;
/**
* Base Class for MLookup, MLocator.
@ -277,14 +283,15 @@ public abstract class Lookup extends AbstractListModel
obj = m_selectedObject;
}
// comment next code because of bug [ 2053140 ] Mandatory lookup fields autofilled (badly)
// if nothing selected & mandatory, select first
if (obj == null && mandatory && p_data.size() > 0)
{
obj = p_data.get(0);
m_selectedObject = obj;
log.finest(getColumnName() + ": SelectedValue SetToFirst=" + obj);
// fireContentsChanged(this, -1, -1);
}
// if (obj == null && mandatory && p_data.size() > 0)
// {
// obj = p_data.get(0);
// m_selectedObject = obj;
// log.finest(getColumnName() + ": SelectedValue SetToFirst=" + obj);
// // fireContentsChanged(this, -1, -1);
// }
m_loaded = true;
fireContentsChanged(this, 0, p_data.size());