tim-hub / obsidian-bible-reference

Take Bible Study notes easily in the popular note-taking app Obsidian, with automatic verse and reference suggestions.
https://antioch.tech/obsidian-bible-reference/
MIT License
221 stars 41 forks source link

Biblebook Tags for numbered Books not working #153

Closed fre98fi closed 3 months ago

fre98fi commented 9 months ago

Describe the bug When a numbered book like "2 Corinthians" is referenced and a Book-Tag is generated, the generated Tag looks like "#2 Corinthians". The problem here is that Obsidian will not recognize this as a Tag. It should look like "#2Corinthians".

To Reproduce Steps to reproduce the behavior:

  1. Activate Book Tags in the Expert Settings
  2. Reference a numbered Book
  3. Look for the used Tags

Expected behavior Tags should be in the right format to be recognized by Obsidian

What Device Do You Use(please complete the following information):

tim-hub commented 9 months ago

Thanks for raising this, I didn't notice it. Will have a look

stale[bot] commented 7 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

yintzgit commented 5 months ago

Hi, similar issue in windows11pro: with numbered books (eg Cor, Thess) it is not possible to get the bible text, same with or without tag. Doesnot work in german bible texts (eg Luther, Menge). English works, french works, spanish works. Regards from Germany,

stale[bot] commented 3 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

abbasou commented 3 months ago

Just noticed this as well. Fixed by adding .replace(/ /g, "") to:

bottom += ((_c = this.settings) == null ? void 0 : _c.bookTagging) ? ` #${this.verseReference.bookName

and

bottom += ((_d = this.settings) == null ? void 0 : _d.chapterTagging) ? ` #${this.verseReference.bookName

in get bottom() Hope that helps.

abbasou commented 3 months ago

Pull request submitted to fix this.

tim-hub commented 3 months ago

https://github.com/tim-hub/obsidian-bible-reference/releases/tag/2.5.2

Fixed here. Credits @abbasou