IDEMPIERE-1233 Problem with using the Create Lines From process on the Material Receipt window.

org.adempiere.base/src/org/compiere/model/MWarehouse.java

@@ -193,6 +193,20 @@ public class MWarehouse extends X_M_Warehouse
 			log.warning("No default locator for " + getName());
 			return locators[0];
 		}
+		else
+		{
+			String whereClause = "M_Warehouse_ID=?";
+			List<MLocator> list = new Query(getCtx(), I_M_Locator.Table_Name, whereClause, null)
+											.setParameters(getM_Warehouse_ID())
+											.setOnlyActiveRecords(false).list();
+			if (!list.isEmpty()) 
+			{
+				if (log.isLoggable(Level.INFO))
+					log.info("All locator is inactive for " + getName());
+				//Do not auto create if there are inactive locator since that could trigger unique key exception
+				return null;
+			}
+		}	
 		//	No Locator - create one
 		MLocator loc = new MLocator (this, "Standard");
 		loc.setIsDefault(true);

org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/apps/form/WCreateFromShipmentUI.java

@@ -20,6 +20,7 @@ import java.util.List;
 import java.util.Vector;
 import java.util.logging.Level;
 
+import org.adempiere.exceptions.AdempiereException;
 import org.adempiere.webui.apps.AEnv;
 import org.adempiere.webui.component.Checkbox;
 import org.adempiere.webui.component.Grid;
@@ -80,6 +81,7 @@ public class WCreateFromShipmentUI extends CreateFromShipment implements EventLi
 		{
 			log.log(Level.SEVERE, "", e);
 			setInitOK(false);
+			throw new AdempiereException(e.getMessage());
 		}
 		AEnv.showWindow(window);
 	}

org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/window/WLocatorDialog.java

@@ -312,7 +312,8 @@ public class WLocatorDialog extends Window implements EventListener<Event>
 		
 		//lstLocator.setModel(m_mLocator);
 		//lstLocator.setValue(m_M_Locator_ID);
-		lstLocator.setSelectedIndex(0);
+		if (lstLocator.getItemCount() > 0)
+			lstLocator.setSelectedIndex(0);
 		lstLocator.addEventListener(Events.ON_SELECT, this);
 		
 		displayLocator();