"Identifier" search regexp fails for TOML front-matter about denote

[protesilaos]

We will need to look for : or =, with optional space before =.

Oh, of course! Sorry about that. Will fix it right away.

Or do you plan to deeply bake in the reliance on #+identifier / identifier: / identifier = matches?

I think this is open to discussion, given that we still are at the point where we can make breaking changes.

[protesilaos]

The "^\\(#\\+\\)?\\(identifier\\)\\([\s\t]+\\)?[:=]\\([\s\t]+?\\)?\"?\\([0-9T]+\\)\"?" matches all of the following with M-x re-builder. Is there some other permutation we need to consider?

identifier = "20220611T114910"
identifier= 20220611T114910
identifier=20220611T114910
identifier= "20220611T114910"
identifier     ="20220611T114910"



identifier : "20220611T114910"
identifier: 20220611T114910
identifier:20220611T114910
identifier: "20220611T114910"
identifier     :"20220611T114910"

[kaushalmodi]

Is there some other permutation we need to consider?

Nope this covers (more than) everything!

---

(more than): Few illegal cases like identifier= 20220611T114910 and identifier=20220611T114910 too. But that's OK.

---

Regexp suggestion:

"\\bidentifier\\s-*[:=]\\s-*\"?\\([0-9T]+\\)"

[protesilaos]

Regexp suggestion:

Vastly superior! Thanks!

I guess now we need to abstract the "indentifier" part, right? That gives the user maximum flexibility.

[kaushalmodi]

Suggestion 2:

"^.?.?\\b\\(identifier\\|ID\\)\\s-*[:=]\\s-*\"?\\([0-9T]+\\)"

[kaushalmodi]

I think ^.?.?\\b will be good enough to avoid any false matches.

[protesilaos]

Very wel!! Since you wrote it, do you want to submit a PR (or send a patch)? If you are on mobile, I can commit with you as the author (if you want).

[kaushalmodi]

I haven't tested it properly and won't have time to do so until tonight (~10 hrs from now).

But I can create a PR and let you decide. If the PR is still open after I am done testing, and I need to update it, I'll do so, else will create another PR

[protesilaos]

Okay, thank you! Will wait then.

[kaushalmodi]

Closing this as it was fixed by #11.

[protesilaos]

Thank you!