search

FriendsOfREDAXO / feeds

Importiert Daten aus API-Quellen wie YouTube, Vimeo, RSS und mehr
https://friendsofredaxo.github.io/
MIT License
45 stars 15 forks source link

Ausgabe-Sortierung geändert #157

Closed tyrant88 closed 4 years ago

tyrant88 commented 4 years ago

damit nach "date" und nicht nach "updatedate" sortiert wird.

In der Beschreibung der Funktion getPreloadedItems() heißt es: "...orderd by date." Es wird aber nach updatedate sortiert.

alxndr-w commented 4 years ago

Plausibel. Ist denn in jedem mitgeliefertem Stream immer ein date-Feld ausgefüllt? Sollte man ggf. nach date, updatedate sortieren?

tyrant88 commented 4 years ago

rex_sql_table::get(rex::getTable('feeds_item')) ->ensureColumn(new rex_sql_column('date', 'datetime')) Das Feld ist zumindest da... Eventuell die Funktion erweitern um einen Parameter "orderBy = date" ?

alxndr-w commented 4 years ago

Ich denke, es würde schon reichen order by date DESC, updatedate DESC einfach beides reinschreiben. Für facebook/twitter/instagram habe ich es mir gerade angeschaut und da passt alles, es wurde immer date befüllt. Ich nehme an, dass dies auch bei anderen Streams so ist.

tyrant88 commented 4 years ago

Ich würde das nicht machen, weil dann das Editieren im REDAXO die Sortierung beeinflusst und ich das nicht abstellen kann.

tbaddade commented 4 years ago

Ich nehme an, dass dies auch bei anderen Streams so ist.

Davon sollte man nicht ausgehen. Das mag für die von Feed bereitgestellten Streams gelten, aber für eigens für das Projekt erstellte Streams mag es schon anders aussehen.

alxndr-w commented 4 years ago

Ich würde das nicht machen, weil dann das Editieren im REDAXO die Sortierung beeinflusst und ich das nicht abstellen kann.

So war es doch bisher auch? @tyrant88

Davon sollte man nicht ausgehen. Das mag für die von Feed bereitgestellten Streams gelten, aber für eigens für das Projekt erstellte Streams mag es schon anders aussehen.

Was schlägst du daher vor? @tbaddade

tbaddade commented 4 years ago

Ich weiß nicht wofür die Methode genutzt wird, ich wollte nur auf den Umstand hinweisen, dass nicht jeder Stream ein Datum liefern kann.

tyrant88 commented 4 years ago

So jetzt mit orderBy Parameter