From 4519b3e6319c9b7338596a601dd4c812b6c58092 Mon Sep 17 00:00:00 2001
From: Vishweshwar Saran Singh Deo <vssdeo@gmail.com>
Date: Mon, 5 Feb 2024 22:35:26 +0530
Subject: [PATCH] [bug 760] working directory feature is broken - as per the
 bug the layout does not get updated when "Save" button in layout is pressed -
 it does get updated if the window is closed by pressing top x close icon. -
 on pressing Save, it seems that the prefseditor takes this from
 current_layout = terminator.describe_layout() and saves it in config -
 whereas the current changes are done per key stroke and config is updated -
 this patch copies the parameters like directory and command when Save is
 press and on_layoutrefreshbutton_clicked() is called - Hence working dir and
 command are copied when Save is pressed using uuid to match terminals. - If
 there is a command registered then the terminal runs the command and exits.
 so one is not able to see the results.

---
 terminatorlib/config.py      | 28 ++++++++++++++++++++++++++++
 terminatorlib/prefseditor.py | 10 +++++++++-
 terminatorlib/terminal.py    |  2 +-
 3 files changed, 38 insertions(+), 2 deletions(-)

diff --git a/terminatorlib/config.py b/terminatorlib/config.py
index 97583570..404a506e 100644
--- a/terminatorlib/config.py
+++ b/terminatorlib/config.py
@@ -499,6 +499,34 @@ class Config(object):
         """Set a layout"""
         return(self.base.set_layout(layout, tree))
 
+    def copy_layout_item(self, src_layout, dst_layout, item):
+        items = {}
+        for child in src_layout:
+            section   = src_layout[child]
+            sec_type  = section.get('type', None)
+            if sec_type != 'Terminal':
+                continue
+
+            cp_item = section.get(item, None)
+            uuid    = str(section.get('uuid', None))
+            if cp_item:
+                items[uuid] = cp_item
+
+        dbg("items to be copied:%s" % items)
+        for child in dst_layout:
+            section   = dst_layout[child]
+            sec_type  = section.get('type', None)
+            if sec_type != 'Terminal':
+                continue
+
+            uuid       = str(section.get('uuid', None))
+            update_item = items.get(uuid, None)
+            if uuid and update_item:
+               dbg("update layout item:(%s) with value:(%s)"
+                                            % (item, update_item))
+               section[item] = update_item
+
+
 class ConfigBase(Borg):
     """Class to provide access to our user configuration"""
     loaded = None
diff --git a/terminatorlib/prefseditor.py b/terminatorlib/prefseditor.py
index 72c14110..e8b7a613 100755
--- a/terminatorlib/prefseditor.py
+++ b/terminatorlib/prefseditor.py
@@ -1594,6 +1594,14 @@ class PrefsEditor:
         (model, rowiter) = selected.get_selected()
         name = model.get_value(rowiter, 0)
 
+        config_layout  = self.config.base.get_layout(name)
+        dbg("layout from terminator:(%s)" % current_layout)
+        dbg("layout from config:(%s)" % config_layout)
+
+        self.config.copy_layout_item(config_layout, current_layout, 'directory')
+        self.config.copy_layout_item(config_layout, current_layout, 'command')
+        dbg("updated layout from terminator:(%s)" % current_layout)
+
         if self.config.replace_layout(name, current_layout):
             treeview.set_cursor(model.get_path(rowiter), column=treeview.get_column(0), start_editing=False)
         self.config.save()
@@ -2163,7 +2171,7 @@ class LayoutEditor:
 
     def on_layout_profile_workingdir_activate(self, widget):
         """A new working directory has been entered for this item"""
-        workdir = widget.get_text()
+        workdir = os.path.expanduser(widget.get_text())
         layout = self.config.layout_get_config(self.layout_name)
         layout[self.layout_item]['directory'] = workdir
         self.config.save()
diff --git a/terminatorlib/terminal.py b/terminatorlib/terminal.py
index 7d58f9d8..3c82deda 100644
--- a/terminatorlib/terminal.py
+++ b/terminatorlib/terminal.py
@@ -1507,7 +1507,7 @@ class Terminal(Gtk.VBox):
     def set_cwd(self, cwd=None):
         """Set our cwd"""
         if cwd is not None:
-            self.cwd = cwd
+            self.cwd = os.path.expanduser(cwd)
 
     def held_open(self, widget=None, respawn=False, debugserver=False):
         self.is_held_open = True