minor improvements

Author: dannyvankooten

includes/allow-svg-uploads.php

@@ -1,6 +1,8 @@
 <?php
 
 add_filter('upload_mimes', function (array $mime_types): array {
-    $mime_types['svg'] = 'image/svg+xml';
+    if (current_user_can('manage_options')) {
+        $mime_types['svg'] = 'image/svg+xml';
+    }
     return $mime_types;
 });

includes/purge-bunny-cdn-cache.php

@@ -4,34 +4,42 @@
 
 function purge_cache_for_url(string $url)
 {
-    $url = urlencode($url);
-    $curl = curl_init();
-    curl_setopt_array($curl, [
-        CURLOPT_URL => "https://api.bunny.net/purge?url=$url",
-        CURLOPT_RETURNTRANSFER => true,
-        CURLOPT_POST => true,
-        CURLOPT_MAXREDIRS => 5,
-        CURLOPT_TIMEOUT => 10,
-        CURLOPT_HTTPHEADER => [
-            "AccessKey: " . constant('BUNNY_API_KEY'),
+    $request_url = 'https://api.bunny.net/purge?url=' . urlencode($url);
+
+    $response = wp_remote_post($request_url, [
+        'timeout' => 10,
+        'redirection' => 5,
+        'headers' => [
+            'AccessKey' => constant('BUNNY_API_KEY'),
         ],
     ]);
 
-    curl_exec($curl);
-    $error = curl_error($curl);
-    if ($error) {
-        error_log("Error purging Bunny CDN cache: $error");
+    if (is_wp_error($response)) {
+        error_log('Error purging Bunny CDN cache: ' . $response->get_error_message());
+        return;
+    }
+
+    $status_code = wp_remote_retrieve_response_code($response);
+    if ($status_code < 200 || $status_code >= 300) {
+        error_log('Error purging Bunny CDN cache: HTTP ' . $status_code);
     }
 }
 
 add_action('save_post', function ($post_id) {
+    // No-op if BUNNY_API_KEY constant is not set
+    if (! defined('BUNNY_API_KEY')) {
+        return;
+    }
+
     // Check if it's not an autosave
-    if (defined('DOING_AUTOSAVE') && DOING_AUTOSAVE) {
+    if ((defined('DOING_AUTOSAVE') && DOING_AUTOSAVE) || wp_is_post_autosave($post_id) || wp_is_post_revision($post_id)) {
         return;
     }
 
-    // No-op if BUNNY_API_KEY constant is not set
-    if (! defined('BUNNY_API_KEY')) {
+
+    $post_type = get_post_type($post_id);
+    $post_type_object = $post_type ? get_post_type_object($post_type) : null;
+    if (! $post_type_object || ! $post_type_object->public) {
         return;
     }
 

includes/set-cache-headers.php

@@ -21,8 +21,8 @@
 
     // cache feeds and XML files (ie sitemap) for 1 day
     } elseif (is_feed() || str_ends_with($_SERVER['REQUEST_URI'] ?? '', '.xml')) {
-        $headers['Cache-Control'] = 'public, max-age=81600';
-    // cache all other pages for 1 year
+        $headers['Cache-Control'] = 'public, max-age=86400';
+    // cache all other pages for 30 days
     } else {
         $headers['Cache-Control'] = 'public, max-age=2592000';
     }

includes/stop-comment-spam.php

@@ -40,7 +40,7 @@
         || (strlen($commentdata['comment_author_email']) > 0 && preg_match('/^\w+_\w+@(yahoo|gmail|hotmail)\.com$/', $commentdata['comment_author_email']) && strlen($commentdata['comment_author_url']) > 0)
 
         // if comment contains a russian character
-        || (mb_strpos($commentdata['comment_content'], "н") !== false)
+        || (function_exists('mb_strpos') && mb_strpos($commentdata['comment_content'], "н") !== false)
 
         // if URL is given and does not contain at least one dot
         || (strlen($commentdata['comment_author_url']) > 0 && !str_contains($commentdata['comment_author_url'], '.'))