On-/Offline für Module/Slices

[Fabchan]

Danke für den Hinweis.

Ich nehme an, der Fehler mit der Routine zusammen, die den Artikel neu generiert. Konnte den Fehler auch nachvollziehen, werde mir das mal anschauen, und versuchen, es zu korrigieren.

[Fabchan]

Okay, ich weiß jetzt, woran es liegt: Wenn ein Artikel im Backend aktualisiert wird, wird der Extension-Point SHOW_SLICE, an welchen sich mein Addon ankoppelt bei der Generierung nicht aufgerufen. Weiß im Moment noch nicht, wie ich das Problem lösen soll, werde mir darüber in den nächsten Tagen mal Gedanken machen oder hat vielleicht sogar jemand eine Idee?

[Fabchan]

Hab den Fehler behoben und das Addon noch ein wenig ergänzt. Steht ab sofort zum Download bereit.

[cwenet]

hallo,

habe versucht zu installieren und bekomme folgenden fehler:

Call to undefined method rex_login_sql::isAdmin()

Das Backend is nun nicht mehr zugänglich.

Muss noch was vorher beachtet werden?

Gruss
Christoph

[holger]

Hallo Fabchan,

habe dein Addon einmal ausprobiert und es gefällt mir sehr gut.
Ein Problem gibt es aber, wenn man noch den Groupletter installiert hat.

Dann erscheinen nämlich beim Anlegen der An-Abmeldeseite im Grouplettermodul unten nochmals zwei "Status: Online/Offline" - Radio-Buttons.

Diese funktionen dann entweder genau anders herum als dein On/Off oder aber man kann diese überhaupt nicht "online"-schalten.

Auf jeden Fall wird dann im Frontend nix angezeigt.

mfg
Holger

[MyPass]

Hallo Fabchan, das AddOn ist super. Ich hoffe, du hast nichts dagegen, dass ich es auch als Basis für ein eigenes AddOn verwendet habe, dass eine ähnliche Struktur verlangt!

Danke & Ciao
MyPass

[Fabchan]

Da habe ich kein Problem mit, überarbeite es zurzeit allerdings nochmal, weil es immer noch einige Probleme mit Addons wie Newslisten gibt, die auf andere Artikel zurückgreifen.

[MyPass]

Hallo,

in einer Mehrbenutzer-Umgebung ist die Ermittlung der letzten ID etwas unsicher. Nur die letzte ID zu holen ist gefährlich.

Ich habe das nun so erweitert, vielleicht brauchts ja jemand:

Code: Alles auswählen

// Zuletzt hinzugefügte ID bestimmen        
$res = $sql->getArray('SELECT MAX(id) AS max_id FROM ' . 
$REX['TABLE_PREFIX'] . 'article_slice 
--NEU-BEGINN--
WHERE article_id = '.$article_id.' 
AND createuser = `' . $REX_USER_LOGIN . '`'); 
--NEU-ENDE--

Ciao
MyPass

[Fabchan]

@MyPass: Dankeschön, werde es in die nächste Version übernehmen.

[Fabchan]

Habe soeben eine komplett überarbeitete Version hochgeladen, wer Lust hat, kann sie sich gerne mal anschauen, diese sollte jetzt endlich funktionieren, hoffe, das tut sie auch!

[cwenet]

Hallo,

ich bekomme diesen Fehler wenn ich versuche in der Strukturverwaltung einen Artikel anzuwählen. Die Installation verleif fehlerfrei.

Fatal error: Call to undefined method rex_login_sql::isAdmin()

Muss ich zuvor noch etwas anderes installieren
Gruss
Christoph

[MyPass]

Verwendest du auch die aktuelle Redaxo-Version 4.1? Da ist die Funktion drinnen und ich habe auch keine Probleme. Ev. gibt des "isAdmin()" in deiner Redaxo-Version nicht... kann ich aber nicht beurteilen, weil ich nur 4.1 kenne!

Bye
MyPass

[cwenet]

Ja, die neuste Version ist im Einsatz.

Gruss Christoph

[Fabchan]

@cwenet: Kann deinen Fehler leider auch nicht nachvollziehen, wenn du einen Bugfix dafür schreiben möchtest, übernehme ich ihn aber gerne in das Addon.

Muss mich bei euch allen entschuldigen, die Version die ich gestern hochgeladen habe enthielt nicht die aktuellste Version des Addons, irgendwie muss sich beim hin und her kopieren auch meinem Rechner ein Fehler eingeschlichen haben. Hab einen Bugfix zur verfügung gestellt. Diesmal sollte es aber wirklich funktionieren!

Mfg, Fabian

[olli_s]

Liebes redaxo-team,

auch wenn ihr euch schon dahingehend geäußert habt, dass ihr eine solche funktion nicht einbauen wollt, auch ich hätte gern eine online/offline-funktion für slices, standardmäßig neben "editieren" und "löschen". das ist meiner meinung nach standard und auch kunden schlecht zu erklären, warum das nicht gehen sollte. bitte nochmal drüber nachdenken. danke.

[Peter.Bickel]

Und was spricht dagegen, das Addon von Fabian zu verwenden?
Man braucht diese Funktion ja nicht bei jeder Site.

Grüße,
Peter.

[olli_s]

Ja, nun hab ichs auch entdeckt. Ja, super-lösung. http://www.redaxo.de/180-0-addon-detail ... don_id=356

[ds]

Zur Info: Das Addon wurde kürzlich auf Version 0.3 geupdatet und hat eine wesentliche Performanceverbessung erfahren. Der On/Off-Status wird nun mit in die REX-Cachefiles geschrieben, so dass der Request für jeden Slice nur noch beim ersten Aufruf eines neuen Artikels rausgehauen wird.

http://www.redaxo.org/de/download/addons/?addon_id=356

Viele Grüße!
_Dirk

[RexDude]

Danke Dirk!

Bei der alten Version musst ich immer vor dem ersten on/offline schalten einmal den Block editieren...hier scheint es gleich auf Anhieb zu funktionieren. Auch die Icons sind super und das mit dem Caching. Allerdings hätte ich noch zwei Ideen:

1.) Das on/offline Icon würde ich ganz zum Schluß setzen. Es sieht optisch finde ich besser aus weil es dann zu den anderen Icons (nach oben / unten schieben) besser passt.

2.) Wenn man den Block editiert find ich es unnötig dort dann den on/offline Status nochmal setzten zu können (über die Radio Buttons). Löschen, nach oben/unten ist ja z.B. auch nur oben. Würd ich also rausnehmen.

Viele Grüße,
Rudi

[RexDude]

Und noch eine Optimierungsmöglichkeit ist mir eingefallen:

3.) Momentan wird ja zu jedem Block im gecachten Artikel eine if-Abfrage eingebaut um zu ermitteln ob der Block dann angezeigt wird oder nicht. Wäre es hier nicht sinnvoller auf die if-Abfrage gänzlich zu verzichten wenn der Block online ist und sie nur reinzunehmen wenn er offline ist. Da in den meisten Fälle fast alle Blöcke eigentlich immer aktiv sind, würde das ein haufen if-Abfragen sparen was performantenr wäre. Wäre doch so sinnvoller, oder?

[Mediastuttgart]

hallo dirk,

die überarbeitung ist super, nur scheint in der klasse in zeile 297 noch ein fehler zu sein. zumindest in dem downloadarchiv auf der addonseite wo ich das package geladen habe.

Code: Alles auswählen

$result = $sql->getRow();

sollte

Code: Alles auswählen

$result = $sql->getRows();

sein

REDAXO 4.2.1

Edit:

außerdem ist mir bei 4.2.1 aufgefallen, das man erst auf system -> cache löschen klicken muss, bevor eine änderung aktiv wird. habe das addon bis jetzt aber nur auf 4.2.1 getested.

vg
michael

[ds]

Danke Euch für die Hinweise! Das Addon wird eigentlich von Fabchan gepflegt, aber wir gucken sicherlich auch nochmal über die genannten Punkte.

[ds]

Damit es beim nächsten Bugfix nicht verpasst wird, hier nochmal der Hinweis auf einen relevanten Bug im Addon:
http://www.redaxo.org/de/forum/post90501.html#p90501