You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
sebastian-meyer
changed the title
[BUG] Recent adjustment of musical sources iterates over non-existing variable
[BUG] Access to non-existent MetsDocument property
Aug 20, 2024
Description
Using the nonexistent
MetsDocument
property inside the loops.musicalStructureInfo
is always null, as it was never defined nor assigned in the class.This problem appears in few places:
DocumentAnnotation
class -foreach ($this->document->getCurrentDocument()->musicalStructureInfo as $key => $musicalInfo)
NavigationController
class -$measureOptions[$i] = '[' . $i . ']' . ($this->document->getCurrentDocument()->musicalStructureInfo[$this->document->getCurrentDocument()->musicalStructure[$i]['measureid']]['orderlabel'] ? ' - ' . htmlspecialchars($this->document->getCurrentDocument()->musicalStructureInfo[$this->document->getCurrentDocument()->musicalStructureInfo[$i]]['orderlabel']) : '')
PageViewController
class -$musicalStruct = $doc->musicalStructureInfo;
->foreach ($musicalStruct as $measureData)
Reproduction
Steps to reproduce the behaviour:
Expected Behavior
The class parameters should be declared and assigned before the usage.
Screenshots and Examples
Environment
The text was updated successfully, but these errors were encountered: