Return types and docblocks for new methods

Author: pabzm

program/actions/mail/get.php

@@ -349,7 +349,13 @@ public static function message_part_frame($attrib)
         return html::iframe($attrib);
     }
 
-    protected function send_html($contents, $inline_warning = null)
+    /**
+     * @param $contents string Content to send as HTTP body.
+     * @param $inline_warning string Something to inject into the beginning of the content.
+     *
+     * @return void
+     */
+    protected function send_html($contents, $inline_warning = null): void
     {
         $rcmail = rcmail::get_instance();
         $rcmail->output->reset(true);
@@ -383,7 +389,14 @@ protected function send_html($contents, $inline_warning = null)
         $rcmail->output->sendExit();
     }
 
-    protected function make_inline_warning($text, $button_label = null, $button_url = null)
+    /**
+     * @param $text string Text content
+     * @param $button_label string Text for the optional button to append to the content
+     * @param $button_url string URL of the button
+
+     * @return string HTML code as string
+     */
+    protected function make_inline_warning($text, $button_label = null, $button_url = null): string
     {
         $text = html::span(null, $text);