Forum aktualisiert

[routeconverter]

Ich habe mal die Option search engine friendly URLs angestellt und hoffe, daß dadurch nichts kaputt geht/gegangen ist...

[routeconverter]

\inc\datahandlers\post.php (Zeile 917)
$emailmessage = $lang->sprintf($emailmessage, $subscribedmember['username'], $post['username'], $mybb->settings['bbname'], $subject, $excerpt, $mybb->settings['bburl'], str_replace("&", "&", get_thread_link($thread['tid'], 0, "newpost")), $thread['tid'], $subscribedmember['subscriptionkey']);

\inc\functions.php (Zeile 4286)
function get_thread_link($tid, $page=0, $action='')
{ ... }

\inc\functions.php (Zeile 4325)
function get_post_link($pid, $tid=0)
{ ... }

Ich denke mal in dem Statement in der post.php müsste man das get_thread_link($thread['tid'], 0, "newpost") durch get_post_link($thread['pid'], $thread['tid']) ersetzen, damit man das von mir gewünschte Ergebnis erhält.

Hallo Dennis,

genau das habe ich mal gemacht. Mal schauen, wie die nächste EMail aussieht.

[herbie]

Hi Christian,

bisher noch das selbe Ergebnis (von dem her, wo der Link dann hin springt), aber es sieht schon besser aus.
Allerdings ist der hintere Teil mit der tid im Ergebnis noch nicht richtig.

Hab nun noch mal geschaut und folgendes gefunden:

\install\resources\mybb_theme.xml (Zeile 5182)
<template name="postbit_posturl" version="1400"><![CDATA[<div style="float: right; width: auto; vertical-align: top"><span class="smalltext"><strong>{$lang->postbit_post} <a href="{$post['postlink']}#pid{$post['pid']}">#{$postcounter}</a></strong>{$post['inlinecheck']}</span></div>]]></template>

Das ist wohl die Stelle, wo eben der Link hinter "Post: #..." bzw. "Beitrag: #..." oben rechts von jedem Beitrag zusammen gebaut wird.

Demnach müsste wohl das Statement in der post.php von
ursprünglich get_thread_link($thread['tid'], 0, "newpost")
durch ein {$post['postlink']}#pid{$post['pid']} ersetzt werden
(und nicht wie zuvor geschrieben durch get_post_link($thread['pid'], $thread['tid'])).

Wenn du das nur bitte noch einmal versuchen könntest.
Wenn das dann auch nichts bringen würde: einfach wieder auf Anfang und vergessen. Dann soll es halt nicht sein.

[routeconverter]

[quote='herbie' pid='2154' dateline='1242460280']
bisher noch das selbe Ergebnis (von dem her, wo der Link dann hin springt), aber es sieht schon besser aus.
[/quote]

Jepp.

[quote='herbie' pid='2154' dateline='1242460280']
Allerdings ist der hintere Teil mit der tid im Ergebnis noch nicht richtig.

Hab nun noch mal geschaut und folgendes gefunden:
[quote]\install\resources\mybb_theme.xml

Wenn du das nur bitte noch einmal versuchen könntest.
[/quote]

Die Datei gehört zu den Installationsdateien von MyBB - d.h. sie existiert gar nicht in meiner Installation.

Ich habe mir daraufhin andere URLs im Code angeschaut und

Code:

newpost

durch

Code:

lastpost

ersetzt. Da sollten jetzt URLs wie

http://www.routeconverter.de/forum/showt...n=lastpost

in der Email stehen. Und die springt zumindest ans Ende des Threads.

[herbie]

Mein Anliegen war es ja nicht, dass du die Daten in der mybb_theme.xml änderst, sondern nur die Daten in der post.php anhand dem Schema, wie es in der mybb_theme.xml gelöst wurde
Weil so mit dem lastpost ist es für Cookies-Löscher wie mich zwar ein guter Kompromiss, aber für die Leute, die die Cookies nicht löschen (was ja Standardeinstellung der Browser ist) leider eine Verschlechterung.

Und da ich ja nicht so egoistisch sein möchte und meine Bedürfnisse vor die Anderer stellen will, würde da schon eher für die ursprüngliche Version mit dem newpost sein anstatt für die mit lastpost

[routeconverter]

Mein Anliegen war es ja nicht, dass du die Daten in der mybb_theme.xml änderst, sondern nur die Daten in der post.php anhand dem Schema, wie es in der mybb_theme.xml gelöst wurde

Jetzt verstehe.

Weil so mit dem lastpost ist es für Cookies-Löscher wie mich zwar ein guter Kompromiss, aber für die Leute, die die Cookies nicht löschen (was ja Standardeinstellung der Browser ist) leider eine Verschlechterung.

Wie meinst Du das? Wo spielen die Cookies da hinein?

[herbie]

Das newpost muss ja irgendwo her wissen, wann jemand das Forum zum letzten Mal besucht hat.
Und gerade wenn man eben nicht eingeloggt ist, kann das ja nur über das Cookie ausgelesen werden, welches u.a. die Zeiten für [lastactive] und [lastvisit] enthält.
Ist dieses Cookie aber nicht vorhanden (wie bei mir z.B. beim Neustart des Firefox) wird ja davon ausgegangen, dass ich noch nie das Forum besucht habe.
Daher springt newpost dann immer zum ersten Beitrag in dem Thema.

Bei den Leuten, die die Cookies aber nicht löschen, springt newpost wirklich zum für den Nutzer ersten neuen Beitrag innerhalb des Themas.
Wenn du da nun lastpost einsetzt, kommen alle durch den Link immer nur zum letzten Beitrag in dem Thema, obwohl seit dem letzten Besuch vielleicht drei oder vier Beiträge geschrieben worden sind und nicht nur der eine letzte in dem Thema.
Daher ist es für die Leute eine Verschlechterung. Hoffe, das war verständlich (und richtig)

[routeconverter]

Hallo Dennis,

danke. Nun habe ich es verstanden. Und ich habe

PHP Code:

get_thread_link($thread['tid'], 0, "newpost") 


durch

PHP Code:

"{$post['postlink']}#pid{$post['pid']} " 


ersetzt. Das funktioniert aber nicht, beim Abschicken von Posts scheint es zu einem Fehler zu kommen und man bekommt eine leere Seite.

[routeconverter]

In der PHP-Referenz von MyBB steht

PHP Code:

get_thread_link($tid, $page=0, $action='')  
Build the thread link.
param: int The thread id of the thread.
param: int (Optional) The page number of the thread.
param: string (Optional) The action we're performing (ex, lastpost, newpost, etc)
return: string The url to the thread. 


und darunter

PHP Code:

get_post_link($pid, $tid=0)   
Build the post link.
param: int The post ID of the post
param: int The thread id of the post. 


Ich rufe nun mal testweise

PHP Code:

get_post_link($post['pid'], $thread['tid') 


auf. Klappt das?

[routeconverter]

Leider auch nicht...