Frage zu Variblen in viewtopic

[Darezettl]

Dies hier ist die email function in der viewtopic.php:

Code: Alles auswählen

		if ( !empty($postrow[$i]['user_viewemail']) || $is_auth['auth_mod'] )
		{
			$email_uri = ( $board_config['board_email_form'] ) ? append_sid("profile.$phpEx?mode=email&" . POST_USERS_URL .'=' . $poster_id) : 'mailto:' . $postrow[$i]['user_email'];

			$email_img = '<a href="' . $email_uri . '"><img src="' . $images['icon_email'] . '" alt="' . $lang['Send_email'] . '" title="E-Mail an ' . $postrow[$i]['username'] . ' senden" border="0" /></a>';
			$email = '<a href="' . $email_uri . '">' . $lang['Send_email'] . '</a>';
		}
		else
		{
			$email_img = '';
			$email = '';
		}

In der viewtopic_body.tpl kann man wunderbar mit {postrow.EMAIL_IMG} dies hier ansprechen
-->

Code: Alles auswählen

$email_img = '<a href="' . $email_uri . '"><img src="' . $images['icon_email'] . '" alt="' . $lang['Send_email'] . '" title="E-Mail an ' . $postrow[$i]['username'] . ' senden" border="0" /></a>';

Ich würde nun aber gerne - da ich mir ein Css Menü eingebaut habe - nicht $email_img für dieses nutzen, sondern nu den text, ohne Bild.

Das logischste wäre, dass dafür die dies hier zu gebrauchen wäre: {postrow.EMAIL}


Jedoch geht das nicht.



Deshlab wollte ich fragten, wie ich in der viewtopic_body.tpl dies hier ansteuern kann: $email = '<a href="' . $email_uri . '">' . $lang['Send_email'] . '</a>';


Vielen Dank für die Hilfe!

[oxpus]

Das hier ist die korrekte Zeile im php-Script:

Code: Alles auswählen

$email = '<a href="' . $email_uri . '">' . $lang['Send_email'] . '</a>';

Und übergeben musst Du das dann mit

Code: Alles auswählen

'EMAIL' => $email,

Ist eigentlich alles bereits in der viewtopic.php vorhanden...

[Darezettl]

Hallo oxpus, vielen Dank für die Antwort.


Also ich hab nun denke ich rausgefunden wieso das mit "nur" {postrow.EMAIL} nicht geht.
In meinem Java Menü kann man zwar Links angeben, aber im Menü muss der <a - tag drinnen sein.

Ich poste mal das Menü, also die relevante Stelle:

Code: Alles auswählen

//Contents for menu 2, and so on
var menu2=new Array()
menu2[0]='<a href="javascript:void(0)">Optionen</a>'
menu2[1]='<a href="{postrow.PROFILE}"><img src="images/dbb1.gif" border="0"> öffentliches Profil ansehen</a>'
menu2[2]='<a href="{postrow.PM}"><img src="images/dbb2.gif" border="0"> PM senden</a>'
menu2[3]='<a href="{postrow.SEARCH}"><img src="images/dbb4.gif" border="0"> Beiträge anzeigen</a>'
menu2[4]='<a href="{postrow.EMAIL}"><img src="images/dbb5.gif" border="0"> Email senden</a>'
menu2[5]='<a href="{postrow.WWW}"><img src="images/dbb3.gif" border="0"> Website besuchen</a>'
menu2[6]='<a href="{postrow.ALBUM}"><img src="images/dbb7.gif" border="0"> Photos anschauen</a>'
var menuwidth='180px' //default menu width
var menubgcolor='f9f9f9'  //menu bgcolor
var disappeardelay=999999999999999999  //menu disappear speed onMouseout (in miliseconds)
var hidemenu_onclick="yes" //hide menu when user clicks within menu?

/////No further editting needed

Daher meine Frage, könnte man dies

Code: Alles auswählen

$email = '<a href="' . $email_uri . '">' . $lang['Send_email'] . '</a>';

in der viewtopic.php irgendwie so umändern, dass ich dann in der viewtopic_body.tpl einen schönen für das Menü passenden Link a la <a href="{U_IRGENDWAS}">bla</a> habe?

Ich denke dann würds gehen..

[oxpus]

Genau diesen Link hatte ich ja bereits gepostet...
$emal wäre auch der Link inkl. des Textes.
Das wäre im Template auch genau der klassische Link ohne Schnörkel aber funktionsfähig.
Normalerweise...

[Darezettl]

Okay, weißt du vielleicht jemanden den ich da fragen könnte, der sich mit Javascripts in phpBB auskennt?

Weil dann werd ich einfach mal bei phpbb.de nachfragen, und ein paar javascript foren abgrasen.


das seltsame ist ja, weil ich hab mit gedacht vielleicht gehen in dem java script gar keine dynamische links, also nur statische wie http://adresse.de/seite.html, aber die "großen" variable ala {U_ALBUM} oder {U_INDEX} werden schon richtig verlinkt, hab ich extra noch ausprobiert.

hm..


aber zu den postrow Variablen.
da schreibt man doch normalerweise auch nur {postrow.EMAIL} zb und nicht <a href="{postrow.EMAIL}">bla</a> gell?
dann kann des ja gar nicht gehen.
und aber wenn ich es ohne das <a schreib wird im menu dann gar nichts angezeigt.

zefix.. jetzt würds so geil ausschauen aber geht nicht.
zum in arsch beißen ist das...


>> Screenshots:
[Menü zu]:


Menü aufgeklappt:

[oxpus]

Nein.
{postrow.EMAIL} ist doch nur ein Platzhalter!
Der kann durch so ziemlich alles ersetzt werden.
Also ein Text, Link, Bild, nichts, etc...
Eben daher auch durch einen HTML-Link, wie gewünscht.
Und {postrow.EMAIL} ist in einem normalen phpBB bereits ein Textlink wie oben angegeben.

[Darezettl]

Na gut, dann sag mir mal was ich falsch gemacht habe:

Also, fangen wir ganz von vorne an:
Es geht um dieses Menü.
In meine viewtopic_body.tpl habe ich ganz am anfang dies hier eingefügt

Code: Alles auswählen

<script type="text/javascript">

/***********************************************
* AnyLink Drop Down Menu- © Dynamic Drive (www.dynamicdrive.com)
* This notice MUST stay intact for legal use
* Visit http://www.dynamicdrive.com/ for full source code
***********************************************/

//Contents for menu 2, and so on
var menu2=new Array()
menu2[0]='<a href="javascript:void(0)">Optionen</a>'
menu2[1]='{postrow.PROFILE}'
menu2[2]='{postrow.PM}'
menu2[3]='{postrow.SEARCH}'
menu2[4]='{postrow.EMAIL}'
menu2[5]='{postrow.WWW}'
menu2[6]='{postrow.ALBUM}'
var menuwidth='180px' //default menu width
var menubgcolor='f9f9f9'  //menu bgcolor
var disappeardelay=999999999999999999  //menu disappear speed onMouseout (in miliseconds)
var hidemenu_onclick="yes" //hide menu when user clicks within menu?

/////No further editting needed

var ie4=document.all
var ns6=document.getElementById&&!document.all

if (ie4||ns6)
document.write('<div id="dropmenudiv" style="visibility:hidden;width:'+menuwidth+';background-color:'+menubgcolor+'" onMouseover="clearhidemenu()" onMouseout="dynamichide(event)"></div>')

function getposOffset(what, offsettype){
var totaloffset=(offsettype=="left")? what.offsetLeft : what.offsetTop;
var parentEl=what.offsetParent;
while (parentEl!=null){
totaloffset=(offsettype=="left")? totaloffset+parentEl.offsetLeft : totaloffset+parentEl.offsetTop;
parentEl=parentEl.offsetParent;
}
return totaloffset;
}


function showhide(obj, e, visible, hidden, menuwidth){
if (ie4||ns6)
dropmenuobj.style.left=dropmenuobj.style.top="-500px"
if (menuwidth!=""){
dropmenuobj.widthobj=dropmenuobj.style
dropmenuobj.widthobj.width=menuwidth
}
if (e.type=="click" && obj.visibility==hidden || e.type=="mouseover")
obj.visibility=visible
else if (e.type=="click")
obj.visibility=hidden
}

function iecompattest(){
return (document.compatMode && document.compatMode!="BackCompat")? document.documentElement : document.body
}

function clearbrowseredge(obj, whichedge){
var edgeoffset=0
if (whichedge=="rightedge"){
var windowedge=ie4 && !window.opera? iecompattest().scrollLeft+iecompattest().clientWidth-15 : window.pageXOffset+window.innerWidth-15
dropmenuobj.contentmeasure=dropmenuobj.offsetWidth
if (windowedge-dropmenuobj.x < dropmenuobj.contentmeasure)
edgeoffset=dropmenuobj.contentmeasure-obj.offsetWidth
}
else{
var topedge=ie4 && !window.opera? iecompattest().scrollTop : window.pageYOffset
var windowedge=ie4 && !window.opera? iecompattest().scrollTop+iecompattest().clientHeight-15 : window.pageYOffset+window.innerHeight-18
dropmenuobj.contentmeasure=dropmenuobj.offsetHeight
if (windowedge-dropmenuobj.y < dropmenuobj.contentmeasure){ //move up?
edgeoffset=dropmenuobj.contentmeasure+obj.offsetHeight
if ((dropmenuobj.y-topedge)<dropmenuobj.contentmeasure) //up no good either?
edgeoffset=dropmenuobj.y+obj.offsetHeight-topedge
}
}
return edgeoffset
}

function populatemenu(what){
if (ie4||ns6)
dropmenuobj.innerHTML=what.join("")
}


function dropdownmenu(obj, e, menucontents, menuwidth){
if (window.event) event.cancelBubble=true
else if (e.stopPropagation) e.stopPropagation()
clearhidemenu()
dropmenuobj=document.getElementById? document.getElementById("dropmenudiv") : dropmenudiv
populatemenu(menucontents)

if (ie4||ns6){
showhide(dropmenuobj.style, e, "visible", "hidden", menuwidth)
dropmenuobj.x=getposOffset(obj, "left")
dropmenuobj.y=getposOffset(obj, "top")
dropmenuobj.style.left=dropmenuobj.x-clearbrowseredge(obj, "rightedge")+"px"
dropmenuobj.style.top=dropmenuobj.y-clearbrowseredge(obj, "bottomedge")+obj.offsetHeight+"px"
}

return clickreturnvalue()
}

function clickreturnvalue(){
if (ie4||ns6) return false
else return true
}

function contains_ns6(a, b) {
while (b.parentNode)
if ((b = b.parentNode) == a)
return true;
return false;
}

function dynamichide(e){
if (ie4&&!dropmenuobj.contains(e.toElement))
delayhidemenu()
else if (ns6&&e.currentTarget!= e.relatedTarget&& !contains_ns6(e.currentTarget, e.relatedTarget))
delayhidemenu()
}

function hidemenu(e){
if (typeof dropmenuobj!="undefined"){
if (ie4||ns6)
dropmenuobj.style.visibility="hidden"
}
}

function delayhidemenu(){
if (ie4||ns6)
delayhide=setTimeout("hidemenu()",disappeardelay)
}

function clearhidemenu(){
if (typeof delayhide!="undefined")
clearTimeout(delayhide)
}

if (hidemenu_onclick=="yes")
document.onclick=hidemenu

</script>

Gut, dann habe ich das hier:

Code: Alles auswählen

{postrow.POSTER_NAME}

mit dem ersetzt:

Code: Alles auswählen

<a href="javascript:void(0)" onClick="return dropdownmenu(this, event, menu2, '230px')" onMouseout="delayhidemenu()">{postrow.POSTER_NAME}</a>

So und nun würde ich gern wissen was ich da falsch gemacht hab / was du da anders gemacht hättest.

[oxpus]

Ist doch so i.o. ...

[Darezettl]

Ok, also nun bin ich schon einen ganzen Schritt weiter und (fast) am Ziel.
Der Fehler war dass das Javascript nicht innerhalb von <-- BEGIN postrow --> war.

Gut, also jetzt isses inhalb dieses Switches, aber: Wenn man jetzt in nem Thread mit meinetwegen 5 Beiträgen darin auf den Namen irgendeines Posters klickt, bezieht sich das Menü immer auf den, der als letztes in diesem Thread gepostet hat, also es werden immer - egal auf welchen Poster man klickt - die Infos zu dem der als letztes im Thread gepostet hat angezeigt.

Das ist jetzt noch der einzige Fehler.. ne Idee wie man den noch wegbekommt?!?


---
Da ich grad an der FH bin und ich bei dem Linux Rechner hier ums verrecken kein WinZip o.ä. find wo ich die viewtopic_body.tpl zippen und hier anhängen kann, hab ich sie einfach in einem meiner Testforen gepostet. Wenn ihr hier klickt, kommt ihr direkt hin. Sorry für die Umstände.. ich hoffe mir wird trotzdem geholfen.
---

[oxpus]

Besser, wenn dann immer anhängen.
Auch die viewtopic.php bitte dazu, da hier ja die Steuerungen/Platzhalter erstellt werden...

[Darezettl]

so, jetzt bin ich endlich zuhause..


also hier wären dann die beiden Dateien.. wär extremst cool wenn du was machen könntest oxpus!



Vielen dank schon mal für die Mühe...

[oxpus]

Also ich kann da stundenlang draufschauen (abgesehen: OXPUS und Javascript ...)...
Ich kann nicht wirklich erkennen, was daran falsch sein sollte.
Was genau geht denn nicht oder wird der Link nur nicht angezeigt?
Zudem auch der Link der Webseite des Users dort enthalten ist.
Wird denn dieser Link angezeigt?

[Darezettl]

Also es wird alles korrekt angezeigt.
Alles passt.

Wenn ich bei dem User, der als letztes einen Beitrag in dem Thread geschrieben habe, auf seinen Namen klicke, wird wunderbar alles angezeigt und auch die Links funzen prima, alles so wie es sein soll.

Wenn ich aber bei irgend einem anderen Poster, der nicht den letzten Beitrag auf dieser Seite des Threads verfasst hat, auf dessen Name klicke, wird das alles angezeigt, als wie wenn ich auf den Namen des letzten Posters im Thread klicken würde.
>> Also sowohl der Name des jenigen, der den letzten Post auf dieser Seite des Threads gepostet hat erscheint im Menu, als auch beziehen sich alle Links im Menü auf ihn, und nicht auf den Poster, wo ich eigentlich draufklickt habe!


Was kann man da nur machen??

[oxpus]

Ah, jetzt verstehe ich...
Du fügst ja immer das Javascript mit jedem Beitrag ein.
Das vergrössert zum einen nicht nur die an den User zu übertragende Seite (der Seitenaufbau braucht dann ja auch zwangsweise länger), sondern es werden hier immer die gleichen Variablen im Javascript verwendet, die eben dann den Link des letzten Posters haben!
Logisch

Öh, hm, was machen...
Das Menü wird ja per Link aufgerufen.
Hier wäre es geschickter, das Script so umzuschreiben, daß eben alle nötigen Angaben mit übergeben werden.
Dann kann man das je Post trennen und das Script selber muss auch nur einmal in die viewtopic_body-tpl eingefügt werden.

Hm, ich will mal nachher schauen, wie ich Zeit habe und ob ich das (zumindest soweit in der Theorie) anpassen kann...

[Darezettl]

[quote="oxpus - Sa 08.Apr, 2006 12:39";p="57483"]
Hier wäre es geschickter, das Script so umzuschreiben, daß eben alle nötigen Angaben mit übergeben werden.
Dann kann man das je Post trennen und das Script selber muss auch nur einmal in die viewtopic_body-tpl eingefügt werden.

Hm, ich will mal nachher schauen, wie ich Zeit habe und ob ich das (zumindest soweit in der Theorie) anpassen kann...[/quote]

Ok, ich verlass mich da mal ganz auf dich und dein Können.
Wär echt a super Sach, wenn du's hinbringen tätst....

[oxpus]

Abwarten.
Mit Javascript ist das immer so eine Sache...

[Darezettl]

Jo, bressiert ja ned.. lass da Zeit.
Aber stylisch ist das Teil schon gell?

[oxpus]

Ja, hat was.

[Darezettl]

Du postet dann einfach hier wenn sich was ergeben hat ja?
Ich frag dann bis dahin nicht mehr nach..

[oxpus]

Genau, so machen wir das.

[Darezettl]

Also ich hab jetzt mal den Author des Javascripts direkt angeschrieben, und der meinte folgendes.
Allerdings peil ich nicht so recht was er damit meint.. hilft dir das irgendwie weiter oxpus, kannst du damit was anfangen? Ich kann es jedenfalls nicht..

>>
The problem most likely is that this:

Code: Alles auswählen

menu2[0]='<a>&raquo; Optionen</a>'
menu2[1]='{postrow.PROFILE}'
menu2[2]='{postrow.PM}'
menu2[3]='{postrow.SEARCH}'
menu2[4]='{postrow.EMAIL}'
menu2[5]='{postrow.WWW}'

is written out and parsed by the browser at the time the page is first loaded. These values will not change unless you write some code to reinitialize them. Like if each time a different post in the thread receives focus or however that is done, if that changes the values represented by those tokens have it also run a function like:

Code: Alles auswählen

function updateTokens(){
menu2[0]='<a>&raquo; Optionen</a>'
menu2[1]='{postrow.PROFILE}'
menu2[2]='{postrow.PM}'
menu2[3]='{postrow.SEARCH}'
menu2[4]='{postrow.EMAIL}'
menu2[5]='{postrow.WWW}'
}

(ab hier gehts weiter mit nem Bild, weil ich die Codes hier irgendwie nicht posten kann)

[oxpus]

Das wäre zwar möglich, verbessert aber die Situation nicht wirklich, da weiterhin das komplette Script je Post mit geladen wird...

[Darezettl]

Ja und könnte man nicht irgendwie des so machen, dass das Script nur 1x geladen wird?
BZW: Wo ist das Problem, dass dies nicht geht?

[oxpus]

Hier eben genau der Versuch, das das Script nur einmal herangezogen wird.
Ich habe das allerdings nicht getestet.
Könnte auch sein, daß da nur Murx rauskommt...

[Darezettl]

Aso ich hab beide Dateien hochgeladen und die Cache des Browser geleert.
Wenn ich dann auf den Namen eines Poster klicke, passiert gar nichts.
bzw. es erscheint ein kleiner waagrerechter Strich, wohl als Zeichen dass schon was da wäre, aber das Menü leer ist.

hm... was nun?