Update function names

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

diff --git a/showmore/showmore.php b/showmore/showmore.php
old mode 100755 (executable)
new mode 100644 (file)
index e4f0de4..f6e5028
--- a/showmore/showmore.php
+++ b/showmore/showmore.php
@@ -7,17 +7,19 @@
  *         based upon NSFW from Mike Macgirvin <http://macgirvin.com/profile/mike>
  *
  */
+use Friendica\Core\Addon;
+use Friendica\Core\PConfig;
 
 function showmore_install() {
-       register_hook('prepare_body', 'addon/showmore/showmore.php', 'showmore_prepare_body');
-       register_hook('plugin_settings', 'addon/showmore/showmore.php', 'showmore_addon_settings');
-       register_hook('plugin_settings_post', 'addon/showmore/showmore.php', 'showmore_addon_settings_post');
+       Addon::registerHook('prepare_body', 'addon/showmore/showmore.php', 'showmore_prepare_body');
+       Addon::registerHook('addon_settings', 'addon/showmore/showmore.php', 'showmore_addon_settings');
+       Addon::registerHook('addon_settings_post', 'addon/showmore/showmore.php', 'showmore_addon_settings_post');
 }
 
 function showmore_uninstall() {
-       unregister_hook('prepare_body', 'addon/showmore/showmore.php', 'showmore_prepare_body');
-       unregister_hook('plugin_settings', 'addon/showmore/showmore.php', 'showmore_addon_settings');
-       unregister_hook('plugin_settings_post', 'addon/showmore/showmore.php', 'showmore_addon_settings_post');
+       Addon::unregisterHook('prepare_body', 'addon/showmore/showmore.php', 'showmore_prepare_body');
+       Addon::unregisterHook('addon_settings', 'addon/showmore/showmore.php', 'showmore_addon_settings');
+       Addon::unregisterHook('addon_settings_post', 'addon/showmore/showmore.php', 'showmore_addon_settings_post');
 }
 
 function showmore_addon_settings(&$a,&$s) {
@@ -29,13 +31,19 @@ function showmore_addon_settings(&$a,&$s) {
 
        $a->page['htmlhead'] .= '<link rel="stylesheet" type="text/css" href="'.$a->get_baseurl().'/addon/showmore/showmore.css'.'" media="all"/>'."\r\n";
 
-       $enable_checked = (intval(get_pconfig(local_user(),'showmore','disable')) ? '' : ' checked="checked"');
-       $chars = get_pconfig(local_user(),'showmore','chars');
+       $enable_checked = (intval(PConfig::get(local_user(),'showmore','disable')) ? '' : ' checked="checked"');
+       $chars = PConfig::get(local_user(),'showmore','chars');
        if(!$chars)
                $chars = '1100';
 
-       $s .= '<div class="settings-block">';
+       $s .= '<span id="settings_showmore_inflated" class="settings-block fakelink" style="display: block;" onclick="openClose(\'settings_showmore_expanded\'); openClose(\'settings_showmore_inflated\');">';
        $s .= '<h3>' . t('"Show more" Settings').'</h3>';
+       $s .= '</span>';
+       $s .= '<div id="settings_showmore_expanded" class="settings-block" style="display: none;">';
+       $s .= '<span class="fakelink" onclick="openClose(\'settings_showmore_expanded\'); openClose(\'settings_showmore_inflated\');">';
+       $s .= '<h3>' . t('"Show more" Settings').'</h3>';
+       $s .= '</span>';
+
        $s .= '<div id="showmore-wrapper">';
 
        $s .= '<label id="showmore-enable-label" for="showmore-enable">'.t('Enable Show More').'</label>';
@@ -45,7 +53,7 @@ function showmore_addon_settings(&$a,&$s) {
        $s .= '<input id="showmore-words" type="text" name="showmore-chars" value="'.$chars.'" />';
        $s .= '</div><div class="clear"></div>';
 
-       $s .= '<div class="settings-submit-wrapper" ><input type="submit" id="showmore-submit" name="showmore-submit" class="settings-submit" value="' . t('Submit') . '" /></div>';
+       $s .= '<div class="settings-submit-wrapper" ><input type="submit" id="showmore-submit" name="showmore-submit" class="settings-submit" value="' . t('Save Settings') . '" /></div>';
 //     $s .= '<div class="showmore-desc">' . t('Use /expression/ to provide regular expressions') . '</div>';
        $s .= '</div>';
 
@@ -58,10 +66,10 @@ function showmore_addon_settings_post(&$a,&$b) {
                return;
 
        if($_POST['showmore-submit']) {
-               set_pconfig(local_user(),'showmore','chars',trim($_POST['showmore-chars']));
+               PConfig::set(local_user(),'showmore','chars',trim($_POST['showmore-chars']));
                $enable = ((x($_POST,'showmore-enable')) ? intval($_POST['showmore-enable']) : 0);
                $disable = 1-$enable;
-               set_pconfig(local_user(),'showmore','disable', $disable);
+               PConfig::set(local_user(),'showmore','disable', $disable);
                info( t('Show More Settings saved.') . EOL);
        }
 }
@@ -69,8 +77,15 @@ function showmore_addon_settings_post(&$a,&$b) {
 function get_body_length($body) {
        $string = trim($body);
 
+       // DomDocument doesn't like empty strings
+       if(! strlen($string)) {
+               return 0;
+       }
+
        // We need to get rid of hidden tags (display: none)
-       $dom = DomDocument::loadHTML($body);
+
+       // Get rid of the warning. It would be better to have some valid html as input
+       $dom = @DomDocument::loadHTML($body);
        $xpath = new DOMXPath($dom);
 
        /*
@@ -85,7 +100,7 @@ function get_body_length($body) {
                }
        }
        // Now we can get the body of our HTML DomDocument, it contains only what is visible
-       $string = $dom->saveHTML($dom->getElementsByTagName('body')->item(0));
+       $string = $dom->saveHTML();
 
        $string = strip_tags($string);
        return strlen($string);
@@ -94,10 +109,10 @@ function get_body_length($body) {
 function showmore_prepare_body(&$a,&$b) {
 
        $words = null;
-       if(get_pconfig(local_user(),'showmore','disable'))
+       if(PConfig::get(local_user(),'showmore','disable'))
                return;
 
-       $chars = (int)get_pconfig(local_user(),'showmore','chars');
+       $chars = (int)PConfig::get(local_user(),'showmore','chars');
        if(!$chars)
                $chars = 1100;
 
@@ -108,9 +123,9 @@ function showmore_prepare_body(&$a,&$b) {
 
        if($found) {
                $rnd = random_string(8);
-               $b['html'] = '<span id="showmore-teaser-'.$rnd.'" style="display: block;">'.$shortened." ".
-                               '<span id="showmore-wrap-'.$rnd.'" style="white-space:nowrap;" class="fakelink" onclick="openClose(\'showmore-'.$rnd.'\'); openClose(\'showmore-teaser-'.$rnd.'\');" >'.sprintf(t('show more')).'</span></span>'.
-                               '<div id="showmore-'.$rnd.'" style="display: none;">'.$b['html'].'</div>';
+               $b['html'] = '<span id="showmore-teaser-'.$rnd.'" class="showmore-teaser" style="display: block;">'.$shortened." ".
+                               '<span id="showmore-wrap-'.$rnd.'" style="white-space:nowrap;" class="showmore-wrap fakelink" onclick="openClose(\'showmore-'.$rnd.'\'); openClose(\'showmore-teaser-'.$rnd.'\');" >'.sprintf(t('show more')).'</span></span>'.
+                               '<div id="showmore-'.$rnd.'" class="showmore-content" style="display: none;">'.$b['html'].'</div>';
        }
 }
 
@@ -141,7 +156,7 @@ function showmore_cutitem($text, $limit) {
        @$doc->loadHTML($doctype."<html><body>".$text."</body></html>");
 
        $text = $doc->saveHTML();
-       $text = str_replace(array("<html><body>", "</body></html>", $doctype), array("", "", ""), $text);
+       $text = str_replace(["<html><body>", "</body></html>", $doctype], ["", "", ""], $text);
 
        return($text);
 }