From 36e41f171bccaae471234c0ff29e87156754c033 Mon Sep 17 00:00:00 2001
From: Nicolas Micoud <nmicoud@tgi.fr>
Date: Wed, 20 Jan 2016 16:18:05 +0100
Subject: [PATCH] IDEMPIERE-3008 Store divider of tree panel location for
 window per user

---
 .../adempiere/webui/adwindow/ADTabpanel.java  | 71 +++++++++++++------
 1 file changed, 48 insertions(+), 23 deletions(-)

diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/ADTabpanel.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/ADTabpanel.java
index 96f479bc4a..fc750ca1f2 100644
--- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/ADTabpanel.java
+++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/ADTabpanel.java
@@ -203,6 +203,8 @@ DataStatusListener, IADTabpanel, IdSpace, IFieldEditorContainer
 
 	public static final String ON_TOGGLE_EVENT = "onToggle";
 	
+	private static final String DEFAULT_PANEL_WIDTH = "300px";
+
 	private static enum SouthEvent {
     	SLIDE(),
     	OPEN(),
@@ -360,7 +362,7 @@ DataStatusListener, IADTabpanel, IdSpace, IFieldEditorContainer
 			treePanel = new ADTreePanel(windowNo, gridTab.getTabNo());
 			West west = new West();
 			west.appendChild(treePanel);
-			west.setWidth("300px");
+			west.setWidth(widthTreePanel());
 			west.setCollapsible(true);
 			west.setSplittable(true);
 			west.setAutoscroll(true);
@@ -1171,6 +1173,17 @@ DataStatusListener, IADTabpanel, IdSpace, IFieldEditorContainer
     	return height;
     }
 
+    private String widthTreePanel() {
+    	String width = null;
+    	int windowId = getGridTab().getAD_Window_ID();
+    	int adTabId = getGridTab().getAD_Tab_ID();
+    	if (windowId > 0 && adTabId > 0) {
+    		width = Env.getPreference(Env.getCtx(), windowId, adTabId+"|TreePanel.Width", false);
+    	}
+    	else
+    		width = DEFAULT_PANEL_WIDTH;
+    	return width;
+    }
     private void navigateTo(DefaultTreeNode<MTreeNode> value) {
     	MTreeNode treeNode = value.getData();
     	//  We Have a TreeNode
@@ -1740,31 +1753,43 @@ DataStatusListener, IADTabpanel, IdSpace, IFieldEditorContainer
 		if (formContainer.getSouth() != null) {
 			if (formContainer.getSouth().isVisible() && formContainer.getSouth().isOpen()) {
 				String height = formContainer.getSouth().getHeight();
-				if (! Util.isEmpty(height)) {
-		    		int windowId = getGridTab().getAD_Window_ID();
-		    		int adTabId = getGridTab().getAD_Tab_ID();
-		    		if (windowId > 0 && adTabId > 0) {
-		    			Query query = new Query(Env.getCtx(), MTable.get(Env.getCtx(), I_AD_Preference.Table_ID), "AD_Window_ID=? AND Attribute=? AND AD_User_ID=? AND AD_Process_ID IS NULL AND PreferenceFor = 'W'", null);
-		    			int userId = Env.getAD_User_ID(Env.getCtx());
-		    			MPreference preference = query.setOnlyActiveRecords(true)
-		    										  .setApplyAccessFilter(true)
-		    										  .setParameters(windowId, adTabId+"|DetailPane.Height", userId)
-		    										  .first();
-		    			if (preference == null || preference.getAD_Preference_ID() <= 0) {
-		    				preference = new MPreference(Env.getCtx(), 0, null);
-		    				preference.setAD_Window_ID(windowId);
-		    				preference.set_ValueOfColumn("AD_User_ID", userId); // required set_Value for System=0 user
-		    				preference.setAttribute(adTabId+"|DetailPane.Height");
-		    			}
-	    				preference.setValue(height);
-		    			preference.saveEx();
-		    			//update current context
-		    			Env.getCtx().setProperty("P"+windowId+"|"+adTabId+"|DetailPane.Height", height);
-		    		}
-				}
+				if (! Util.isEmpty(height))
+					savePreference("DetailPane.Height", height);
 			}
 		}
+
+		if (treePanel != null && formContainer.getWest() != null) {
+			if (formContainer.getWest().isVisible() && formContainer.getWest().isOpen()) {
+				String width = formContainer.getWest().getWidth();
+				if (! Util.isEmpty(width))
+					savePreference("TreePanel.Width", width);
+			}
+		}
+
 		super.onPageDetached(page);
 	}
 
+	void savePreference(String attribute, String value)
+	{
+		int windowId = getGridTab().getAD_Window_ID();
+		int adTabId = getGridTab().getAD_Tab_ID();
+		if (windowId > 0 && adTabId > 0) {
+			Query query = new Query(Env.getCtx(), MTable.get(Env.getCtx(), I_AD_Preference.Table_ID), "AD_Window_ID=? AND Attribute=? AND AD_User_ID=? AND AD_Process_ID IS NULL AND PreferenceFor = 'W'", null);
+			int userId = Env.getAD_User_ID(Env.getCtx());
+			MPreference preference = query.setOnlyActiveRecords(true)
+					.setApplyAccessFilter(true)
+					.setParameters(windowId, adTabId+"|"+attribute, userId)
+					.first();
+			if (preference == null || preference.getAD_Preference_ID() <= 0) {
+				preference = new MPreference(Env.getCtx(), 0, null);
+				preference.setAD_Window_ID(windowId);
+				preference.set_ValueOfColumn("AD_User_ID", userId); // required set_Value for System=0 user
+				preference.setAttribute(adTabId+"|"+attribute);
+			}
+			preference.setValue(value);
+			preference.saveEx();
+			//update current context
+			Env.getCtx().setProperty("P"+windowId+"|"+adTabId+"|"+attribute, value);
+		}
+	}
 }