assets/js/contact_form.js

@@ -1,45 +1,29 @@
-const contactFormAjaxUrl = '/php/contact_form.php';
+const ajaxUrl = '../php/contact_form.php';
+const contact_form = document.getElementsByClassName('content__contact_form')[0];
+const subject = document.getElementsByClassName('contact_form__subject')[0];
+const message = document.getElementsByClassName('contact_form__message')[0];
+const name = document.getElementsByClassName('contact_form__name')[0];
+const email = document.getElementsByClassName('contact_form__email')[0];
+const captcha = document.getElementsByClassName('contact_form__captcha')[0];
+const now = (new Date().getTime()/1000).toFixed();
+const feedback = document.getElementsByClassName('contact_form__feedback')[0];
 
-window.addEventListener('DOMContentLoaded', function() {
-    const contact_form = document.getElementsByClassName('content__contact_form')[0];
-    if (contact_form) {
-        contact_form.addEventListener('submit', wtf_submitContactForm, false);
-        wtf_startContactFormSession();
-    }
-});
-
-function wtf_startContactFormSession() {
+window.addEventListener('DOMContentLoaded', function(event) {
     let formData = new FormData();
     formData.append('action', 'start_session');
-    fetch(contactFormAjaxUrl, {
+    fetch(ajaxUrl, {
         method: 'POST',
         mode: 'same-origin',
         body: formData,
     })
-    .then(response => {
-        if (!response.ok) {
-            throw new Error('Response was not OK');
-        }
-
-        return response.json();
-    })
+    .then(response => response.json())
     .then(json => {
         console.log(json);
     })
-    .catch(error => {
-        console.error('Could not start the session:', error);
-    });
-}
+});
 
-function wtf_submitContactForm(event) {
+contact_form.addEventListener('submit', function(event) {
     event.preventDefault();
-
-    const subject = document.getElementsByClassName('contact_form__subject')[0];
-    const message = document.getElementsByClassName('contact_form__message')[0];
-    const name = document.getElementsByClassName('contact_form__name')[0];
-    const email = document.getElementsByClassName('contact_form__email')[0];
-    const captcha = document.getElementsByClassName('contact_form__captcha')[0];
-
     let formData = new FormData();
     formData.append('action', 'handle_form');
     formData.append('subject', subject.value);
@@ -55,14 +39,13 @@ function wtf_submitContactForm(event) {
         return;
     }
 
-    fetch(contactFormAjaxUrl, {
+    fetch(ajaxUrl, {
         method: 'POST',
         mode: 'same-origin',
         body: formData,
     })
     .then(response => response.json())
     .then(json => {
-        const feedback = document.getElementsByClassName('contact_form__feedback')[0];
         console.log(json);
         if (json.errors) {
             feedback.classList.remove('--success');
@@ -92,4 +75,4 @@ function wtf_submitContactForm(event) {
         }
     })
     .catch(error => console.log(error));
-}
+}, false);

assets/js/contact_form_toggle.js

@@ -1,7 +1,5 @@
 /* Unhide contact form if JS is enabled */
 window.addEventListener('DOMContentLoaded', (event) => {
     const contact_form_wrapper = document.getElementsByClassName('content__contact_form_wrapper')[0];
-    if (contact_form_wrapper) {
-        contact_form_wrapper.style.setProperty('display', 'block');
-    }
+    contact_form_wrapper.style.setProperty('display', 'block');
 });

assets/php/contact_form.php

@@ -6,16 +6,17 @@ function sanitize_text(string $name, string $type) {
         'text' => FILTER_SANITIZE_SPECIAL_CHARS,
         'email' => FILTER_SANITIZE_EMAIL,
     );
-    $text = filter_var(trim($_POST[$name]), $filters[$type]);
+    $text = trim($text);
+    $text = filter_var($_POST[$name], $filters[$type]);
     $text = stripslashes($text);
 
     return $text;
 }
 
-function prepare_message_body(string $message, string $name) {
+function prepare_message_body($message) {
     // Replace HTML-Entities with actual carriage returns and line feeds
     $message = str_replace("
", "\r", $message);
-    $message = str_replace("
", "\n", $message);
+    $message = str_replace("
", "\r", $message);
 
     // Ensure line breaks via carriage return + line feed
     $message = str_replace("\r\n", "\n", $message);
@@ -33,30 +34,22 @@ function prepare_message_body(string $message, string $name) {
  *  mail(): Braucht auf dem Server einen korrekt konfigurierten Mailserver
  *  phpmailer: Bibliothek, der per Composer installiert wird. Tut ganz gut mit SMTP.
  */
-function send_message_to_office(string $subject, string $message, string $name, string $email) {
-    $returnPath = filter_var(getenv('WTF_RETURN_PATH'), FILTER_VALIDATE_EMAIL);
-    $to = filter_var(getenv('WTF_CONTACT_TO'), FILTER_VALIDATE_EMAIL);
-
-    if (!$returnPath || !$to) {
-        error_log('Address for "To" or "Return-Path" is invalid');
-        return false;
-    }
-
+function send_message_to_office($subject, $message, $name, $email) {
     return mail(
-        $to,
+        getenv('WTF_CONTACT_TO'),
         "=?UTF-8?B?" . base64_encode($subject) . "?=",
-        prepare_message_body($message, $name),
-        array(
+        prepare_message_body($message),
+        $additional_headers = array(
             "From" => getenv('WTF_CONTACT_FROM'),
             "Reply-To" => $email,
+            "Return-Path" => getenv('WTF_RETURN_PATH'),
             "Content-Type" => "text/plain; charset=utf-8",
             "Content-Transfer-Encoding" => "base64",
         ),
-        "-f $returnPath"
     );
 }
 
-function send_response(array $response_data) {
+function send_response($response_data) {
     $json = json_encode($response_data);
     if ($json === false) {
         // Avoid echo of empty string (which is invalid JSON), and

templates/layout.html

@@ -63,7 +63,7 @@ __        ____________________
   {% if 'manifest.json'|asseturl is defined -%}
     <link rel="manifest" href="{{ 'manifest.json'|asseturl }}">
   {%- endif %}
-  {% if '/js/contact_form_toggle.js'|asseturl is defined and this.title == 'Kontakt' -%}
+  {% if '/js/contact_form_toggle.js'|asseturl is defined -%}
     <script type="text/javascript" src="{{ '/js/contact_form_toggle.js'|asseturl }}"></script>
   {%- endif %}
 </head>
@@ -131,7 +131,7 @@ __        ____________________
     {%- if '/js/nav_toggle.js'|asseturl is defined -%}
       <script type="text/javascript" src="{{ '/js/nav_toggle.js'|asseturl }}"></script>
     {%- endif %}
-    {% if '/js/contact_form_toggle.js'|asseturl is defined and this.title == 'Kontakt' -%}
+    {% if '/js/contact_form_toggle.js'|asseturl is defined -%}
       <script type="text/javascript" src="{{ '/js/contact_form.js'|asseturl }}"></script>
     {%- endif %}
 </body>