DIV ausblenden per PHP

slice · 10. Mai 2014, 16:04

Englisch kannst du aber schon, oder? Du hast zweimal die Methode post_validieren() deklariert, was einmal zu viel ist.

mumpel · 10. Mai 2014, 16:25

1. Englisch kann ich nicht.
2. "post_validieren" war nur einmal drin.
3. Ich habe den Code jetzt mal neu eingebaut. Die Seite wird zwar korrekt aufgebaut. Aber der PHP-Code bleibt wirkungslos.

3daycliff · 10. Mai 2014, 19:41

zu 1.) Leo.org, dict.cc oder auch der Google-Übersetzer hätten die zumindest ansatzweise geholfen
zu 2.) Die Funktion gab es bei dir definitiv mehr als einmal

zu 3.) Ein div in HTML kannst du mit PHP bestenfalls beim nächsten Seitenaufruf ausblenden (Link hat das ja schon skizziert). Wenn es ohne komplett neuen Seitenaufruf/-aufbau ausblenden möchtest, musst du JavaScript verwenden.
Wenn ich schon lese, dass Fehlermeldungen per define "zurückgegeben" werden, will ich gar nicht wissen, wie der Rest aussieht. Ich hätte da schon mein Geld zurückverlangt. Naja, ist bzw. war ja deins ; )

Also, wenn du wirklich das komische Script verwenden möchtest, dann schreib doch mal genau wann du welche Funktionen daraus wie aufrufst, was die machen (Funktion, Rückgabewert, Ausgabe, defines, ...) und was du hoffst damit zu erreichen und wo es Probleme gibt.

mumpel · 10. Mai 2014, 20:32

Ein paar der "define" habe ich selber eingebaut, da ich die Fehlermeldungen nicht in separaten weissen Seiten haben möchte, sondern in der selben Seite wie das Formular. Und die Meldungen sollen auch im Content-Bereich angezeigt werden, und nicht irgendwo im Hintergrund wo es keiner sieht. Ich benutze wie bereits erwähnt HTML5 (mit header, nav, content, footer und aside). "Echo" schreibt aber immer in eine separate Seite.
Ich habe die angepasste PHP-Datei mal (vorübergehend, und anonymisiert)) auf meinem Webspace abgelegt-> Formmailer. Die Funktionen werden nach Klick auf den Senden-Button ausgelöst.

[...] und wo es Probleme gibt [...]

Das einzige Problem ist dass bei Erfolg das Formular verschwinden und nur die Erfolgsmeldung bleiben soll. Eine separate Erfolgseite möchte ich vermeiden. Das Formular selber funktioniert wie gewünscht. Später einmal soll das Formular dynamisch in die Hauptseite geladen werden, siehe Erklärung in meinem anderen Thread.

Edit:
Ich habe auch schon versucht, anstelle der Dankeseite eine Meldung auszugeben. Gelingt mir aber nicht.

3daycliff · 10. Mai 2014, 22:49

Gibt die Lizenz das her, dass du das teilweise veröffentlichen darfst? Ansonsten mach das mal schnell rückgängig.

Zum Code... ich weiß gar nicht wo ich anfangen soll. Da bekommt man ja Alpträume von. Sicherheitstechnisch sieht das auch nicht so toll aus...
Wer hat noch mal gesagt, dass du keinen Code bekommst? xD -.-
Ausgehend von #post896593 und dem aktuellen Stand deines "Scripts" *könntest* du wie folgt vorgehen:

Deine Seite sieht in etwa so aus:

PHP-Quellcode

<?php
$status = '';
$message = '';
if(isset($_POST['email'])) {
require('vionlink-formmailer.php');
if(isset($GLOBALS['vionlink_status'], $GLOBALS['vionlink_message'])) {
$status = $GLOBALS['vionlink_status'];
$message = $GLOBALS['vionlink_message'];
}
}
?>
<html>
<body>
<?php if($status === '' || $status === 'error') { ?>
<form method="POST" action="Name + ggf. Pfad zur dieser PHP-Datei">
<?php if($status === 'error') { ?>
<div><?php echo $message; ?></div>
<?php } ?>
</form>
<?php } elseif($status === 'succcess') { ?>
<div><?php echo $message; ?></div>
<?php } ?>
</body>
</html>

Entsprechend löscht du aus dem Script alle Zeilen mit define wobei du das erste Vorkommen (Zeile 442) durch $GLOBALS['vionlink_status'] = 'success'; $GLOBALS['vionlink_message'] = $Anfang.$Anrede.$leerZeichen.$NameVorname.$Ende; und das zweite Vorkommen (Zeile 448) durch $GLOABLS['vionlink_status'] = 'error'; $GLOABLS['vionlink_message'] = $needed_err.$endeFehler; ersetzt.
Ich hab es nicht getestet, eventuelle (Syntax)Fehler solltest du allein beheben können.
Das ganze macht das Grundproblem natürlich nicht besser und bei dem Gebrauch von exit und die zerhaust du natürlich alles.

Mein Rat wäre ja wegschmeißen und neu (selber!) machen. Gerade wenn du HTML5 erwähnst (was im übrigen absolut nichts mit deinem Problem zu tun hat) wären wahrscheinlich einige Verbesserung angeraten. Dann HTML5, eigentlich generell HTML, besteht nicht nur in der Verwendung von neuen Tags...

mumpel · 11. Mai 2014, 10:07

3daycliff schrieb:

[...] Ansonsten mach das mal schnell rückgängig [...]

Das mache ich sowieso. Darum liegt das auch auf meinem Webspace, da kann ich das schnell löschen wenn wir hier fertig sind.

3daycliff schrieb:

[...] Mein Rat wäre ja wegschmeißen und neu (selber!) machen [...]

Das würde ich machen wenn ich PHP könnte....

mumpel · 11. Mai 2014, 11:06

Danke! Das Steuern des Formulars funktioniert jetzt wie gewünscht. Nur die Meldungen erscheinen nicht. "session_destroy()" und "session_start()" habe ich mal entfernt, aber trotzdem keine Meldungen zu sehen.

Die Meldungen soll(t)en auch dort erscheinen wo die Meldungen mit "define" platziert wurden!? Also anstelle von "<?php echo MELDUNGEN; ?>" die Meldungen mit "<?php echo $message; ?>".

Edit:
Problem behoben. Habe übersehen dass da "$GLOABLS" stand.

mumpel · 11. Mai 2014, 12:14

mumpel schrieb:

"
<?php echo MELDUNGEN; ?>" die Meldungen mit "<?php echo $message; ?>".

Ist auch erledigt. Funktioniert wie ich es möchte. Danke an alle Helfer!

3daycliff · 11. Mai 2014, 12:14

Mein Fehler (den du hättest auch sehen können ; )
$GLOABLS['vionlink_status'] = 'error'; $GLOABLS['vionlink_message'] = $needed_err.$endeFehler;
Sollte natürlich jeweils $GLOBALS heißen.

mumpel schrieb:

Das würde ich machen wenn ich PHP könnte....

Dann lern es! Kann nicht schaden, besonders wenn man eh an Webseiten rumbastelt. Später kommt man da auch kaum herum...

mumpel schrieb:

Die Meldungen soll(t)en auch dort erscheinen wo die Meldungen mit "define" platziert wurden!?

Nope. Falscher Ansatz. Besser ist es, Logik und Darstellung soweit wie möglich zu trennen. Daher lässt du die Logik in dem gekauften Script und machst die Ausgabe dann wie oben beschrieben. Der Weg mit $GLOBALS ist natürlich auch totaler Mist, aber alles andere ist für dich wohl einige Wochen/Monate an Arbeit entfernt.

Edit: Okay.

mumpel · 11. Mai 2014, 12:18

3daycliff schrieb:

[...] den du hättest auch sehen können [...]

Sorry, wahr mein Fehler. Habe ich schon am Anfang bemerkt, aber aus Schusslichkeit nicht geändert.

mumpel · 11. Mai 2014, 12:59

3daycliff schrieb:

[...] Zum Code... ich weiß gar nicht wo ich anfangen soll. Da bekommt man ja Alpträume von. Sicherheitstechnisch sieht das auch nicht so toll aus [...]

Dazu noch eine Anmerkung. Vor einiger Zeit hatte ich noch den Swift-Mailer (swiftmailer.org). Der hatte aber keine Badword-List. Was ist vom Swift-Mailer zu halten? Müsste aber erst an die aktuelle PHP-Version angepasst werden.

3daycliff · 11. Mai 2014, 13:40

Auf jeden Fall besser als dein Script, allerdings ist es kein Script was du so einfach benutzen kannst. Es bietet dir lediglich die Funktionalität die du selber per PHP nach deinen Wünschen benutzen kannst. Und da du kein PHP kannst... tja... wird das schwierig : )

Zum Thema der bösen Wortliste... so was kann man natürlich auch selber programmieren und zusammen mit dem swiftmailer verwenden. Aber oft taugen die auch nichts. Auf den ersten Blick scheint es sinnvoll zu sein, z.B. das Wort "sex" zu blockieren, aber beim Wort "Sicherheitsexperte" ist das dann doch blöd. Und für jede Regel lässt sich dann meist ein Beispiel finden, wo man argumentieren kann, dass es in dem Fall besser wäre, ein Wort oder eine Phrase nicht zu blockieren (siehe z.B. spiegel.de/politik/deutschland…bedienstete-a-952273.html).

mumpel · 11. Mai 2014, 13:49

Das Problem mit dem "Sicherheitsexperten" umgehe ich mit Leerzeichen (also: ' xxx '), "Sicherheitsexperte" wird also als gut eingestuft. Irgendwo habe ich ein Script welches prüft ob das "böse Wort" alleine steht oder innerhalb eines anderen Wortes. Und so wichtig ist die Badword-List nicht. Ich möchte nur umgehen dass Spammer ihren Müll bei mir abladen, und die meisten nutzen die englische Sprache.

Link275 · 12. Mai 2014, 09:06

Hi,

offenbar hast du eine Datei 2x includet, kann das sein?

Link :thumbup:

mumpel · 12. Mai 2014, 10:06

Nein, ich habe jede Datei nur einmal drin. Was mir aber aufgefallen ist: Die "counter.php" darf ich erst nach dem von "3daycliff" geposteten Code "includieren", sonst funktioniert der Code nicht.

Link275 · 12. Mai 2014, 10:22

Passt schon, hab nicht gesehen dass es schon eine zweite Seite mit Beiträgen gibt zu diesem Thread und entsprechend nur auf den letzten Beitrag auf der ersten Seite geantwortet.

Die "counter.php" darf ich erst nach dem von "3daycliff" geposteten Code "includieren", sonst funktioniert der Code nicht

Nicht generell, es kommt auf den Code an, der in der counter.php steht. Wenn's ne Ausgabe ist, muss die Datei dort inkludiert werden wo die Ausgabe stattfinden soll. Wenn die inkludierte Datei Klassen oder Funktionen enthält dann ist es egal wo sie inkludiert wird.

Link :thumbup:

mumpel · 12. Mai 2014, 10:42

Die counter.php gehört zum Besucherzähler (Statistiktool) "expCounter".

mumpel · 12. Mai 2014, 15:50

So wie es aussieht werde ich das Kontakformular abschalten müssen. Da hat nämlich offensichtlich jemand am Kontaktformular vorbei direkt auf die "vionlink-formmailer.php" zugegriffen und so eine Mail versand. Der Aufbau sieht aus wie folgt, was aber so nicht sein kann wenn man über das Kontaktformular sendet. Da frage ich mich doch wie das möglich ist.

PHP-Quellcode

/
0
explore
upload

Normal ist:

PHP-Quellcode

Herr
Max Mustermann
Musterststraße 1
12345 Musterstadt

3daycliff · 16. Mai 2014, 19:31

mumpel schrieb:

Da hat nämlich offensichtlich jemand am Kontaktformular vorbei direkt auf die "vionlink-formmailer.php"

Ja und? *Im Prinzip* macht der Browser das gleiche (wenn du obigen Code verwendest). Jemand hätte die Daten auch so eingeben können und dann hättest du das Gleiche bekommen. Sämtliche Prüfungen sind doch in deiner angegebenen PHP-Datei.
Daher ist das eher ein Zeichen dafür, dass deine Prüfungen (bzw. die von vionlink) nichts taugen ; )

mumpel · 16. Mai 2014, 19:51

Bei der Anrede (Zeile 1) kann man im Formular nur zwei Werte wählen. Separate Eingabe nicht möglich. Da habe ich jetzt aber eine Prüfung nachgerüstet.
Bei Zeile 2 lässt sich nichts logisch prüfen. Da müssen auch Zahlen möglich sein. Könnte ja jemand "Max der 3." heissen.

Bei Zeile 4 habe ich das Problem dass PLZ und Ort in einem Eingabefeld sein müssen (technisch bedingt). Wie man das korrekt prüft weiss ich nicht. Vielleicht genauso oder so ähnlich wie im Pattern-Attribut im Formular?
Ob man bei Zeile 3 logisch prüfen kann weiss ich nicht.

3daycliff schrieb:

[..] (bzw. die von vionlink) nichts taugen [...]

Das solltest Du besser nicht dem Hersteller sagen. Der reagiert da empfindlich.

DIV ausblenden per PHP

PHP-Quellcode

3daycliff schrieb:

3daycliff schrieb:

mumpel schrieb:

mumpel schrieb:

mumpel schrieb:

3daycliff schrieb:

3daycliff schrieb:

PHP-Quellcode

PHP-Quellcode

mumpel schrieb:

3daycliff schrieb:

Ähnliche Themen

4 Benutzer haben hier geschrieben