-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add copy row #18
Comments
Do I see it correctly that since version 3.4.0 only one row can be edited? Since then I always get an error when I want to add a new row. Field-Config: array(
'label' => &$GLOBALS['TL_LANG']['tl_mydca']['myField'],
'inputType' => 'multiColumnWizard',
'eval' => [
'tl_class' => 'w50 clr',
'dragAndDrop' => false,
'columnFields' => [
'from' => [
'label' => &$GLOBALS['TL_LANG']['tl_mydca']['from'],
'inputType' => 'select',
'options_callback' => array('tl_dcaclass', 'listFields'),
'eval' => ['style' => 'width:100%']
],
'to' => [
'label' => &$GLOBALS['TL_LANG']['tl_mydca']['to'],
'inputType' => 'select',
'options_callback' => array('tl_dcaclass', 'listFields'),
'eval' => ['style' => 'width:100%']
],
],
],
'sql' => 'blob NULL',
); Contao Version: 4.8.7 Is it possible to say when the bug will be fixed? |
Schade, dass ihr das bei Contao 4.9 rausgenommen habt. Funktionierte doch noch in Contao 4.4?! |
das hat mit Contao nicht direkt was zu tun - die Funktion ist mit der Umstellung au ein Bundle raus ... nun kommen die neuen Zeilen per Ajax und nicht per "JavaScript"... daher ist die Initialisierung einer neuen Reihe nicht trivial |
"Ajax" in dem Sinne gibt es doch gar nicht. Beruht doch auch auf Javascript. Ich hoffe nur ihr werdet nicht auf diese extrem nervige Erweiterung "Haste" als Basis setzen. Die bringt so viele sinnlose Dateien mit für ein bisschen Ajax Gedöns, das ich in 30 Zeilen Code selbst schreibe. |
schon klar, deshalb habe ich ja auch JavaScript in "" geschrieben... in der alten Erweiterung wurde die Zeile direkt in JS kopiert bzw. eine leere erzeugt - mit vielen Problemen und Anpassungen, wenn sich in C mal was ändert... nun kommt eine neue Zeile sozusagen immer direkt von C per Ajax - bei einer Kopie, müssten die Infos erst zu C dann neue Zeile zurück - und das ist nicht trivial... An dem Thema kann sich aber auch jeder selbt beteiligen - da öffentliches Projekt :D |
Danke für die Erklärung. Hmmm das stellt leider ein größeres Problem dar, da es ein Kunde unbedingt wieder haben will und nun die ganze Umstellung von Contao 3.5 auf Contao 4.9 in Frage stellt. Ich kanns irgendwo verstehen... wer will eine neuere Version, die ihm persönlich nur Nachteile bringt? Kann man das manuell (update unsicher) wieder zurückbauen auf die alte Version? Wenn ja, welche Datei müsste man da anpassen? |
@Total-Reality Bitte bis zu ende lesen. Wir haben eine der "größten Nachteil" aus der Welt geschafft, dass nämlich die Initialisierung von TinyMce, Picker und andere JS Sonderfunktionen nicht funktionierten. Da nur Contao weiß wie die Widgets richtig gebaut werden und wie das JS richtig zusammengebaut wird, wurde die Erstellung einer neuen Zeile wieder an Contao übergeben. Daher wird beim Erstellen einer neuen leeren Zeile eine AJAX an Contao gesendet (via MCW als Controller) und als Antwort kommen dann die richtige Daten zurück. Die Kopierfunktion mit dem alten MCW (nicht Bundle) funktionierten noch, davon abgesehen, dass das JS und die Sonderfunktion zu jederzeit kaputt gehen können. Mit anderen Worten, wir konnten uns aussuchen was uns mehr stört, dass wir Daten in einem Funktionierenden Widget erneut eingeben müssen oder das wir Daten im Widget stehen haben, diese aber nicht mehr ändern können, weil das Widget wegen einem JS Error nicht mehr reagiert. @Total-Reality du siehst also, dass wir hier zwei Möglichkeiten hatten. Wir haben uns für die Sicherheit der Core Funktionen gegenüber einer Komfortfunktion entschieden. Ein Zurückpatchen kann funktionieren, wir werden für die "alte" Version aber keine Bugfixes mehr bereitstellen, da dies bedeuten würde, dass wir viele Änderungen von Contao direkt in die Core Klassen oder das JS von MCW übernehmen müssten und für jede Contao Version immer wieder neu anpassen müssten. Das wäre mit zu viel Arbeit verbunden und würde viele sondern Funktionen und Versionsprüfungen im System bedeuten. Dass das "die ihm persönlich nur Nachteile bringt?" leider immer von Person zu Person unterschiedlich ist, ist mir klar, aber ich hoffe du verstehst nun besser warum wir diesen Weg gegangen sind. Wenn es dringend ist kannst du eine Anfrage an die Werbeagentur Men at Work stellen - contao@men-at-work.de Ich kann es dir nicht zu 100% versprechen, ich würde aber zumindest einmal in das System schauen, ob es eine Möglichkeit gibt, das Kopieren iwie wieder einzubauen, ohne einem extreme großen Aufwand. Im Moment wäre die erste Idee, dass die Zeile, welche kopiert werden soll, an Contao via AJAX gesendet wird und validiert wird. Wenn die Validierung nicht funktioniert, würde das Kopieren mit einem Fehler abbrechen. Wenn alles durchläuft, könnte ich dann versuchen, die Nutzdaten in eine neue Zeile einzubauen und Contao die Widgets und JS Daten dafür erstellen zu lassen. Aber dafür müsste ich erst einmal prüfen, ob dies überhaupt Möglich ist und wo wir überall anecken. |
See PR #89 for a WIP version. |
Die erste Version der Funktion steht. Damit ist das PoC fertig. Es geht, ich muss mich hier auch bei @zonky2 bedanken, mit dem ich das Problem besprochen habe und wir die Basis einer Möglichen Umsetzung gesetzt haben. Wir haben einen neuen PR dafür aufgemacht, es müssen noch die Widgets geprüft werden. Wie ich mir schon dachte, muss ich einige Sachen anpassen, damit ich wieder mit Contao im Einklang bin. Aber es scheint sich dabei nur um einige kleinere Sachen zu handeln. |
add at #89 |
Vielen Dank für deine Zeit und die genauen Erklärungen. Das ist natürlich alles total verständlich und wirklich besser, dass ihr den Weg jetzt so gewählt habt. |
Gibt's schon Neuigkeiten zu dem Thema? Der PR war doch schon ziemlich gut. |
hat keiner
|
Es war doch aber schon programmiert? Z.B. " 'copy' => 'copy.gif'," @zeile 152 Aber in der neuesten Version ist nicht mehr drin? Irgendwie versteh ich die Logik von GitHub nicht, was übersehe ich? Welches Budget wird denn benötigt? |
Aktuelle Entwicklung ist noch hier https://github.com/menatwork/contao-multicolumnwizard-bundle wir ziehen erst mit MCW 4 zur CCA komplett um bei Kosten müsste @stefanheimes mal ein Statement abgeben |
…php8 Fixed several PHP8 warnings
add copy row
The text was updated successfully, but these errors were encountered: