epwalsh / obsidian.nvim

Obsidian 🤝 Neovim
Apache License 2.0
3.3k stars 150 forks source link

Convert paths to strings before concatenating #602

Closed ridulfo closed 1 month ago

ridulfo commented 1 month ago

Description

When renaming a note, paths are logged. These paths are table values and cannot be concatenated without first being converted to strings.

How to test

Simply run :ObsidianRename new-name --dry-run

These errors should not appear anymore:

Do you want to continue? [Y/n] Error executing Lua callback: ...nvim/lazy/obsidian.nvim/lua/obsidian/commands/rename.lua:157: attempt to concatenate local 'ne
w_note_path' (a table value)
stack traceback:
        ...nvim/lazy/obsidian.nvim/lua/obsidian/commands/rename.lua:157: in function 'func'
        ...e/nvim/lazy/obsidian.nvim/lua/obsidian/commands/init.lua:68: in function <...e/nvim/lazy/obsidian.nvim/lua/obsidian/commands/init.lua:67>

PS: This is my first PR in Lua, I am gladly open for improvements.