Author Topic: Additional Tagging & Reporting Tools  (Read 1206985 times)

boroda

  • Hero Member
  • *****
  • Posts: 5309
Just wondering:

$SentenceCase(<Title>)
will work without any additional parameters

$TitleCase(<Title>)
doesn't work without additional parameters

hmm. indeed. i don't think it's MB bug. i'll check my code.

boroda

  • Hero Member
  • *****
  • Posts: 5309
Just wondering:

$SentenceCase(<Title>)
will work without any additional parameters

$TitleCase(<Title>)
doesn't work without additional parameters

as usual, i haven't tested what i did. fixed version:

https://www.mediafire.com/file/h2t08o9562efboi/mb_TagTools_latest.zip/file

thanks for bug report.

Dizza17

  • Full Member
  • ***
  • Posts: 192
@ Boroda,

Just a little shout out for your absolute dedication to consistently improving your plug in and going out of your way to accommodate the requests of the MB community. You’re definitely up there with the best of the best people who along side Steven, continue to make Musicbee a joy to use.

Cheers Aaron

When the rich wage war, it’s the poor who suffer.

hiccup

  • Hero Member
  • *****
  • Posts: 9172
After some further testing I have a suggestion and a question:

1.
word_separators

if you do:
$TitleCase(<Title>," the "," nyc "," : ")

on the title:
test the title: the nyc song

expected:
Test the Title: The NYC Song

result:
Test the Title: the NYC Song

I think 'word_separators' should overrule'lower_case_words'?


2.
lower_case_after_chars
quote: "consists of a list of characters, after which the words become lowercase (e.g., ' for something like "Someone's")"

I don't understand how or when to use this one.

If you do:
$TitleCase(<Title>," the "," nyc "," : ",','," ' ")

on the title:
hey c'mon, let's test someone's title

It will give the same result (Hey C'mon, Let's Test Someone's Title) either with or without that parameter?
 
Last Edit: July 26, 2024, 05:09:16 PM by hiccup

boroda

  • Hero Member
  • *****
  • Posts: 5309
Just a little shout out for your absolute dedication to consistently improving your plug in and going out of your way to accommodate the requests of the MB community. You’re definitely up there with the best of the best people who along side Steven, continue to make Musicbee a joy to use.

thanks. it's my hobby. everybody has some hobby  ;)

boroda

  • Hero Member
  • *****
  • Posts: 5309
word_separators

if you do:
$TitleCase(<Title>," the "," nyc "," : ")

on the title:
test the title: the nyc song

expected:
Test the Title: The NYC Song

result:
Test the Title: the NYC Song

I think 'word_separators' should overrule'lower_case_words'?

firstly, it's a major bug just because the first word must be capitalized always. secondly, probably you are right about overrule, but i need to check this. what rule do you think should take precedence?

lower_case_after_chars
quote: "consists of a list of characters, after which the words become lowercase (e.g., ' for something like "Someone's")"

I don't understand how or when to use this one.

If you do:
$TitleCase(<Title>," the "," nyc "," : ",','," ' ")

on the title:
hey c'mon, let's test someone's title

It will give the same result (Hey C'mon, Let's Test Someone's Title) either with or without that parameter?

i don't understand either  ;D it's just the option of dedicated "change case" command (which is used as backend for VT functions), which is useless in this case. i'll remove it. i think i'll keep it. maybe there are other useful use cases.
Last Edit: July 26, 2024, 07:58:44 PM by boroda

boroda

  • Hero Member
  • *****
  • Posts: 5309
word_separators

if you do:
$TitleCase(<Title>," the "," nyc "," : ")

on the title:
test the title: the nyc song

expected:
Test the Title: The NYC Song

result:
Test the Title: the NYC Song

I think 'word_separators' should overrule'lower_case_words'?

i can't reproduce this. i get:

The NYC Song

but:

Nyc the Song

i consider the second case a bug. i would expect in this case:

NYC the Song

i'll correct this.
Last Edit: July 27, 2024, 12:18:13 AM by boroda

hiccup

  • Hero Member
  • *****
  • Posts: 9172
i can't reproduce this. i get:
The NYC Song
but:
Nyc the Song
i consider the second case a bug. i would expect in case:
NYC the Song
i'll correct this.
I can't do testing now, but perhaps you misunderstood the full title that I used as an example?
It's the whole:
test the title: the nyc song

boroda

  • Hero Member
  • *****
  • Posts: 5309
I can't do testing now, but perhaps you misunderstood the full title that I used as an example?
It's the whole:
test the title: the nyc song

ahh! yes, i did. i'll re-test VT function.

meanwhile:

numerous though minor "change case" command and related VT functions fixes. "NYC the Song" case is fixed too.

https://www.mediafire.com/file/h2t08o9562efboi/mb_TagTools_latest.zip/file

Dizza17

  • Full Member
  • ***
  • Posts: 192
Hi Boroda,

After some testing it appears the “sort involved people by their role preset isn’t working as expected. No changes are generated. Is there a small change to the pre-filled details that might need to be changed?

Regards Aaron
When the rich wage war, it’s the poor who suffer.

boroda

  • Hero Member
  • *****
  • Posts: 5309
word_separators

if you do:
$TitleCase(<Title>," the "," nyc "," : ")

on the title:
test the title: the nyc song

expected:
Test the Title: The NYC Song

result:
Test the Title: the NYC Song

I think 'word_separators' should overrule'lower_case_words'?

why would you expect "Test the Title: The NYC Song"? word separators are just word separators. they are only custom addition to spaces. and it's senseless in this case because there is an already space after the colon. what i can do is add new parameter "override all exception after chars". or change the meaning of the existing last parameter (which seems useless anyway) to this.

boroda

  • Hero Member
  • *****
  • Posts: 5309
Hi Boroda,

After some testing it appears the “sort involved people by their role preset isn’t working as expected. No changes are generated. Is there a small change to the pre-filled details that might need to be changed?

Regards Aaron

i've just changed "sort performers" preset to sort all "artists" (not only performers). and i don't remember what "sort performers" preset does. it would be better if you just set me a task from scratch.

Dizza17

  • Full Member
  • ***
  • Posts: 192
“Sort performers” sorts the artist tag list of performers by their instrument as written in the custom text box.

What i am after is to be able to sort the involved people tag by their role as written in the custom text box. Not just by alphabetical order of the role name but by any order as depicted in the custom text box. Ie: Engineer; producer; writer; lyricist; arranger; mixer.

Cheers
When the rich wage war, it’s the poor who suffer.

boroda

  • Hero Member
  • *****
  • Posts: 5309
@hiccup, i've changed the meaning of the last parameter of $TitleCase function to something like "override all exceptions after chars besides the words from uppercase list". i even don't fully understand how exactly it works. too complicated code. but $TitleCase(<Title>," the "," nyc ",`,`,`," : ") converts:

test the title: the nyc song

to:

Test the Title: The NYC Song

p.s. the words from the uppercase list must become uppercase always, without any exceptions.

https://www.mediafire.com/file/h2t08o9562efboi/mb_TagTools_latest.zip/file

boroda

  • Hero Member
  • *****
  • Posts: 5309
@Dizza17, ok, i've checked the code, it seems good. could you give me an example of roles and artists tag, so i can check what is happening in debugger?