Additional work for PR 3778

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

diff --git a/forumlist/forumlist.php b/forumlist/forumlist.php
index 4ca063853fb8d82429c8dd80401cd13a2afc25a5..63a8de84111644156d0e6f09725ce257fdf6fdb1 100644 (file)
--- a/forumlist/forumlist.php
+++ b/forumlist/forumlist.php
@@ -6,6 +6,7 @@
  * Author: Mike Macgirvin <mike@macgirvin.com>
  * based on pages plugin by
  * Author: Michael Vogel <ike@piratenpartei.de>
+ * Status: Unsupported
  *
  */
 
@@ -65,7 +66,7 @@ function forumlist_network_mod_init($a,$b) {
 
        $forumlist .= '<div id="hide-forum-list" class="fakelink" onclick="openClose(\'forum-list\');" >' 
                                . t('show/hide') . '</div>'
-                               . '<div id="forum-list" style="display: none;">';
+                               . '<div role="menu" id="forum-list" style="display: none;">';
 
 
        $randomise = intval(get_pconfig(local_user(),'forumlist','randomise'));
@@ -74,13 +75,13 @@ function forumlist_network_mod_init($a,$b) {
 
        if(count($contacts)) {
                foreach($contacts as $contact) {
-                       $forumlist .= '<a href="' . $a->get_baseurl() . '/redir/' . $contact["id"] . '" title="' . $contact['url'] . '" class="label sparkle" target="external-link"><img class="forumlist-img" height="20" width="20" src="' . $contact['micro'] .'" alt="' . $contact['url'] . '" /></a> <a href="' . $a->get_baseurl() . '/network?f=&cid=' . $contact['id'] . '" >' . $contact["name"]."</a><br />";
+                       $forumlist .= '<div role="menuitem"><a href="' . $a->get_baseurl() . '/redir/' . $contact["id"] . '" title="'.t('External link to forum').'" class="label sparkle" target="_blank"><img class="forumlist-img" height="20" width="20" src="' . $contact['micro'] .'" alt="'.t('External link to forum').'" /></a> <a href="' . $a->get_baseurl() . '/network?f=&cid=' . $contact['id'] . '" >' . $contact["name"]."</a></div>";
                }
        }
        else {
                $forumlist .= t('No forum subscriptions');
        }
-               
+
        $forumlist .= "</div></div>";
        if (sizeof($contacts) > 0)
                $a->page['aside'] = $forumlist . $a->page['aside'];
@@ -124,14 +125,8 @@ function forumlist_profile_advanced($a,&$b) {
 
 
 function forumlist_plugin_settings_post($a,$post) {
-
-        logger('forum list save 1');
-
        if(! local_user() || (! x($_POST,'forumlist-settings-submit')))
                return;
-
-        logger('forum list save 2');
-
 //     set_pconfig(local_user(),'forumlist','max_forumlists',intval($_POST['forumlist_max_forumlists']));
        set_pconfig(local_user(),'forumlist','randomise',intval($_POST['forumlist_random']));
        set_pconfig(local_user(),'forumlist','show_on_profile',intval($_POST['forumlist_profile']));
@@ -160,12 +155,17 @@ function forumlist_plugin_settings(&$a,&$s) {
 
        $network = intval(get_pconfig(local_user(),'forumlist','show_on_network'));
        $network_checked = (($network) ? ' checked="checked" ' : '');
-       
-       
+
+
        /* Add some HTML to the existing form */
+       $s .= '<span id="settings_forumlist_inflated" class="settings-block fakelink" style="display: block;" onclick="openClose(\'settings_forumlist_expanded\'); openClose(\'settings_forumlist_inflated\');">';
+       $s .= '<h3>' . t('Forumlist') . '</h3>';
+       $s .= '</span>';
+       $s .= '<div id="settings_forumlist_expanded" class="settings-block" style="display: none;">';
+       $s .= '<span class="fakelink" onclick="openClose(\'settings_forumlist_expanded\'); openClose(\'settings_forumlist_inflated\');">';
+       $s .= '<h3>' . t('Forumlist') . '</h3>';
+       $s .= '</span>';
 
-       $s .= '<div class="settings-block">';
-       $s .= '<h3>' . t('Forumlist Settings') . '</h3>';
        $s .= '<div id="forumlist-settings-wrapper">';
        $s .= '<label id="forumlist-random-label" for="forumlist-random">' . t('Randomise forum list') . '</label>';
        $s .= '<input id="forumlist-random" type="checkbox" name="forumlist_random" value="1" ' . $randomise_checked . '/>';
@@ -181,7 +181,7 @@ function forumlist_plugin_settings(&$a,&$s) {
 
        /* provide a submit button */
 
-       $s .= '<div class="settings-submit-wrapper" ><input type="submit" name="forumlist-settings-submit" class="settings-submit" value="' . t('Submit') . '" /></div></div>';
+       $s .= '<div class="settings-submit-wrapper" ><input type="submit" name="forumlist-settings-submit" class="settings-submit" value="' . t('Save Settings') . '" /></div></div>';
 
 }