fix "fopen_flags"

roundcube · Dec 16, 2023 · ef1d8a9 · ef1d8a9
1 parent 0bd227e
commit ef1d8a9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -60,6 +60,7 @@
         'heredoc_to_nowdoc' => false,
         'no_useless_else' => false,
         'phpdoc_no_empty_return' => false,
+        'psr_autoloading' => false,
         'single_line_comment_style' => false,
 
         // TODO
@@ -122,7 +123,6 @@
         'visibility_required' => false,
 
         // TODO - risky
-        'fopen_flags' => false,
         'function_to_constant' => false,
         'no_unset_on_property' => false,
         'php_unit_construct' => false,
@@ -131,7 +131,6 @@
         'php_unit_set_up_tear_down_visibility' => false,
         'php_unit_strict' => false,
         'php_unit_test_case_static_method_calls' => false,
-        'psr_autoloading' => false,
         'random_api_migration' => false,
         'self_accessor' => false,
         'static_lambda' => false,

diff --git a/program/include/rcmail_attachment_handler.php b/program/include/rcmail_attachment_handler.php
@@ -224,7 +224,7 @@ public function body($size = null, $fp = null)
                     $result = fwrite($fp, $size ? substr($attachment['data'], 0, $size) : $attachment['data']) !== false;
                 }
                 else if ($attachment['path']) {
-                    if ($fh = fopen($attachment['path'], 'rb')) {
+                    if ($fh = fopen($attachment['path'], 'r')) {
                         $result = stream_copy_to_stream($fh, $fp, $size ?: -1);
                     }
                 }