Sortierung in View nach Bearbeitung einer Institution #4

oschihin · 2014-01-30T15:54:56Z

Problem

Die Sortierung einer Instititution in der View richtet sich nach dem Wert im Feld 'position' im entsprechenden json-File:

{
id": 2392,
"bib_code": "A100",
"sys_code": "A100",
"position": "638",
"favorite": true,
...
}

Bearbeitet man eine Institution, wird sie in libadmin in der Ansicht "View -> Institutionen sortieren" bei allen Views an erster Stelle angezeigt. Im json-File erhält sie einen Wert position=0

{
id": 2392,
"bib_code": "A100",
"sys_code": "A100",
"position": "0",
"favorite": true,
...
}

Weswegen nach Bearbeitungen in libadmin in jeder View die Institutionen neu sortiert werden müssen. (siehe letzten Kommentar von @mschwendener hier: swissbib/sbvf2#107)

Lösung

Analyse: Wieso wird der Sortierwert auf 0 gesetzt, wie entsteht er und wie bleibt er erhalten?
Lösung: Der Sortierwert bleibt beim speichern einer Institution erhalten und wird korrekt ausgegeben über die Schnittstelle

maechler · 2014-03-19T08:28:57Z

Das Problem ist, dass beim Bearbeiten einer Institution die View- und Gruppenzugehörigkeit geändert werden kann. Das kann dazu führen, dass zwei Institutionen den selben Sortierwert erhalten.

Wir haben das so gelöst, dass jetzt nur noch wenn die View- oder Gruppenzugehörigkeit geändert wird, die Sortierung gelöscht wird. Andere Änderungen (Name, Adresse, etc.) haben keinen Einfluss auf die Sortierung.

oschihin · 2014-03-21T13:37:06Z

@mmchler @guenterh
Die Lösung hat offenbar einen gröberen Bug zur Folge:

Bearbeiten einer Institution (z.B. Adresskorrektur)
Speichern
Im Tab "Gruppen und Views" der Institution sieht's dann so aus:

Hier sieht man auch schon die Folge in den Daten:
4. In den Views > "Institutionen sortieren" erscheint die Institution nur noch bei green, und zwar zuoberst (Sortierwert 0)
5. Prüfung der Datenausgabe via JSON: Die bearbeitete Institution ist aus allen Views verschwunden, mit Ausnahme von green.json

Aktuell ist der neueste Stand auf libadmintest installiert. Wir können dies unter keinen Umständen produktiv schalten. Korrektur dringend!

maechler · 2014-03-21T14:39:53Z

@oschihin @guenterh

Das sieht gar nicht gut aus! Ich werde mich so bald wie möglich darum kümmern.
Ich habe bei mir noch alte Testdaten mit nur wenigen Views und konnte eben den Fehler nicht reproduzieren.
Gibt es irgendwo einen aktuellen DB-Dump, oder könnt ihr vielleicht einen machen ?
Ich schaue aber ob ich auch so schon einen Fehler finde.

maechler · 2014-03-27T08:56:33Z

@oschihin @guenterh

Vielen Dank für die aktuellen Daten!
Ich konnte den Fehler leider immer noch nicht reproduzieren. Ich vermute deshalb, dass es evtl mit unterschiedlichen Datenbankkonfigurationen zusammenhängen könnte.
Ich habe die Änderung gemacht, bei der ich denke es könnte das Problem lösen.

Bitte teste doch den aktuellen Stand nochmals auf eurem development Host.

oschihin · 2014-03-27T13:53:38Z

Das Problem besteht weiterhin, Einträge zu Views im "Institutionen > Gruppen & Views"
werden vervielfacht, mit dem Titel "This label will be changed (by the app)". Aktiviert
man eine Gruppe, so bleibt das Häkchen nicht bestehen.

Eine Änderung in der Adresse einer Institution führt dazu, dass die Institution aus der
View fällt (geprüft mit A154 > ../orange.json)

Nachprüfbar unter admin.swissbib.ch/libadmintest

-------- Original-Nachricht --------
Betreff: Re: [libadmin] Sortierung in View nach Bearbeitung einer Institution (#4)
Von: Markus Mächler notifications@github.com
An: swissbib/libadmin libadmin@noreply.github.com
Kopie (CC): Oliver Schihin oliver.schihin@unibas.ch
Datum: 27.03.2014 09:56

@oschihin https://github.com/oschihin @guenterh https://github.com/guenterh

Vielen Dank für die aktuellen Daten!
Ich konnte den Fehler leider immer noch nicht reproduzieren. Ich vermute deshalb, dass es
evtl mit unterschiedlichen Datenbankkonfigurationen zusammenhängen könnte.
Ich habe die Änderung gemacht, bei der ich denke es könnte das Problem lösen.

Bitte teste doch den aktuellen Stand nochmals auf eurem development Host.

—
Reply to this email directly or view it on GitHub
#4 (comment).

maechler · 2014-04-03T13:16:55Z

Fazit:
Das Problem ist ein ZF2 Bug, der ab Version 2.2.6 auftritt. Ich konnte keine schnelle Lösung finden. Am Besten gehen wir zurück auf ZF 2.2.5 und hoffen, dass der Bug in einer nächsten Version behoben wird.

mschwendener · 2014-04-03T13:20:13Z

Ok für mich, ist kein grosser Mehraufwand.

@ToVie muss einfach auch daran denken, wenn er an libadmin etwas ändert ;)

guenterh · 2014-04-03T13:20:30Z

@mmchler
Hallo Markus,

kannst Du auf
https://github.com/zendframework/zf2/issues
einen issue erfassen? Du hast es Dir angesehen und kannst es am besten kurz beschreiben
Wenn das niemand weiss, haben wir schlechte Chancen, dass der Bug behoben wird.

Danke und Grüsse!

Günter

maechler · 2014-04-03T13:25:28Z

@guenterh @oschihin
Um Fehler zu vermeiden, könnte ich noch das composer.json auf Version 2.2.5 fixieren, was meint ihr?

@guenterh
Es ist etwas schwierig da ich den Fehler nicht auf ein paar Code-Zeilen einschränken kann. Ich werde aber versuchen ein brauchbares Issue zu erfassen!

oschihin · 2014-04-03T13:28:10Z

composer.json freezen finde ich gut

-------- Original-Nachricht --------
Betreff: Re: [libadmin] Sortierung in View nach Bearbeitung einer Institution (#4)
Von: Markus Mächler notifications@github.com
An: swissbib/libadmin libadmin@noreply.github.com
Kopie (CC): Oliver Schihin oliver.schihin@unibas.ch
Datum: 03.04.2014 15:25

@guenterh https://github.com/guenterh @oschihin https://github.com/oschihin
Um Fehler zu vermeiden, könnte ich noch das composer.json auf Version 2.2.5 fixieren, was
meint ihr?

@guenterh https://github.com/guenterh
Es ist etwas schwierig da ich den Fehler nicht auf ein paar Code-Zeilen einschränken kann.
Ich werde aber versuchen ein brauchbares Issue zu erfassen!

—
Reply to this email directly or view it on GitHub
#4 (comment).

guenterh · 2015-11-24T13:43:23Z

Meine aktuelle Analyse ergibt folgendes:
Durch den Wechsel von ZF 2.2.5 zu 2.2.6 hat es unter anderem in der Implementierung der add - Element Methode eines Fieldset Elements eine Änderung gegeben.
https://github.com/swissbib/libadmin/blob/feature/nextMaster_mergedWithDevelop_moreCode/module/Libadmin/src/Libadmin/Form/InstitutionFieldset.php#L227

In 2.2.5 gab es noch folgendes Stückchen code zum Vorbereiten eines Fieldset Elements

    if ($elementOrFieldset instanceof FieldsetInterface) {
        if ($elementOrFieldset instanceof FieldsetPrepareAwareInterface) {
            $elementOrFieldset->prepareFieldset();
        }

prepareFieldset ist in 2.2.6 weggefallen.

Vor allem bei eingebundenen Collections führt das dazu, dass die Collectionobjekte nicht mehr gebunden werden. Das kann man möglicherweise selber steuern, indem man die prepare Methode selber aufruft (was ich an dieser Stelle im Controller mache: https://github.com/swissbib/libadmin/blob/feature/nextMaster_mergedWithDevelop_moreCode/module/Libadmin/src/Libadmin/Controller/InstitutionController.php#L95

Der dahinter stehende Mechanismus verlangt die Implemenierung von bestimmten Schnittstellen (Traversable und ArrayAccess) was ich mal testweise für das ModelItem InstitutionRelation gemacht habe.
Ein Aufruf ist nun ohne Fehler möglich, die Werte werden aber immer noch nicht gebunden ....

Weitere Hinweise zu Anpassungen durch den ZF Releaseupdate:
http://framework.zend.com/changelog/2.2.6/
zendframework/zendframework#5502

@maechler: any ideas?

@oschihin z.K.

Der Branch, auf dem die aktuellen Anpassungen implementiert sind:
https://github.com/swissbib/libadmin/tree/feature/nextMaster_mergedWithDevelop_moreCode

mschwendener · 2015-12-01T09:52:16Z

Aktueller Stand testlibadmin:

Änderung an Bibliotheksname: die geänderte Bibliothek steht bei den Views nicht mehr zuoberst - aber die Sortierung wird nicht aktualisiert!! D.h. im Gegensatz zur Produktion merke ich jetzt nicht mehr, dass die Sortierung falsch ist. Bsp: Alexandria umbenannt zu TEST Alexandria - sortiert weiterhin unter Al.. & Nationalbibliothek umbenannt zu Schweizerische ..., sortiert weiterhin unter Na..

oschihin · 2015-12-01T11:30:43Z

@maechler @mschwendener @guenterh
Im Issue (ganz weit oben ;-) steht als Lösung: "Der Sortierwert bleibt beim speichern einer Institution erhalten und wird korrekt ausgegeben über die Schnittstelle." Das neue Verhalten entspricht dem offenbar. Was heisst das:

Ein Bearbeiten einer Institution ändert den Sortierwert nicht (z.B. ändern der Telefonnummer, Notiz hinzu). Super!
Ein Ändern des Bibliotheksnamens (also des Sortierwerts) hat denselben Effekt, der Sortierwert bleibt erhalten. Schade.

Nun könnte man natürlich (theoretisch, die Frage der Machbarkeit überlasse ich @maechler) nach dem Ändern dieses Felds eine Sortieraktion in allen Views auslösen. Ev. ist das etwas dick aufgetragen. Bibliotheksnamen ändern nicht alle Tage, ich könnte auch mit der Anleitung leben: Ändert man den Namen, muss man manuell neu sortieren.

mschwendener · 2015-12-01T11:40:55Z

Längerfristig gedacht müsste es eine Lösung sein, die auch dann noch funktioniert, falls wir eine Weboberfläche haben, auf der die Bibliotheken ihre eigenen Angaben ändern können. Den Anzeigenamen in swissbib sollen sie aber sowieso nicht ändern können, d.h. Feld "Display Namen" würden wir weiterhin selbst verwalten und können Sortierung von Hand auslösen.

guenterh · 2015-12-01T17:14:41Z

@maechler @mschwendener @oschihin

Hallo zusammen,
Manu: wenn ich Deine Comments richtig interpretiere, entspricht die getestete Funktionalität der der produktiven Instanz. Konkret: Es gehen keine Informationen bei Mutationen mehr verloren (wie in diesem issue zu Anfang beschrieben) und Du kannst Informationen korrekt hinzufügen oder bewusst löschen.

an alle: ich bin dagegen, jetzt die Machbarkeit von wünschbaren Sortierfunktionen zu prüfen. Mein Standpunkt:

wir sollten uns darauf konzentrieren, jetzt wieder eine stabile Applikation auf Basis der ursprünglich von snowflake implementierten Funktionalität zu erhalten, die als Basis für weitere Ergänzungen dienen kann
wenn 1) erreicht, können wir die Erweiterung von Eweiterung um zusätzliche Felder #7 angehen. Danach werde 'ich' vorerst keine Arbeit mehr für Libadmin investieren. Auch Markus sollte das nicht machen (müssen)

Morgen werde ich noch die API von libadmintest testen. Wenn das ok ist, bereite ich ein brauchbares Deployment vor, räume die bestehenden Branches auf (einschliesslich Merge einer Formeta-API für Metafacture) und verteile ein Release als produktive Version

Günter

maechler · 2015-12-01T21:17:24Z

Hallo Zusammen,

Zur Sortierung:
Die Sortierung ist nicht rein alphabetisch. Es hat zwar einen Knopf für die alphabetische Sortierung, man kann die Institutionen aber auch beliebig von Hand sortieren. Somit kann man bei der Änderung des Namens die Institution auch nicht einfach neu alphabetisch einsortieren.
Wäre eine rein alphabetische Sortierung gewünscht, könnte man das wohl ohne allzu viel Aufwand aus der View entfernen und die Institutionen einfach vor der Rückgabe an VuFind über die API sortieren.

mschwendener · 2015-12-02T08:12:02Z

"Wäre eine rein alphabetische Sortierung gewünscht"
Ja, wäre sie (also jeweils innerhalb einer Gruppe, s. https://www.swissbib.ch/Libraries) - inkl. korrekter Sortierung von Sonderzeichen.

"Es gehen keine Informationen bei Mutationen mehr verloren"
Situation auf dem Screenshot von osc (#4 (comment)) ist jedenfalls geflickt.

guenterh · 2015-12-07T15:58:21Z

API auf testvf gegen admin.swissbib.ch/libadmintest gestetet
TestCase

Ändere link zu Bibliotheksinformationen im deutecsh Kontext auf www.woz.ch
wurde nach Import korrekt übernommen

guenterh · 2015-12-07T16:04:40Z

Mapportal API gegen libadmintest ok
getestet auf sb-vf4

guenterh · 2015-12-07T17:29:06Z

Deployment von https://github.com/swissbib/libadmin/releases/tag/R1.0 für libadmin und libadmintest
Anpassung der scripte für libadmin in deployrep
@witzigs : Bitte Ergebnis der API Nutzung in der kommenden Nacht für Basel / Bern überprüfen
Danke! Günter

ghost assigned maechler Jan 30, 2014

oschihin mentioned this issue Jan 30, 2014

Bibliotheksnamen: Sortierung der Institutionen in Vollanzeige und Trefferliste swissbib/sbvf2#107

Closed

maechler pushed a commit that referenced this issue Mar 19, 2014

#4 Bugfix: only reset sorting when institution relations get changed

8a9b021

maechler pushed a commit that referenced this issue Mar 27, 2014

#4 Bugfix: empty groups and views on institution save

dac8cc9

maechler pushed a commit that referenced this issue Mar 28, 2014

#4 Bugfix: improved implementation

59dacd3

maechler pushed a commit that referenced this issue Apr 3, 2014

#4 Bugfix: Fix the zendframework version to 2.2.5

4f411a8

guenterh closed this as completed Dec 7, 2015

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sortierung in View nach Bearbeitung einer Institution #4

Sortierung in View nach Bearbeitung einer Institution #4

oschihin commented Jan 30, 2014

maechler commented Mar 19, 2014

oschihin commented Mar 21, 2014

maechler commented Mar 21, 2014

maechler commented Mar 27, 2014

oschihin commented Mar 27, 2014

maechler commented Apr 3, 2014

mschwendener commented Apr 3, 2014

guenterh commented Apr 3, 2014

maechler commented Apr 3, 2014

oschihin commented Apr 3, 2014

guenterh commented Nov 24, 2015

mschwendener commented Dec 1, 2015

oschihin commented Dec 1, 2015

mschwendener commented Dec 1, 2015

guenterh commented Dec 1, 2015

maechler commented Dec 1, 2015

mschwendener commented Dec 2, 2015

guenterh commented Dec 7, 2015

guenterh commented Dec 7, 2015

guenterh commented Dec 7, 2015