Warum prüfst Du dann eigentlich nicht explizit auf $_GET['mailform']
AdvancedForm_XH 1.1 Ersatz des Mailformulars
Re: AdvancedForm_XH 1.1 Ersatz des Mailformulars
Ja, hat zumindest bei Advancedform_XH geklappt. (Details siehe https://github.com/cmb69/advancedform_x ... 1ff7ac3f11.) Allerdings eben nur für PHPMailer 5 – Version 6 könnte dann doch recht anders sein, erfordert aber, meine ich, PHP >= 5.5.
Weil es um CMSimple(_XH) geht. Probier mal http://example.com/?&sitemap&mailform. Man könnte wohl auf $f === 'mailform' prüfen – wäre wohl sauberer.
Christoph M. Becker – Plugins for CMSimple_XH
Re: AdvancedForm_XH 1.1 Ersatz des Mailformulars
Na ja, "&sitemap&mailform" wird wohl nie automatisch generiert. Du kannst Dich natürlich nicht dagegen wehren, dass ein Besucher solchen Quark eingibt. Angezeigt wird dann halt - richtigerweise - in dem Fall die Sitemap.cmb wrote: ↑Fri Jul 20, 2018 7:39 pmWeil es um CMSimple(_XH) geht. Probier mal http://example.com/?&sitemap&mailform. Man könnte wohl auf $f === 'mailform' prüfen – wäre wohl sauberer.
Und auf $f kannst Du ja nicht prüfen, weil es, wenn es richtig läuft, erst gar nicht gesetzt wird wenn "&mailform" nicht in der URL vorkommt.
Was mir an Deinem Fix nicht gefällt:
Du kannst eigentlich nicht auf $su == '' prüfen, denn wenn ein Formular auf der ersten Seite eingebettet ist, und die Seite einfach mit http://example.com aufgerufen wird (also keine Seite gewählt ist), dann ist $su == '' und in ation steht dann in dem Fall auch "&mailform", was ja dann falsch ist (ich hab's auch eben extra noch einmal getestet, obwohl ich Dir ja eigentlich immer glaube ).
Aus meiner Sicht ist die Prüfung mit isset($_GET['mailform']) die einzig eindeutige Variante.
Re: AdvancedForm_XH 1.1 Ersatz des Mailformulars
Hat leider „etwas“ länger gedauert.
Da hast du natürlich Recht!
Das tut CMSimple_XH auch nicht, und mit diesem Feature soll ja CMSimple_XHs Mailformular ersetzt werden. In der Praxis würde es das wohl tun, aber
Doch, das ist ja genau wie es beim eingebauten Mailformular auch läuft. Und auch da kann mailform der erste Query-Parameter sein. Mir scheint das die CMSimple_XH konformste Lösung zu sein.
Christoph M. Becker – Plugins for CMSimple_XH