yakamara / redaxo_xform

XForm für redaxo4
22 stars 21 forks source link

Feld anlegen :: Name darf nie "id" sein #197

Open tbaddade opened 10 years ago

tbaddade commented 10 years ago

Wenn man "id" beim Feldname eingibt sollte ein Fehler wie "Feldname reserviert" kommen.

hirbod commented 10 years ago

Wenn man da schon eine Abfrage einbaut, würde ich auch gleich alle aus der Liste der Reserved-Words von MySQL mit reinnehmen. Hatte vor kurzem ein Problem das ich reservierte Bezeichnungen von MySQL genutzt hatte und ständig Fehlermeldung erhielt. Nach dem ich gegoogelt hab, fand ich heraus, das ich diese beim SELECT in Anführungszeichen angeben musste.

http://dev.mysql.com/doc/refman/5.5/en/reserved-words.html

Nur so ne Idee...

dergel commented 10 years ago

finde ich auch .. neben id wollte ich auch noch gerne ein paar weitere xform manager felder reservieren. ich schreibs mal hier rein, damit ich es nicht vergesse ;) ..

xform_online_datetime xform_offline_datetime xform_published xform_updated_user xform_created_user xform_updated_datetime xform_created_datetime

die MySQL reservierten Felder sollten "eigentlich" auch gehen wenn sie richtig gequoted werden. Es kann sein, dass bei den älteren Version, dass noch nicht richtig beachtet wurde. Wenn es bei der aktuellen auch nicht geht, gerne hier als Issue mit dem entsprechenden Beispiel.

tbaddade commented 10 years ago

xform_updated_user xform_created_user xform_updated_datetime xform_created_datetime

wäre es nicht sinnvoller die REDAXO Felder zu nutzen

updateuser
createuser
updatedate
createdate

da könntest einfach diese 2 Methoden verwenden und die Felder werden befüllt

https://github.com/redaxo/redaxo4/blob/master/redaxo/include/classes/class.rex_sql.inc.php#L916-L938