Fix rendering of securemail user settings
authorSt John Karp <git@fuzzjunket.com>
Sun, 24 Sep 2017 09:45:12 +0000 (02:45 -0700)
committerSt John Karp <git@fuzzjunket.com>
Sun, 24 Sep 2017 09:45:12 +0000 (02:45 -0700)
The securemail addon was overwriting all of the user settings form
rendered before it. I've added a concatenation operator to ensure
we're not losing any content on this page.

securemail/securemail.php

index 79f5e2dac8f71b53fa431cc305b7cb5e9e9708bd..b20eab9c21edb1522f47bb914be511ab3927a85f 100644 (file)
@@ -60,7 +60,7 @@ function securemail_settings(App &$a, &$s){
 
     $t = get_markup_template('admin.tpl', 'addon/securemail/');
 
-    $s = replace_macros($t, array(
+    $s .= replace_macros($t, array(
         '$title' => t('"Secure Mail" Settings'),
         '$submit' => t('Save Settings'),
         '$test' => t('Save and send test'), //NOTE: update also in 'post'