search

yakamara / redaxo_xform

XForm für redaxo4
22 stars 21 forks source link

Einstellung der Standard-Sortierung #134

Closed NGWNGW closed 10 years ago

NGWNGW commented 11 years ago

Einstellung der Standard-Sortierung: Ist das gewünschte Feld nicht vorhanden, wird Fallback auf id aktiv.

NGWNGW commented 11 years ago

Irgendwo ist noch ein kleiner Fehler. Ich kümmere mich darum...

NGWNGW commented 11 years ago

In der class.rex_xform_manager.inc.php müsste zusätzlich die Zeile 138:

(in_array($fieldarray)) ? $sortfield = $table['list_sortfield'] : $sortfield = 'id';

durch

(in_array('`'.$table['list_sortfield'].'`',$fieldarray)) ? $sortfield = $table['list_sortfield'] : $sortfield = 'id';

ersetzt werden.

Soll ich einen neuen Pullrequest schicken oder würdest du die Änderung übernehmen? (Falls du überhaupt mergen willst ;) )

Gruß Nico

dergel commented 11 years ago

Finde ich sehr gut .. Ein Problem sehe ich noch. Updates. D.h. wir müssten irgendwie schauen, wie das, z.B. beim Aufruf der XForm Verwaltung, die Tabellenspalten geprüft und eventuell neu angelegt werden.

NGWNGW commented 11 years ago

Wie genau meinst du das? Falls beim Update der Tabellenstruktur das Feld nicht mehr existiert, gibts nen Fallback.

dergel commented 11 years ago

ich meine diese beiden list_sortfield list_sortorder Tabellenfelder. Wenn ich eine XForm bereits habe, und z.B. reinstalliere, dann werden diese Felder nicht erstellt. Deswegen würde ich vorschlagen, dass du in Deinem PR noch in der manager install diese felder rausziehst, so dass diese, falls nicht vorhanden, nachträglich eingetragen werden.

sowas in der Art: $sql->setQuery('ALTER TABLE rex_xform_table ...... list_sortfield varchar(100) NOT NULL,

NGWNGW commented 11 years ago

Okay, ich habs in einem zusätzlichen Pull-Request erweitert.

tbaddade commented 10 years ago

:+1: bitte zeitnah übernehmen :)