GX-Bug #49557
geschlossen
Löschen aller Eigenschaftenkombinationen für einen Artikel funktioniert nicht, wenn es zu viele Kombinationen gibt, die zum Löschen vorher alle ausgewählt werden.
100%
Löschen aller Eigenschaftenkombinationen für einen Artikel funktioniert nun auch, wenn es zu viele Kombinationen gibt, die zum Löschen vorher alle ausgewählt werden.
Beschreibung
Wenn man in der Eigenschaftenbearbeitung eines Artikels alle Kombinationswerte auswählt und auf den Button "Kombinationen löschen" klickt, wird eine Lightbox geöffnet. Die Lightbox erscheint, weil dazu ein Request an den LightboxPluginAdminAjaxHandler gesendet wird. Mitgesendet werden per GET alle ausgewählten Kombis. Bei vielen Kombis wird das Limit für die maximale Länge einer Url überschritten, so dass alle Parameter nach den Kombi-Ids nicht beim Server angekommen. Das führt dazu, dass dann in der Lightbox der Löschen-Button keine Funktion hat.
Lösungsvorschlag:
schnell: Die ID-Parameter an das Ende der Url schreiben, damit die für die Funktion relevanten Parameter auf jeden Fall ankommen
gut: Die IDs per POST übertragen, damit sicher alle an den Server übermittelt werden
Von Mirko Janssen vor mehr als 6 Jahren aktualisiert
- Zugewiesen an wurde auf Mirko Janssen gesetzt
Von Mirko Janssen vor mehr als 6 Jahren aktualisiert
- Status wurde von Gemeldet zu Testing geändert
- Zugewiesen an wurde von Mirko Janssen zu Core-Entwickler geändert
- Zielversion wurde von 133 zu 3.9.1.0 beta1 geändert
- Release Notes Langtext aktualisiert (Vergleich)
Von Mirko Janssen vor mehr als 6 Jahren aktualisiert
Die Methode zum Laden von Templates wurde generell von GET auf POST umgestellt. Das Lightbox-Plugin sollte daher generell getestet werden.
Im bestehenden Code gab es nur 3 Ajax-Calls an die LightboxPluginAdminAjaxHandler-Klasse, welche von mir angepasst wurden. (Vielleicht auch nochmal schauen, falls ich doch was übersehen habe.)
Von Christian Schmidt vor mehr als 6 Jahren aktualisiert
- Zugewiesen an wurde von Core-Entwickler zu Christian Schmidt geändert
Von Christian Schmidt vor mehr als 6 Jahren aktualisiert
- Status wurde von Testing zu Erledigt geändert
- % erledigt wurde von 0 zu 100 geändert