accidentalType boolean

• Feb 5, 2022 - 03:49

I tried to check if a note had an accidental by checking the type with: if (curScore.selection.elements[0].accidentalType=="NONE") and it does not seem to work but accidentalType==0 works. Why is that? If that's the case, shouldn't there be a list of corresponding numbers in the API: https://musescore.github.io/MuseScore_PluginAPI_Docs/plugins/html/names…

Comments

Jojo-Schmitz

• Feb 5, 2022 - 06:56

"NONE" is a string, you'd need AccidentalType.NONE (and accidentalType is not a boolean but an enum)

jonarnold

• Feb 5, 2022 - 17:18

As part of an if statement? if (curScore.selection.elements[0].accidentalType.NONE) does not work. I'm using strings for element types in if statements.

Jojo-Schmitz

• Feb 5, 2022 - 17:25

I meant if (curScore.selection.elements[0].accidentalType == AccidentalType.NONE)

jonarnold

• Feb 5, 2022 - 17:28

Ah thanks. So the accidental type is not really a string? I'm trying to understand why we have to do what you suggest with that value but if (elementsList[i].name != "Accidental") works.

jonarnold

• Feb 5, 2022 - 17:31

Also, that code results in ReferenceError: AccidentalType is not defined

Jojo-Schmitz

• Feb 5, 2022 - 17:47

Dunno, but if (cursor.element.type === Element.REST) is valid code, so I don't see how if (curScore.selection.elements[0].accidentalType === AccidentalType.NONE) is not

jonarnold

• Feb 5, 2022 - 18:25

Right. So should I file an issue?

Jojo-Schmitz

• Feb 5, 2022 - 18:29

Hmm, maybe. Maybe that in enum AccidentalType : unsigned char { that : unsigned char is the problem

parkingb

• Feb 5, 2022 - 21:42

It is tricky. The enumeration name is not "AccidentalType" but "Accidental".
var sel=curScore.selection.elements;
var el=sel[0];

  console.log(el.accidentalType + " - " + Accidental.NONE + " - " + Accidental.SHARP");
  console.log((el.accidentalType == Accidental.NONE)?"NONE":"OTHER");

jonarnold

• Feb 5, 2022 - 21:50

Wow, that's not at all clear in the documentation. Especially when the property reads from accidentalType.

Jojo-Schmitz

• Feb 5, 2022 - 22:01

Indeed, very confusing

jeetee

• Feb 5, 2022 - 22:41

Caused by this comment: https://github.com/musescore/MuseScore/blob/3.x/mscore/plugin/api/qmlpl…

Jojo-Schmitz

• Feb 5, 2022 - 22:43

Mak s sense, but renders the documentation useless