Update sign_msg message & make compose_delivery_receipt checked by default

Author: Shadow243

language/az.php

@@ -411,7 +411,7 @@
     'Added SMTP server!' => 'SMTP server əlavə edilmişdir!',
     'Added repository' => 'Anbar əlavə edilmişdir',
     'Added server!' => 'Server əlavə edilmişdir',
-    'Are you sure you wanto to delete this server? ' => 'Siz əminsiniz?',
+    'Are you sure you want to delete this server?' => 'Siz əminsiniz?',
     'Could not find that repository/owner combination at github.com' => 'Github.com-da kombinasiya anbar/sahib tapılmamışdır',
     'Disable prompts when deleting' => 'Silinmə vaxtı sorğunu kəsmək',
     'Enable keyboard shortcuts' => 'Qaynar klavişləri qoşmaq',

modules/core/message_list_functions.php

@@ -120,14 +120,13 @@ function message_list_meta($input, $output_mod) {
         $since = DEFAULT_SINCE;
     }
     $date = sprintf('%s', mb_strtolower($output_mod->trans($times[$since])));
-    $max = sprintf($output_mod->trans('sources@%d each'), $limit);
+    $max = sprintf($output_mod->trans('%d items per source'), $limit);
 
     return '<div class="list_meta d-flex align-items-center fs-6">'.
         $date.
-        '<b>-</b>'.
+        '<b> :</b>'.
         '<span class="src_count"></span> '.$max.
-        '<b>-</b>'.
-        '<span class="total"></span> '.$output_mod->trans('total').'</div>';
+        '</div>';
 }}
 
 /**

modules/profiles/modules.php

@@ -213,7 +213,7 @@ protected function output() {
  */
 class Hm_Output_compose_signature_button extends Hm_Output_Module {
     protected function output() {
-        return '<input type="hidden" value="'.$this->trans('You need at least one configured profile to sign messages').'" id="sign_msg" />'.
+        return '<input type="hidden" value="'.$this->trans('You need at least one profile with a signature to sign messages. Please configure a signature in your profile settings.').'" id="sign_msg" />'.
             '<input class="compose_sign btn btn-light float-end mt-3 me-2 border" type="button" value="'.$this->trans('Sign').'" />';
     }
 }

modules/smtp/modules.php

@@ -1193,7 +1193,7 @@ protected function output() {
                     (!$html ? '<label for="compose_body">'.$this->trans('Message').'</label>': '').
                 '</div>';
                 if($this->get('enable_compose_delivery_receipt_setting')) {
-                    $res .= '<div class="form-check mb-3"><input value="0" name="compose_delivery_receipt" id="compose_delivery_receipt" type="checkbox" class="form-check-input" /><label for="compose_delivery_receipt" class="form-check-label">'.$this->trans('Request a delivery receipt').'</label></div>';
+                    $res .= '<div class="form-check mb-3"><input value="1" name="compose_delivery_receipt" id="compose_delivery_receipt" type="checkbox" class="form-check-input" checked/><label for="compose_delivery_receipt" class="form-check-label">'.$this->trans('Request a delivery receipt').'</label></div>';
                 }
         if ($html == 2) {
             $res .= '<link href="'.WEB_ROOT.'modules/smtp/assets/markdown/editor.css" rel="stylesheet" />'.

tests/phpunit/modules/core/message_list_functions.php

@@ -29,9 +29,9 @@ public function test_get_message_list_settings() {
     public function test_get_message_list_meta() {
         $mod = new Hm_Output_Test(array('foo' => 'bar', 'bar' => 'foo'), array('bar'));
         $this->assertEquals('', message_list_meta(array(), $mod));
-        $this->assertEquals('<div class="list_meta d-flex align-items-center fs-6">last 7 days<b>-</b><span class="src_count"></span> sources@20 each<b>-</b><span class="total"></span> total</div>', message_list_meta(array('list_meta' => 'foo', 'message_list_since' => '-1 week'), $mod));
-        $this->assertEquals('<div class="list_meta d-flex align-items-center fs-6">last 7 days<b>-</b><span class="src_count"></span> sources@5 each<b>-</b><span class="total"></span> total</div>', message_list_meta(array('list_meta' => 'foo', 'per_source_limit' => 5), $mod));
-        $this->assertEquals('<div class="list_meta d-flex align-items-center fs-6">last 7 days<b>-</b><span class="src_count"></span> sources@5 each<b>-</b><span class="total"></span> total</div>', message_list_meta(array('list_meta' => 'foo', 'per_source_limit' => 5, 'message_list_since' => '-1 week'), $mod));
+        $this->assertEquals('<div class="list_meta fs-6">last 7 days<b> :</b><span class="src_count"></span> 20 items per source</div>', message_list_meta(array('list_meta' => 'foo', 'message_list_since' => '-1 week'), $mod));
+        $this->assertEquals('<div class="list_meta fs-6">last 7 days<b> :</b><span class="src_count"></span> 5 items per source</div>', message_list_meta(array('list_meta' => 'foo', 'per_source_limit' => 5), $mod));
+        $this->assertEquals('<div class="list_meta fs-6">last 7 days<b> :</b><span class="src_count"></span> 5 items per source</div>', message_list_meta(array('list_meta' => 'foo', 'per_source_limit' => 5, 'message_list_since' => '-1 week'), $mod));
     }
     /**
      * @preserveGlobalState disabled