freetag 3.70.9: Avoid php warnings in RSS feed

Reported in s9y/Serendipity#858
s9y · Jan 18, 2025 · 4c9cb57 · 4c9cb57
1 parent da16610
commit 4c9cb57
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/serendipity_event_freetag/ChangeLog b/serendipity_event_freetag/ChangeLog
@@ -1,3 +1,5 @@
+3.70.9: Avoid more warnings under PHP 8.2, thanks @bauigel
+
 3.70.8: Avoid more warnings under PHP 8.2
 
 3.70.7: Avoid deprecation warning under PHP 8.2

diff --git a/serendipity_event_freetag/serendipity_event_freetag.php b/serendipity_event_freetag/serendipity_event_freetag.php
@@ -66,7 +66,7 @@ function introspect(&$propbag)
             'smarty'      => '2.6.7',
             'php'         => '7.0'
         ));
-        $propbag->add('version',       '3.70.8');
+        $propbag->add('version',       '3.70.9');
         $propbag->add('event_hooks',    array(
             'frontend_fetchentries'                             => true,
             'frontend_fetchentry'                               => true,
@@ -689,14 +689,14 @@ function event_hook($event, &$bag, &$eventData, $addData = null) {
 
                 case 'frontend_display:rss-2.0:per_entry':
                 case 'frontend_display:rss-0.91:per_entry':
-                    $eventData['display_dat'] .= $this->getFeedXmlForTags('category', $eventData['properties']['freetag_tags'] ?? []);
+                    $eventData['display_dat'] = ($eventData['display_dat'] ?? '') . $this->getFeedXmlForTags('category', $eventData['properties']['freetag_tags'] ?? []);
                     return true;
 
                 case 'frontend_display:rss-1.0:per_entry':
                 case 'frontend_display:rss-0.91:per_entry':
                 case 'frontend_display:atom-0.3:per_entry':
                 case 'frontend_display:atom-1.0:per_entry':
-                    $eventData['display_dat'] .= $this->getFeedXmlForTags('dc:subject', $eventData['properties']['freetag_tags']);
+                    $eventData['display_dat'] = ($eventData['display_dat'] ?? '') . $this->getFeedXmlForTags('dc:subject', $eventData['properties']['freetag_tags']);
                     return true;
 
                 case 'external_plugin':