Fix: properly escaping pipes in Link.obsidianLink()

[GottZ]

Closes #2264

please tell me if this is desired behavior or not.

edit:

TS2550: Property 'replaceAll' does not exist on type 'string'. Do you need to change your target library? Try changing the 'lib' compiler option to 'es2021' or later.

so.. are you willing to go that step? adding this change is more complicated than replacing it with replace(/\|/g, "\\|")

[blacksmithgu]

Obsidian uses a consistent electron version so as long as es2021 is supported in Obsidian it's fine.

Based on checking the developer console this looks fine to me, it seems to be supported.

[GottZ]

ok nice. I'll edit the pull request later this day accordingly :)
thanks!

[GottZ]

@blacksmithgu the linter is checking for , comma characters at the end of object definitions for good reason.
would you agree that such a big codebase change should be done after the current pull-requests got reviewed or before?
I'll create a separate pull request to make the linter happy anyways, I could also keep it up2date with master, so any linting errors stay visible as file changes.

[blacksmithgu]

I'll fix the lints in a followup commit to avoid polluting this one.