fix: prioritize user config for theme-loader on windows (#1213)

[daephx]

No description provided.

[daephx]

@shadmansaleh Man I even referenced the commit in my issue and didn't notice lualine_require just duplicates the same logic... Should I patch that one too? I've never used the custom require function so I idk in what circumstances it will have the same problem.

Edit2:
At first I was uncertain but after some testing it seems this very well should be fixed, as it blocks the ability to load some user extensions.

lualine.nvim/lua/lualine_require.lua

Lines 65 to 67 in b5e8bb6

	table.sort(paths, function(a, b)
	return vim.startswith(a, user_config_path) or not vim.startswith(b, user_config_path)
	end)

Edit:
I don't know if it even matters for this function.
The code around it uses vim.api.nvim_get_runtime_file(pattern_path, false) and checks if the number of paths in the table is greater than 0. But the all=false parameter would only return the first match anyway. So either I'm misunderstanding the docs or it seems there's just never anything to sort! 😆

function vim.api.nvim_get_runtime_file(name: string, all: boolean) -> string[]
...
Parameters: ~
  • {name}  pattern of files to search for
  • {all}   whether to return all matches or only the first

lualine.nvim/lua/lualine_require.lua

Lines 56 to 73 in b5e8bb6

	pattern_path = table.concat { 'lua/', module:gsub('%.', '/'), '.lua' }
	local paths = vim.api.nvim_get_runtime_file(pattern_path, false)
	if #paths <= 0 then
	pattern_path = table.concat { 'lua/', module:gsub('%.', '/'), '/init.lua' }
	paths = vim.api.nvim_get_runtime_file(pattern_path, false)
	end
	if #paths > 0 then
	-- put entries from user config path in front
	local user_config_path = vim.fn.stdpath('config')
	table.sort(paths, function(a, b)
	return vim.startswith(a, user_config_path) or not vim.startswith(b, user_config_path)
	end)
	local mod_result = dofile(paths[1])
	package.loaded[module] = mod_result
	return mod_result
	end
	return require(module)