MetaModels / attribute_alias

The alias attribute
GNU Lesser General Public License v3.0
0 stars 6 forks source link

Wenn Pflichtfeld, keine Erstgenerierung #4

Closed Tastaturberuf closed 11 years ago

Tastaturberuf commented 11 years ago

Wenn der Alias als Pflichtfeld leer ist, kommt nur die Meldung das man es ausfüllen soll. Ohne Pflichtfeld klappt alles wunderbar. Da der Alias nicht leer sein kann ist das Pflichtfeld eh obsolet, es sollte immer generiert werden wenn es leer ist.

Oder ist das Verhalten gewünscht?

discordier commented 11 years ago

Das Verhalten ist logischerweise nicht gewynscht. Aus diesem Grunde fangen wir im Alias auch genau diesen Fall ab. Ich konnte das Problem bei mir nun auch nicht nachstellen, kannst du uns evtl. deine Konfiguration etwas erlaeutern?

Screenshots der Palettenconfig des Alias Attributswaeren hier wynschenswert.

Tastaturberuf commented 11 years ago

palette attribut BE palette

Sobald man den Haken bei Pflichtfeld rausnimmt, läuft alles wie gewünscht.

andreasisaak commented 11 years ago

Sollte hiermit behoben sein: https://github.com/MetaModels/attribute_translatedalias/commit/cb44241b86839b7e7df7643785efe8d9b61d2010

andreasisaak commented 11 years ago

Zu früh gefreut

discordier commented 11 years ago

Koennte behoben sein, jedoch gibt es definitiv noch ein Problem beim saveNClose.

@stefanheimes, kannst du dir das bitte ansehen warum der movelSaved() call bei saveNclose nicht mehr kommt?

discordier commented 11 years ago

@stefanheimes ist das Problem mit dem alwayssave behoben?

stefanheimes commented 11 years ago

Ich hab die Alias Attribute gerade aktualisiert. Wenn force_alias aktiviert ist, wird das alwaysSave auch aktiviert. Somit wird immer das Model gespeichert.

Ich hab es auf meinem Testsystem geprüft sieht gut aus. Muss aber auf Live Systeme noch geprüft werden.

Tastaturberuf commented 11 years ago

Das eigentliche Problem ist nach wie vor vorhanden. Wenn Pflichtfeld true und das Feld ist leer, kommt der Hinweis das es ein Pflichtfeld ist.

andreasisaak commented 11 years ago

@Tastaturberuf Ist der Fehler auch bei dir behoben? Ich schließe das Ticket weil ich es mit dem dev Branch nicht mehr nachstellen kann. Wenn ich was übersehen habe, kurz melden und wir schauen es uns zum dritten Mal an ...

Aybee commented 9 years ago

Ich muss mich hier nochmal dran hängen, weil ich da noch ne Kleinigkeit gefunden habe.

Alias aus Textfeld erstellt

Eingabemaske

Ich hatte zuerst vergessen ein Alias-Feld anzulegen. Jetzt habe ich schon Datensätze ohne Alias.

Jetzt muss ich im Prinzip jeden Datensatz zum Bearbeiten öffnen, den Wert im Textfeld ändern, speichern, wieder zurück ändern und wieder speichern und schließen.

Ist ein ganz simples MetaModel für ne Taxonomie mit nur einem Textfeld und einem Alias.

dev-tng @ 8ceb31 - 2015-01-29 17:36:33 auf Contao 3.4.5

discordier commented 9 years ago

Das ist nichts was MM abfangen kann, da der Datensatz an sich ja nicht gespeichert wird. Das faengt der DCG schon ab und somit "weiss" MetaModels gar nicht, dass du speichern wolltest (weil effektiv ja nicht gespeichert wird).

Aybee commented 9 years ago

Ah, ich verstehe, also findet vor einer Speicherung eine Prüfung statt, ob sich überhaupt etwas am Datensatz geändert hat. Wenn nicht, wird auch nicht gespeichert.? Und wenn in dem alias Feld z.B. NULL oder sowas drinn stehen würde, dann müsste es doch eigentlich gehen, oder? Na ja, war ja nur ne Ausnahmesituation.

ps Ich glaube so ein Feature, dass man Aliase neu generieren kann, wurde früher schon mal angefragt. Das würde sich für den Fall anbieten, wenn man die Datensätze aus einer CSV Datei eingespielt hat.

pps Ich hatte vor ein paar Tagen auch mal einen Datensatz, wo ich auf Teufel komm raus keinen Alias abgespeichert bekam. Ich habe den dann manuell direkt in die Datenbank eingetragen. Wenn mir das nochmal unterkommt, mache ich aber ein extra Ticket dazu auf.