Get the variants working #768
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
discordier
added a commit
that referenced
this pull request
Mar 3, 2015
Rewrite create variant button and model duplication to event subscriber.
|
Looks good - merged. |
|
Hi there, doesn't work with for me with contao 3.4.4 The first time after the update I get the button "Neuer Datensatz" then I get the clipboard-button and after pushing the button I get the following error: Any idea? |
|
@cptshit Hm der Fehler kommt anscheinend nun woanders her (sorting Property) - hast du eine Standard Sortierung / Gruppierung in der Eingabemaske angelegt? |
|
Jepp, Sortierung ist angelegt:
Habe die Sortierung gelöscht, Fehler tritt trotzdem auf. |
|
@tim-bec Kann ich irgendwie helfen? Testen, Installation anlegen, verschiedene Setups erstellen? Das Thema ist mir wichtig und ich würde mich (bei meinen beschränkten Fähigkeiten) gerne einbringen. |
|
Hatte gleiches Problem. Wenn ich beim Sorting eine Definition mit Häkchen bei Enable maual sorting setze erscheint der Fehler nicht mehr. |
|
So. Ja, dadurch verändert sich was. Grundsätzlich kann ich jetzt zwar Varianten anlegen, aber es tritt noch folgender Fehler auf, wenn ich einen neuen Datensatz anlegen will: Wenn ich dann das MM neu lade, ist der Datensatz angelegt. Es läuft also grundsätzlich, aber suboptimal. Ohne Fehlermeldungen krieg ich keinen neuen Datensatz hin. |
|
@cirby83 Wenn Du eine Variante erstellst: Musst Du dann auch erst den Text der "Mutter"-Variante löschen? |
|
Ja das ist momentan bei mir auch so. Aber bei mir sind das nur zwei Textfelder. Also kein Problem bei meinem Projekt. |
|
@discordier Ich teste mit Contao 3.4.4 Ist das ein Problem? Denn die Fehlermeldung verschwindet nicht. Ich habe die Varianten "aus- und wieder angeschaltet", unter Systemwartung die Daten bereinigt, aber das Problem bleibt: Fehlermeldung kommt, aber Datensatz wird leer angelegt. Varianten lassen sich fast problemlos anlegen, einziger Wermutstropfen: Jede neue Variante öffnet sich nicht "leer", sondern enthält schon den Text der "Mutter"-Variante. |
|
Das ist so gedacht, denn sie ist ja eine Variante des Hauptelements. |
|
Alles klar, verstehe ich. Bleibt die Fehlermeldung: |
|
@discordier Kannst Du noch neue Datensätze (keine Varianten) anlegen? Bei mir ist es (auf einmal?) nicht mehr möglich... Hä? |
|
In meinem Testsystem habe ich hier keine Probleme... hast du yberhaupt ein sorting definiert? |
|
Koennte sein dass du hier in contao-community-alliance/dc-general#93 rennst. |
|
@discordier Hatte tatsächlich kein sorting eingerichtet. Danke. Jetzt kommt folgende Fehlermeldung: Und wenn ich dann im Browser-History einen zurück springe, ist ein neuer, leerer Datensatz eingerichtet. Was aber praktisch nie wieder erscheint ist der Button "Neuer Datensatz". |
|
Genau der sollte doch im aktuellsten MM Core behoben sein....? |
|
Hmmm, ich habe MM Version (8f3a486) installiert. Habe eben auch noch mit dem Sortier-Modus experimentiert, aber manuelles Sortieren und die Default-Einstellung sind notwendig, ansonsten erhalte ich eine Fehlermeldung OHNE neue Datensatz. U.U. habe ich in den tiefen des Systems einen Killer? Ich habe ein Rocksolid-Theme installiert. Was kann denn vielleicht sonst noch ins MM grätschen? Soll ich Dir einen Zugang legen :-) |
|
Sollte nun klappen seit 0b2f164. |
|
Es tut mir leid. Muss ich noch was am Sorting ändern? Es bleibt der Fehler "ToolboxFile.php on line 714". Habe jetzt mal alle unnötigen Erweiterungen rausgekegelt, es hilft (wie ja fast zu erwarten) nichts. Habe das "bundle_all" installiert. Sollte ich da händisch etwas nachziehen? Kommt auf die Fehlerseite, ohne sich anzumelden? http://wortreich.eu/new/contao/main.php?do=metamodel_mm_mymusic&act=paste&into=mm_mymusic::0&children=Array&rt=ea01d52ea17423c09b7c2e5ab1b0e695&ref=c14b6870 |
|
Contao 3.2.19 Habe noch das Problem keine neuen Datensätze anlegen zu können oder nur über Umwege. Bei nicht manueller Sortierung geht das Anlegen gar nicht (siehe auch #731). Wenn ich eine manuelle Sortierung verwende, kommt beim Anlegen eines neuen Datensatzes: wenn ich zurückgehe bleiben die Clipboardsymbole und ein leerer Datensatz ist angelegt: aber das Anlegen funktioniert. Obwohl ich als render mode "flat" angegeben habe, sieht die Liste wie "hierarchical" aus (Kopfzeile mit Icon). Bei manueller Sortierung fehlen die grünen Sortiericons. |
|
Dito |
|
Die Varianten sind auch ein tree view, wir haben leider bislang noch keinen eigenen render mode hierfyr. Das invalid file array sollte sich mit MetaModels/attribute_translatedfile@3132ddb erledigt haben. Das missing argument ist jedoch noch ein bug der behoben werden muss. |
discordier
added
bug
|
Moin, |
|
Leider nein: Beim Anlegen eines neuen Artikels (manuelle Sortierung): Beim Anlegen mit Sortierung nach Artikelnummer: |
|
Ist da eine Lösung in Sicht? |
|
Nach Update von dev-tng nach 2.0.x funktioniert es. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some fixes and refactoring to get the variants working, see: #763
@discordier Can you review it as fast as possible, we need the working variants.