[pageheader] Avoid notice

[friendica-addons.git/.git] / pageheader / pageheader.php

diff --git a/pageheader/pageheader.php b/pageheader/pageheader.php
index 977bee4..473c5e6 100644 (file)
--- a/pageheader/pageheader.php
+++ b/pageheader/pageheader.php
@@ -1,6 +1,4 @@
 <?php
-
-
 /**
  * Name: Page Header
  * Description: Inserts a page header
@@ -11,22 +9,25 @@
  */
 
 use Friendica\Core\Config;
+use Friendica\Core\Hook;
+use Friendica\Core\L10n;
+use Friendica\Core\Renderer;
 
 function pageheader_install() {
-    register_hook('page_content_top', 'addon/pageheader/pageheader.php', 'pageheader_fetch');
-       register_hook('plugin_settings', 'addon/pageheader/pageheader.php', 'pageheader_addon_settings');
-       register_hook('plugin_settings_post', 'addon/pageheader/pageheader.php', 'pageheader_addon_settings_post');
+    Hook::register('page_content_top', 'addon/pageheader/pageheader.php', 'pageheader_fetch');
+       Hook::register('addon_settings', 'addon/pageheader/pageheader.php', 'pageheader_addon_settings');
+       Hook::register('addon_settings_post', 'addon/pageheader/pageheader.php', 'pageheader_addon_settings_post');
 
 }
 
 
 function pageheader_uninstall() {
-    unregister_hook('page_content_top', 'addon/pageheader/pageheader.php', 'pageheader_fetch');
-       unregister_hook('plugin_settings', 'addon/pageheader/pageheader.php', 'pageheader_addon_settings');
-       unregister_hook('plugin_settings_post', 'addon/pageheader/pageheader.php', 'pageheader_addon_settings_post');
+    Hook::unregister('page_content_top', 'addon/pageheader/pageheader.php', 'pageheader_fetch');
+       Hook::unregister('addon_settings', 'addon/pageheader/pageheader.php', 'pageheader_addon_settings');
+       Hook::unregister('addon_settings_post', 'addon/pageheader/pageheader.php', 'pageheader_addon_settings_post');
 
        // hook moved, uninstall the old one if still there. 
-    unregister_hook('page_header', 'addon/pageheader/pageheader.php', 'pageheader_fetch');
+    Hook::unregister('page_header', 'addon/pageheader/pageheader.php', 'pageheader_fetch');
 
 }
 
@@ -42,21 +43,19 @@ function pageheader_addon_settings(&$a,&$s) {
 
     /* Add our stylesheet to the page so we can make our settings look nice */
 
-    $a->page['htmlhead'] .= '<link rel="stylesheet"  type="text/css" href="' . $a->get_baseurl() . '/addon/pageheader/pageheader.css' . '" media="all" />' . "\r\n";
+    $a->page['htmlhead'] .= '<link rel="stylesheet"  type="text/css" href="' . $a->getBaseURL() . '/addon/pageheader/pageheader.css' . '" media="all" />' . "\r\n";
 
 
        $words = Config::get('pageheader','text');
        if(! $words)
                $words = '';
 
-    $s .= '<div class="settings-block">';
-    $s .= '<h3>' . t('"pageheader" Settings') . '</h3>';
-    $s .= '<div id="pageheader-wrapper">';
-    $s .= '<label id="pageheader-label" for="pageheader-words">' . t('Message to display on every page on this server (or put a pageheader.html file in your docroot)') . ' </label>';
-    $s .= '<textarea id="pageheader-words" type="text" name="pageheader-words">' . $words . '</textarea>';
-    $s .= '</div><div class="clear"></div>';
-
-    $s .= '<div class="settings-submit-wrapper" ><input type="submit" id="pageheader-submit" name="pageheader-submit" class="settings-submit" value="' . t('Save Settings') . '" /></div></div>';
+       $t = Renderer::getMarkupTemplate("settings.tpl", "addon/pageheader/");
+       $s .= Renderer::replaceMacros($t, [
+                                       '$title' => L10n::t('"pageheader" Settings'),
+                                       '$phwords' => ['pageheader-words', L10n::t('Message'), $words, L10n::t('Message to display on every page on this server (or put a pageheader.html file in your docroot)')],
+                                       '$submit' => L10n::t('Save Settings')
+       ]);
 
        return;
 
@@ -67,9 +66,11 @@ function pageheader_addon_settings_post(&$a,&$b) {
        if(! is_site_admin())
                return;
 
-       if($_POST['pageheader-submit']) {
-               Config::set('pageheader','text',trim(strip_tags($_POST['pageheader-words'])));
-               info( t('pageheader Settings saved.') . EOL);
+       if(!empty($_POST['pageheader-submit'])) {
+               if (isset($_POST['pageheader-words'])) {
+                       Config::set('pageheader', 'text', trim(strip_tags($_POST['pageheader-words'])));
+               }
+               info(L10n::t('pageheader Settings saved.') . EOL);
        }
 }
 
@@ -82,7 +83,7 @@ function pageheader_fetch($a,&$b) {
        }
 
     $a->page['htmlhead'] .= '<link rel="stylesheet" type="text/css" href="'
-        . $a->get_baseurl() . '/addon/pageheader/pageheader.css' . '" media="all" />' . "\r\n";
+        . $a->getBaseURL() . '/addon/pageheader/pageheader.css' . '" media="all" />' . "\r\n";
     
     if(! $s)
         $s = '';