✔ Marquee funktioniert bei Apostroph nicht.

[Andreas Späth]

Hallo,

und wiedermal wende ich mich mit einem Problem an euch.

<!-- Using javascript for the marquee since W3 doesn't like marquees. -->
<script type="text/javascript">
<!--
document.write('<marquee direction="$vboptions[toplist_forumhome_marquee_direction]" onmouseover="this.scrollAmount=0" onmouseout="this.scrollAmount=$vboptions[toplist_forumhome_marquee_scrollamount]">$toplist_images</marquee>');
//-->
</script>
<!--
<marquee scrollamount="$vboptions[toplist_forumhome_marquee_scrollamount]">$toplist_images</marquee>
-->

Wobei $toplist_images folgendes ausgibt.

<a href="bla.html" target="_blank"><img src="baum.jpg" alt="Günter's homepage"></a>

Und das ganze natürlich mehrfach wäre ja doof nur eine grafik scrollen zu lassen....

Mein Problem ist folgendes, das Javascript funkioniert nicht sobald ein ' (Apostroph) in $toplist_images vorkommt.
Fehlermeldung vom im IE ist. Firefox gibt mir nichtmal eine.

fehler ')' erwartet

Da ich nun die Leute schlecht gewalttätig dazu nötigen kann kein Apostroph zu benutzen, und garantiert einige nicht darauf achten werden, suche ich eine Lösung dieses Problem.
Also irgendeine möglichkeit damit es auch mit Apostroph funktioniert.

Ich hoffe wiedermal auf eure hilfe

Greetinx, Andy

 

[Quaese]

Hi,

vielleicht hilft es, wenn Du die Singlequotes mit einem Backslash maskierst.

str_replace("'", "\'", $toplist_images)

Ciao
Quaese

 

[Sven Mintel]

Auch wenn es nur indirekt damit zu tun hat :

Ich halte es ehrlichgesagt für recht albern, invalides HTML dadurch valide zu machen zu wollen, dass man es per JS in das Dokument schreibt....das musste ich einfach mal loswerden.

Du machst das doch in 1. Linie für Menschen, also warum schliesst du einen Teil derselben aus, nur um einen Validator zu befriedigen?

 

[Andreas Späth]

Singlequotes maskieren wäre eine Idee, mussich mir mal das Script anschauen ob ich da ein schönes str_replace() php seitig machen kann.

@Sven: Meine Idee war das nicht, ist halt ein vBulletin Addon welches schon so gekommen ist. Mir fällt aber auch ehrlichgesagt keine gute Methode ein die bilder automatisch scrollen zu lasen. Obwohl vieleicht ein automatisch scrollender <div> Container eine Lösung darstellen könnte.
Ich hab erst angefangen über diesen Teil des Scriptes nachzudenken als er auf einmal nichtmehr funktioniert

 

[Sven Mintel]

Dann ist das eine dumme Idee von vbulletin :suspekt:

Wenns dir nicht darauf ankommt, ein kleines Validitätsbildchen auf der Seite zu haben, dann lasse das Skriptzeugs doch einfach weg, und gebe es als schlichtes HTML aus...ein paar Bytes spart das auch noch

 

[Andreas Späth]

Das Addon ist von einem vBulletin.org Communitymitglied, nicht von Jelsoft selbst. Die fabrizieren sowas glaube ich nicht

Stimmt, das Script ist ja total überflüssig... Ah es gibt so Tage da klappt das mit dem denken nicht so ganz. Dadurch hätte sich ja auch mein Problem erledigt.. oh man.

Und der Button ist mir nur wichtig bei komplett selbstentwickelten Scripten, die sind alle schön valide und crossbrowserkompatibel. Wie sich das eben gehört