OPUS4 / framework

OPUS 4 Database Implementation
Other
2 stars 7 forks source link

To/From Array Funktionen für CollectionRole-Objekte #283

Open j3nsch opened 2 years ago

j3nsch commented 2 years ago

Die toArray Funktion für die CollectionRole-Klasse ist deprecated, wobei nicht ganz klar ist warum. Auch Opus\CollectionRole-Objekte sollten sich in ein Array umwandeln lassen und sie sollten aus einem Array heraus erzeugt werden können.

Das gewünschte Verhalten muss genau definiert und dokumentiert werden. Es sollte möglich sein, eine komplette Sammlungshierarchy aus einem Array heraus zu erzeugen. Da das kostspielig sein kann, muss darüber nachgedacht werden, ob es sinnvoll ist die Konvertierung auf eine oder mehrere Ebenen zu beschränken.

Das wichtigste ist, dass das Verhalten des Frameworks definiert und erklärt wird.

In der CollectionRole-Klasse gibt es unter anderem folgenden Kommentar vom 29. September 2010.

// FIXME: We have to reload model to get correct results!

Das stimmt und wurde seit dem nicht weiter dokumentiert oder behoben. Ähnliche Probleme treten in vielen anderen Klassen auf. Während der DOI-Entwicklung in 2018 wurde ein Bug-Ticket für ein solches Problem für Identifier angelegt. Es geht hier um das Verhalten der Klassen, die das Datenmodell definieren, den grundlegendsten Komponenten, dem Fundament, dem Framework. Wenn es dort wackelige Stellen gibt, wird die ganze Entwicklung beeinträchtigt und verlangsamt. Deshalb ist es wichtig diese Stellen zu fixen oder zumindest erst einmal ganz klar festzuhalten.

Intern: https://tickets.zib.de/jira/browse/OPUSVIER-3943

j3nsch commented 2 years ago

Dieser Code wird momentan völlig überarbeitet, als Vorbereitung für den Umstieg auf Doctrine und im Zuge einer kompletten Umstrukturierung der Implementation des Datenmodells. Damit sollte sich das Problem hier mit erledigen.