bei onclick php variable an js Funktion übergeben

[Spunk95]

Heyho liebes Forum,
soweit ich bei Google gesehn habe, hatten schon viele dieses Problem, allerdings hat mir keine der angebotenen Lösungen geholfen.

Hier mein Problem :

function edit_text(text){
 document.getElementById('show_text').innerHTML = '<textarea class="show_text">'+text+'</textarea>';
  }

<div id="show_text">
    <div class="show_text" onClick="edit_text('<?echo $text_edit;?>');"><?echo $text;?></div>
</div>

$text_edit wird weiter oben mit htmlspecialchars($row->text) definiert, falls das wichtig ist.

Wenn ich bei der Funktion zB edit_text('test'); eingeben, klappt es. Mit php klappt es leider nicht.

Wäre dankbar für Hilfe

[h3ll]

"klappt nicht" ist keine Fehlerbeschreibung.

[Spunk95]

"klappt nicht" ist keine Fehlerbeschreibung.

Aber wie beschreibt es man sonst wenn nichts passiert bei onclick? ^^

[h3ll]

Du könntest zB. den HTML-Code zeigen, der von PHP ausgegeben wird. Du könntest eventuelle Fehlermeldungen aus der JavaScript-Konsole zeigen. Du könntest deine Seite mit dem W3C-Validator auf HTML-Fehler testen.

[AmicaNoctis]

Ist short_open_tags bei dir aktiviert? Wenn nicht, schreib PHP-Blöcke als gültige PIs (<?php ... ?>). Wenn doch, such den Schuldigen und erklär ihm, dass die 90er vorbei sind

[Spunk95]

Danke für den Tipp mit der Konsole h3ll, wusste gar nicht, dass mein Browser das kann. Ist sehr praktisch

Ist short_open_tags bei dir aktiviert? Wenn nicht, schreib PHP-Blöcke als gültige PIs (<?php ... ?>). Wenn doch, such den Schuldigen und erklär ihm, dass die 90er vorbei sind

Es klappt jetzt zwar, allerdings gibt die Konsole diesen Fehler aus:

TypeError: Result of expression 'document.getElementById(div)' [null] is not an object.

Danke schonmal für die Hilfe

[Kropff]

Also wenn schon, dann

document.getElementById('div') 

Sieht aber eher nach einem

document.getElementsByTagName('div') 

aus. Allerdings kann ich da ohne den relevanten Code nur raten.

Peter

[Spunk95]

Danke für die schnelle Antwort, hab jetzt erst begriffen, dass es sich nicht um eine allgemeine Fehlerbeschreibung handelt und deshalb ohne '' ist, sondern dass es sich auf diesen Code bezieht:

function show(div,val) { 
		document.getElementById(div).style.visibility = val; 
	}

[h3ll]

Offenbar gibts das Element nicht, das den Namen hat, der in der Variable div steht.

[Spunk95]

Offenbar gibts das Element nicht, das den Namen hat, der in der Variable div steht.

Aufgerufen wird die Funktion durch

onMouseOver="show('clicktoedit','visible');" onMouseOut="show('clicktoedit','hidden');"

und das Element clicktoedit existiert... Oder meinst du etwas anderes?

[h3ll]

JavaScript sagt, dass es das nicht gibt. Und ich glaub nicht, dass JavaScript lügt. Also solltest du das überprüfen (Debug-Ausgaben).

[Spunk95]

JavaScript, dass es das nicht gibt. Und ich glaub nicht, dass JavaScript lügt. Also solltest du das überprüfen (Debug-Ausgaben).

Du drückst dich irgendwie immer ein bisschen unklar aus. Was ist Javascript, dass es nicht gibt? ^^
Ich glaub auch nicht, dass Javascript mich anlügt

[h3ll]

Sorry, da wurde irgendwie ein Wort verschluckt.

[Spunk95]

Sorry, da wurde irgendwie ein Wort verschluckt.

Okay, dann hab ichs jetzt verstanden ^^ Hab nachgeschaut im Quelltext, da ist clicktoedit eindeutig vorhanden, insgesamt 3 mal (onmouseover,onmouseout,id). Versteh nicht, was Javascript daran auszusetzen hat. Und es klappt nur solange man keine Zeilenumbrüche im Anfangstext hat oO