Merge pull request #881 from NeogitOrg/fix/log-popup-files

lua/neogit/config.lua

@@ -226,7 +226,6 @@ function M.get_default_values()
       "NeogitPushPopup--force-with-lease",
       "NeogitPushPopup--force",
       "NeogitPullPopup--rebase",
-      "NeogitLogPopup--",
       "NeogitCommitPopup--allow-empty",
     },
     mappings = {

lua/neogit/lib/popup/builder.lua

@@ -194,6 +194,10 @@ function M:option(key, cli, value, description, opts)
     opts.separator = "="
   end
 
+  if opts.setup then
+    opts.setup(self)
+  end
+
   table.insert(self.state.args, {
     type = "option",
     id = opts.key_prefix .. key,

lua/neogit/lib/state.lua

@@ -79,15 +79,21 @@ local function gen_key(key_table)
 end
 
 ---Set option and write to disk
----@param key table
+---@param key string[]
 ---@param value any
 function M.set(key, value)
   if not M.enabled() then
     return
   end
 
-  if not vim.tbl_contains(config.values.ignored_settings, gen_key(key)) then
-    M.state[gen_key(key)] = value
+  local cache_key = gen_key(key)
+  if not vim.tbl_contains(config.values.ignored_settings, cache_key) then
+    if value == "" then
+      M.state[cache_key] = nil
+    else
+      M.state[cache_key] = value
+    end
+
     M.write()
   end
 end

lua/neogit/popups/log/init.lua

@@ -29,6 +29,16 @@ function M.create()
       key_prefix = "-",
       separator = "",
       fn = actions.limit_to_files,
+      setup = function(popup)
+        local value = require("neogit.lib.state").get { "NeogitLogPopup", "" }
+        value = vim.split(value, " ", { trimempty = true })
+        value = require("neogit.lib.util").map(value, function(v)
+          local result, _ = v:gsub([["]], "")
+          return result
+        end)
+
+        popup.state.env.files = value
+      end,
     })
     :switch("f", "follow", "Follow renames when showing single-file log")
     :arg_heading("Commit Ordering")