fix BookingCodesMessage:sendMessage() wrong $attachment for prepareMail

[nelarsen]

Mit Bezug auf #1605 , das Ziel dieser PR ist, diese Fehlermeldungen in phpunit unter "WP latest on PHP 8.2" loszuwerden. Diese Sequenz trat insgesamt zwei mal auf und wurden durch die Test-Funktionen Service::BookingCodesTest::testSendBookingCodesMessage() und View::BookingCodesTest::testSendBookingCodesMessage() ausgelöst. Siehe Commitbeschreibung für Details.

PHP Deprecated:  preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated in /tmp/wordpress/wp-includes/PHPMailer/PHPMailer.php on line 1880
PHP Deprecated:  is_file(): Passing null to parameter #1 ($filename) of type string is deprecated in /tmp/wordpress/wp-includes/PHPMailer/PHPMailer.php on line 1895
PHP Deprecated:  strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in /tmp/wordpress/wp-includes/PHPMailer/PHPMailer.php on line 1897

Mit dieser PR verschwinden die o.g. Warnungen, aber sonst ändert sich nichts.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.98%. Comparing base (076ae51) to head (871f0c7).
Report is 2 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1606      +/-   ##
============================================
+ Coverage     49.79%   49.98%   +0.18%     
  Complexity     2682     2682              
============================================
  Files            97       97              
  Lines         10712    10712              
============================================
+ Hits           5334     5354      +20     
+ Misses         5378     5358      -20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[hansmorb]

Danke für den PR! Das mit dem doppelt verschaltelten Array im Attachment haben wir wohl übersehen! Dann hätte das attachment aber eigentlich vorher nicht funktionieren sollen, oder?

[nelarsen]

Danke für den PR! Das mit dem doppelt verschaltelten Array im Attachment haben wir wohl übersehen! Dann hätte das attachment aber eigentlich vorher nicht funktionieren sollen, oder?

Doch, tatsächlich funktionert beides. Das hat mich auch gewundert und ich habe mich gefragt, ob mein Fix doch etwas bricht. Aber das ist nicht der Fall, denn in Message::addStringAttachments() gibt es "Magie" (is_multidimensional_array), die mit mehrdimensionellen Arrays umgeht.

[hansmorb]

Na dann! Die Magie ist ein importiertes Snippet ;)