ow_randomnews macht keinen Zufall

[rusty]

Die Erweiterung ow_randomnews soll eine zufällige News ausgeben. Obwohl sie nur bis Contao 3.0.6 freigegeben ist, versuche ich sie unter 3.1.2 einzusetzen.

Problem: Sie gibt im FE immer die gleiche Nachricht aus, nämlich die erste. Man kann es bereits im Backend an der Vorschau des CE erkennen:
Bildschirmfoto 2013-09-29 um 13.26.37.png

Hier ist immer {{news_open::1}} zu sehen, die Zahl ändert sich nie.

Liegt es an der fehlenden Kompatibilität mit Contao 3.1.x?

[magicsepp]

Hi rusty,

ich habe es mit der 3.1.3 getestet...

in der Musik Academie wechselt die news_open::id wenn ich die Anzeige im Browser aktualisiere.

Wie ist den das Modul im Frontend eingebaut direkt als Modul oder auch per inserttag ?

{{insert_module::*}}

wobei * durch die Module Id von randomnews zu ersetzten ist.

[rusty]

Ich habe das Modul als Content-Element/Typ Modul eingefügt.
Wenn ich es per Insert-Tag einsetze, verhält es sich genau so: Es wird immer die selbe Nachricht gezeigt, obwohl mehrere im angegebenen Archiv vorhanden sind.

[magicsepp]

da ist noch ein Fehler in der Erweiterung bei nur einem Archiv wird nicht mehr gewürfelt

Der Schutz von Nachrichten wird auch nicht berücksichtigt.

[rusty]

... und bei mehreren Archiven aus jedem Archiv jeweils immer die erste Nachricht.

Habe das von mir angestrebte Ziel inzwischen mit der Erweiterung dk_caroufredsel erreicht, die auch eine Newsticker-Variante sowie eine Zufallsfunktion bereit hält.
Danke fürs Mitdenken und Testen!

[buzzjoe]

Die Lösung des Problems:

./system/modules/ow_randomnews/classes/ow_randomnews.php, Zeile 51:

$objNews = $this->Database->prepare("SELECT id, headline, teaser, addImage, singleSRC, size, caption FROM tl_news WHERE pid = ? AND published = 1") 

Das SQL-Statement um folgende Anweisung ergänzen:

ORDER BY RAND()

Das sieht dann so aus:

$objNews = $this->Database->prepare("SELECT id, headline, teaser, addImage, singleSRC, size, caption FROM tl_news WHERE pid = ? AND published = 1 ORDER BY RAND()") 

Da hat der Entwickler schlicht und einfach die Zufallsfunktion vergessen

[zonky]

... hatte mich auch gewundert, dass nix random - nach einem Blick in den Quelltext habe ich das auch gesehen ... man könnte noch ein "LIMIT 1" ansetzen

        $objNews = $this->Database->prepare("SELECT id, headline, teaser, addImage, singleSRC, size, caption FROM tl_news WHERE pid = ? AND published = 1 ORDER BY rand() LIMIT 1")
            ->execute($newsarchiv); 

... frage mich, wieso eine Erweiterung dann random heißt??

[grashalm]

@ buzzjoe: herzlichen dank für deinen super tipp und die genaue anleitung der einfachen lösung. jetzt funktionieren meine ow_randomnews endlich einwandfrei!

hatte das modul schon vor jahren dreimal untereinander bei mir eingebunden und in jedem modul jeweils 10 andere newsarchive aktiviert. da er unter den archiven selbst schon zufällig auswählt (oder ging er der reihe nach???) waren bei jeder seitenaktualisierung 3 andere news verlinkt, aber diese haben sich überdurchschnittlich häufig wiederholt, so dass ich immer den eindruck hatte, er wählt in jedem modul aus nur rund 5 bis 6 news aus (in wahrheit waren es wahrscheinlich genau diese 10 verschiedenen -aber jeweils genau ersten news- der 10 verschiedenen archive! und ich dachte immer: warum fangen fast alle newslinks mit einem "A" an, aber eben nicht alle??? bin natürlich nie darauf gekommen, dass er immer nur den ersten newstext auswählt und somit quasi gar keine zufallsfunktion enthalten hat. erklärung jetzt: in vielen meiner archive beginnt der erste newstext mit einem "A". jetzt habe ich endlich erklärung und lösung. vielen lieben dank noch mal an alle, die hier mitdiskutiert haben...

@ zonky: ich habe als wenig script-kundiger noch eine frage: was bewirkt deine ergänzung "LIMIT 1"? bedeutet das, dass er pro modul nur einen newstext auswählen soll?

danke dir für eine kurze antwort!

grashalm

[Samson1964]

@ zonky: ich habe als wenig script-kundiger noch eine frage: was bewirkt deine ergänzung "LIMIT 1"? bedeutet das, dass er pro modul nur einen newstext auswählen soll?

Ich bin nicht zonky, aber LIMIT 1 bedeutet, daß die Anzahl der zurückgegebenen Datensätze genau auf einen beschränkt wird, nämlich den ersten. Ohne diese Angabe würde die Liste aller veröffentlichten Nachrichten in einer Zufallsreihenfolge zurückgegeben werden.

[zonky]

Hi grashalm, guck mal ins MySQL-Manual... ja, Limit begrenzt die Ausgabe auf das erste Item...

weil die Fragestellung so war (siehe #5)

... und bei mehreren Archiven aus jedem Archiv jeweils immer die erste Nachricht.

[grashalm]

@ Samson1964:

Ich bin nicht zonky, aber LIMIT 1 bedeutet, daß die Anzahl der zurückgegebenen Datensätze genau auf einen beschränkt wird, nämlich den ersten. Ohne diese Angabe würde die Liste aller veröffentlichten Nachrichten in einer Zufallsreihenfolge zurückgegeben werden.

das wundert mich, denn ich habe kein "LIMIT 1" ergänzt, und trotzdem zeigt er mir für jedes eingebettete modul nur einen datensatz an, also genau so, wie ich es auch haben will. wenn ich dich richtig verstehe, würde LIMIT 1 quasi den effekt künstlich erzeugen, den wir vorher ohne den random-befehl hatten, nämlich dass er aus jedem archiv immer nur den allerersten datensatz ausliest. das würde ja dann bedeuten, dass der befehl ORDER BY RAND () von LIMIT 1 gleich wieder aufgehoben wird. oder habe ich da was falsch verstanden? jedenfalls benutze ich derzeit den befehl LIMIT 1 nicht, und er macht genau das, was er soll, nämlich einen datensatz aus mehreren archiven mit mehreren datensätzen zufällig auszuwählen.

@zonky:

weil die Fragestellung so war (siehe #5)

ja, kann sein, dass ich sie falsch interpretiert habe... ich hatte rusty so verstanden, als wolle er sagen, dass der effekt gerade unerwünscht sei, so, wie er auftritt, und nicht erwünscht.

jedenfalls danke für eure beiträge,

grashalm