imageselectorplus entries_header

[Timbalu]

Hi Garvin

Ich sitze gerade nochmal am imageselectorplus Plugin, weil da noch so einige Altlasten sind.
Kannst du mir sagen wieso, warum und wann dort der entries_header output generiert wird. Ich finde da momentan einfach keinen Ansatz.

Per index.php?serendipity[subpage]=s9yisp&serendipity[id]=XX kann ein einzelnes Bild aus einem Blogbeitrag dargestellt werden.

entries_header wird ja auch z.b. von staticpages benutzt um eine einzelne Blogseite zu übernehmen.

Danke. Aber da passiert rein gar nichts, da die preg_matches nicht wahr werden ... Weder in 1.7x noch 2.0. Irgendwie habe ich da gerade einen dicken Balken vor den Augen und wahrscheinlich die falschen Testbeispiele...

Wann hat das so das letzte mal funktioniert? Auch über staticpages sehe ich da keinen Einstieg.
Wann bzw in welchem Fall wird denn der link für

Code: Select all

if (preg_match('@<a title="([^"]+)" id="s9yisp' . $imageid . '"></a>@imsU', $entry['body'], $imgmatch))

im $entry['body'] überhaupt so generiert, dass imageselectorplus entries_header weiter ausgegeben würde?

[garvinhicking]

Hi!

Die serendipity_imageSelectorPlus_done() javascriptfunktion prüft, ob die "serendipity[isLink]" Option auf "plugin" gesetzt wurde, und fügt ein Bild dann mit diesem Link in den body/extended body des Blogeintrag ein.

Diese _done() funktion wiederum wird über den Hook "frontend_image_selector" ausgeführt. Ich weiß nicht ob das in der 2.0 noch funktioniert, in der 1.x müsste es eigentlich klappen.

Sprich, das Plugin erweitert eigentlich die Einfüge-Optionen über das Mediendatenbank-Icon im Eintragseditor, und fügt da die neuen Optionen ein...

VG;
GArvin

[Timbalu]

Siehste... da haben wir ja meinen Balken. tztztz
Ich hatte u.a. diese js Funktion im Plugin entfernt, weil dort steht: "depreceated, in new version >0.9 already in admin_image_selector.php", was dann wohl nicht stimmt.
Das ist also nur zuständig für case plugin, ja? Ich muss da mit etwas Zeit nochmal sorgfältig drübergehen, aber das könnte ich eventuell schon anderweitig gelöst haben.

[garvinhicking]

Siehste... da haben wir ja meinen Balken. tztztz
Ich hatte u.a. diese js Funktion im Plugin entfernt, weil dort steht: "depreceated, in new version >0.9 already in admin_image_selector.php", was dann wohl nicht stimmt.
Das ist also nur zuständig für case plugin, ja? Ich muss da mit etwas Zeit nochmal sorgfältig drübergehen, aber das könnte ich eventuell schon anderweitig gelöst haben.

Keine Ahnung wie es zu dieser "depreceated" Notiz kam. Ich schreibe das Wort eigentlich anders, aber in diesem CVS commit:

http://php-blog.cvs.sourceforge.net/vie ... 25&r2=1.26

wurde es eingefügt. Ich kann mich leider nicht mehr dran erinnern, ob ich da einen fremden Patch eingebunden habe. Wie auch immer, deprecated ist diese Stelle eigentlich nicht. Ein guter Hinweis, ich habe es jetzt einmal herausgenommen. Tut mir leid für die Verwirrung!

[Timbalu]

Trotzdem sind da noch allerlei Merkwürdigkeiten. Was zB soll der Umweg über {{s9yisp_entryid}} bewirken?

Case 'plugin' funktioniert normalerweise zb über ML in ein entry so:

Code: Select all

<a class="serendipity_image_link" href="http://domain/serendipity_admin_image_selector.php?serendipity[step]=showItem&serendipity[image]=36" id="s9yisphref36" onclick="javascript:this.href = this.href + '&serendipity[from]=' + self.location.href;"><!-- s9ymdb:36 --><img alt="" class="serendipity_image_center" src="/uploads/testimage.jpg" style="height:565px; width:850px" /></a>

und umläuft isp 'entries_header' komplett.

Daher war ich auch der Meinung das 'frontend_image_selector' mit dem script + 'entries_header' + 'frontend_image_selector_imagelink' mit display_target_selectbox(), welches ebenfalls unbenutzt herumlag und dieselbe deprecated Notiz zeigte, irgendein altes Relikt waren.

Daher habe ich select target überhaupt erstmal in quickblog eingebaut und auch eine Lösung gefunden, wie das auch für case 'plugin' die richtigen Werte erzeugen kann. Bis ich dann zum Grund meiner Frage kam.

Ich kann mir immer noch vorstellen, das die genannten hooks und funktions und die {{s9yisp_entryid}} replacements unnötig sind, um quickblog auch mit targets korrekt anzeigen zu können. Außer eben ich übersehe da etwas.

[garvinhicking]

Trotzdem sind da noch allerlei Merkwürdigkeiten. Was zB soll der Umweg über {{s9yisp_entryid}} bewirken?

Da beim einfügen eines Bildes die ID des Eintrags noch nicht zur Verfügung steht, muss der Link über backend_entry_presave angepasst werden damit da die korrekte Entry-ID mit drin steht. Die braucht man ja, um später beim dynamischen Aufruf des Scripts diese übermitteln zu können.

Dein Hinweis auf die serendipity_admin_image_selector.php ist ganz interessant. Im Grunde hast Du da dein Finger auf eine wichtige Sache gelegt; man kann ja damit auch das einzelne Bild anzeigen in einem eigenen "Info-Fenster", das würde also konzeptionell was ähnliches tun. Ich stecke da jetzt nicht unbedingt tief drin derzeit, klappt diese Verlinkung denn? Das würde dann wirklich entries_header und den display_script kram deprecated machen.

[Timbalu]

Ich schick dir mal meinen (unfinished) Entwurf als diiff. (Noch ohne Änderungen an den save hooks.)

[garvinhicking]

Sieht recht gut aus für mich! Hast Du das in 1.7 und 2.0 getestet? Wenn es da in deinen Testcases klappt kannst Du es von mir aus gerne so committen, würde ich sagen...

[Timbalu]

Ich meine JA. Allerdings werde ich noch ein wenig weiterbasteln, zur Sicherheit.
Die hooks 'backend_entry_presave', 'backend_publish' und 'backend_save' würde ich dann vorerst untouched lassen, oder? Auch 'entries_header' müsste IMHO aus Compat Gründen so bleiben.

[garvinhicking]

Ich meine JA. Allerdings werde ich noch ein wenig weiterbasteln, zur Sicherheit.
Die hooks 'backend_entry_presave', 'backend_publish' und 'backend_save' würde ich dann vorerst untouched lassen, oder? Auch 'entries_header' müsste IMHO aus Compat Gründen so bleiben.

Es dürfte ja eigentlich nix böses machen, ich würd's daher auch erstmal noch drin lassen; so viel Code ist es ja nicht.