MrPetovan/friendica
2
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #3928 from zeroadam/ForumManager-#3878

Browse source 
ForumManager to src
This commit is contained in:
Hypolite Petovan 2017-11-21 11:54:19 -05:00 committed by GitHub
parent 33ad6b6d7e 7cee150868
commit 4eb8e43daf
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
5 changed files with 77 additions and 74 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
include/identity.php
View file

@ -4,6 +4,7 @@
*/ */
use Friendica\App; use Friendica\App;
use Friendica\Content\ForumManager;
use Friendica\Core\Cache; use Friendica\Core\Cache;
use Friendica\Core\Config; use Friendica\Core\Config;
use Friendica\Core\PConfig; use Friendica\Core\PConfig;
@ -12,7 +13,6 @@ use Friendica\Core\Worker;
use Friendica\Database\DBM; use Friendica\Database\DBM;
use Friendica\Object\Contact; use Friendica\Object\Contact;
require_once 'include/ForumManager.php';
require_once 'include/bbcode.php'; require_once 'include/bbcode.php';
require_once 'mod/proxy.php'; require_once 'mod/proxy.php';
@ -795,7 +795,7 @@ function advanced_profile(App $a)
//show subcribed forum if it is enabled in the usersettings //show subcribed forum if it is enabled in the usersettings
if (feature_enabled($uid, 'forumlist_profile')) { if (feature_enabled($uid, 'forumlist_profile')) {
$profile['forumlist'] = array( t('Forums:'), ForumManager::profile_advanced($uid)); $profile['forumlist'] = array( t('Forums:'), ForumManager::profileAdvanced($uid));
} }
if ($a->profile['uid'] == local_user()) { if ($a->profile['uid'] == local_user()) {

8
mod/network.php
View file

@ -1,6 +1,9 @@
<?php <?php
/**
* @file mod/network.php
*/
use Friendica\App; use Friendica\App;
use Friendica\Content\ForumManager;
use Friendica\Core\System; use Friendica\Core\System;
use Friendica\Core\Config; use Friendica\Core\Config;
use Friendica\Core\PConfig; use Friendica\Core\PConfig;
@ -11,7 +14,6 @@ require_once 'include/conversation.php';
require_once 'include/group.php'; require_once 'include/group.php';
require_once 'include/contact_widgets.php'; require_once 'include/contact_widgets.php';
require_once 'include/items.php'; require_once 'include/items.php';
require_once 'include/ForumManager.php';
require_once 'include/acl_selectors.php'; require_once 'include/acl_selectors.php';
function network_init(App $a) { function network_init(App $a) {
@ -152,7 +154,7 @@ function network_init(App $a) {
} }
$a->page['aside'] .= (feature_enabled(local_user(),'groups') ? group_side('network/0','network','standard',$group_id) : ''); $a->page['aside'] .= (feature_enabled(local_user(),'groups') ? group_side('network/0','network','standard',$group_id) : '');
$a->page['aside'] .= (feature_enabled(local_user(),'forumlist_widget') ? ForumManager::widget(local_user(),$cid) : ''); $a->page['aside'] .= (feature_enabled(local_user(), 'forumlist_widget') ? ForumManager::widget(local_user(), $cid) : '');
$a->page['aside'] .= posted_date_widget('network',local_user(),false); $a->page['aside'] .= posted_date_widget('network',local_user(),false);
$a->page['aside'] .= networks_widget('network',(x($_GET, 'nets') ? $_GET['nets'] : '')); $a->page['aside'] .= networks_widget('network',(x($_GET, 'nets') ? $_GET['nets'] : ''));
$a->page['aside'] .= saved_searches($search); $a->page['aside'] .= saved_searches($search);

4
mod/ping.php
View file

@ -3,6 +3,7 @@
* @file include/ping.php * @file include/ping.php
*/ */
use Friendica\App; use Friendica\App;
use Friendica\Content\ForumManager;
use Friendica\Core\Cache; use Friendica\Core\Cache;
use Friendica\Core\System; use Friendica\Core\System;
use Friendica\Core\PConfig; use Friendica\Core\PConfig;
@ -12,7 +13,6 @@ use Friendica\Util\XML;
require_once 'include/datetime.php'; require_once 'include/datetime.php';
require_once 'include/bbcode.php'; require_once 'include/bbcode.php';
require_once 'include/ForumManager.php';
require_once 'include/group.php'; require_once 'include/group.php';
require_once 'mod/proxy.php'; require_once 'mod/proxy.php';
require_once 'include/enotify.php'; require_once 'include/enotify.php';
@ -162,7 +162,7 @@ function ping_init(App $a)
} }
if (intval(feature_enabled(local_user(), 'forumlist_widget'))) { if (intval(feature_enabled(local_user(), 'forumlist_widget'))) {
$forum_counts = ForumManager::count_unseen_items(); $forum_counts = ForumManager::countUnseenItems();
if (DBM::is_result($forums_counts)) { if (DBM::is_result($forums_counts)) {
foreach ($forums_counts as $forum_count) { foreach ($forums_counts as $forum_count) {
if ($forum_count['count'] > 0) { if ($forum_count['count'] > 0) {

93
include/ForumManager.php → src/Content/ForumManager.php
View file

@ -1,39 +1,37 @@
<?php <?php
/**
* @file src/Content/ForumManager.php
* @brief ForumManager class with its methods related to forum functionality
*/
namespace Friendica\Content;
use Friendica\App; use Friendica\App;
use Friendica\Core\System; use Friendica\Core\System;
use Friendica\Database\DBM; use Friendica\Database\DBM;
use dba;
/** /**
* @file include/ForumManager.php * @brief This class handles methods related to the forum functionality
* @brief ForumManager class with its methods related to forum functionality *
*/ */
class ForumManager
/** {
* @brief This class handles metheods related to the forum functionality
*/
class ForumManager {
/** /**
* @brief Function to list all forums a user is connected with * @brief Function to list all forums a user is connected with
* *
* @param int $uid of the profile owner * @param int $uid of the profile owner
* @param boolean $showhidden * @param boolean $lastitem Sort by lastitem
* Show frorums which are not hidden * @param boolean $showhidden Show frorums which are not hidden
* @param boolean $lastitem * @param boolean $showprivate Show private groups
* Sort by lastitem
* @param boolean $showprivate
* Show private groups
* *
* @returns array * @return array
* 'url' => forum url * 'url' => forum url
* 'name' => forum name * 'name' => forum name
* 'id' => number of the key from the array * 'id' => number of the key from the array
* 'micro' => contact photo in format micro * 'micro' => contact photo in format micro
* 'thumb' => contact photo in format thumb * 'thumb' => contact photo in format thumb
*/ */
public static function get_list($uid, $showhidden = true, $lastitem, $showprivate = false) { public static function getList($uid, $lastitem, $showhidden = true, $showprivate = false)
{
$forumlist = array(); $forumlist = array();
$order = (($showhidden) ? '' : ' AND NOT `hidden` '); $order = (($showhidden) ? '' : ' AND NOT `hidden` ');
@ -43,7 +41,9 @@ class ForumManager {
$select = '(`forum` OR `prv`)'; $select = '(`forum` OR `prv`)';
} }
$contacts = dba::p("SELECT `contact`.`id`, `contact`.`url`, `contact`.`name`, `contact`.`micro`, `contact`.`thumb` FROM `contact` $contacts = dba::p(
"SELECT `contact`.`id`, `contact`.`url`, `contact`.`name`, `contact`.`micro`, `contact`.`thumb`
FROM `contact`
WHERE `network`= 'dfrn' AND $select AND `uid` = ? WHERE `network`= 'dfrn' AND $select AND `uid` = ?
AND NOT `blocked` AND NOT `hidden` AND NOT `pending` AND NOT `archive` AND NOT `blocked` AND NOT `hidden` AND NOT `pending` AND NOT `archive`
AND `success_update` > `failure_update` AND `success_update` > `failure_update`
@ -51,8 +51,9 @@ class ForumManager {
$uid $uid
); );
if (!$contacts) if (!$contacts) {
return($forumlist); return($forumlist);
}
while ($contact = dba::fetch($contacts)) { while ($contact = dba::fetch($contacts)) {
$forumlist[] = array( $forumlist[] = array(
@ -76,30 +77,28 @@ class ForumManager {
* in the settings, it appears at the notwork page sidebar * in the settings, it appears at the notwork page sidebar
* *
* @param int $uid The ID of the User * @param int $uid The ID of the User
* @param int $cid * @param int $cid The contact id which is used to mark a forum as "selected"
* The contact id which is used to mark a forum as "selected"
* @return string * @return string
*/ */
public static function widget($uid,$cid = 0) { public static function widget($uid, $cid = 0)
{
if(! intval(feature_enabled(local_user(),'forumlist_widget'))) if (! intval(feature_enabled(local_user(), 'forumlist_widget'))) {
return; return;
}
$o = ''; $o = '';
//sort by last updated item //sort by last updated item
$lastitem = true; $lastitem = true;
$contacts = self::get_list($uid,true,$lastitem, true); $contacts = self::getList($uid, $lastitem, true, true);
$total = count($contacts); $total = count($contacts);
$visible_forums = 10; $visible_forums = 10;
if (DBM::is_result($contacts)) { if (DBM::is_result($contacts)) {
$id = 0; $id = 0;
foreach($contacts as $contact) { foreach ($contacts as $contact) {
$selected = (($cid == $contact['id']) ? ' forum-selected' : ''); $selected = (($cid == $contact['id']) ? ' forum-selected' : '');
$entry = array( $entry = array(
@ -116,14 +115,16 @@ class ForumManager {
$tpl = get_markup_template('widget_forumlist.tpl'); $tpl = get_markup_template('widget_forumlist.tpl');
$o .= replace_macros($tpl,array( $o .= replace_macros(
$tpl,
array(
'$title' => t('Forums'), '$title' => t('Forums'),
'$forums' => $entries, '$forums' => $entries,
'$link_desc' => t('External link to forum'), '$link_desc' => t('External link to forum'),
'$total' => $total, '$total' => $total,
'$visible_forums' => $visible_forums, '$visible_forums' => $visible_forums,
'$showmore' => t('show more'), '$showmore' => t('show more'))
)); );
} }
return $o; return $o;
@ -137,13 +138,13 @@ class ForumManager {
* *
* @param int $uid The ID of the User * @param int $uid The ID of the User
* @return string * @return string
*
*/ */
public static function profile_advanced($uid) { public static function profileAdvanced($uid)
{
$profile = intval(feature_enabled($uid,'forumlist_profile')); $profile = intval(feature_enabled($uid, 'forumlist_profile'));
if(! $profile) if (! $profile) {
return; return;
}
$o = ''; $o = '';
@ -153,21 +154,23 @@ class ForumManager {
//don't sort by last updated item //don't sort by last updated item
$lastitem = false; $lastitem = false;
$contacts = self::get_list($uid,false,$lastitem,false); $contacts = self::getList($uid, $lastitem, false, false);
$total_shown = 0; $total_shown = 0;
foreach($contacts as $contact) { foreach ($contacts as $contact) {
$forumlist .= micropro($contact,false,'forumlist-profile-advanced'); $forumlist .= micropro($contact, false, 'forumlist-profile-advanced');
$total_shown ++; $total_shown ++;
if($total_shown == $show_total) if ($total_shown == $show_total) {
break; break;
} }
}
if(count($contacts) > 0) if (count($contacts) > 0) {
$o .= $forumlist; $o .= $forumlist;
return $o; return $o;
} }
}
/** /**
* @brief count unread forum items * @brief count unread forum items
@ -178,10 +181,11 @@ class ForumManager {
* 'id' => contact id * 'id' => contact id
* 'name' => contact/forum name * 'name' => contact/forum name
* 'count' => counted unseen forum items * 'count' => counted unseen forum items
*
*/ */
public static function count_unseen_items() { public static function countUnseenItems()
$r = q("SELECT `contact`.`id`, `contact`.`name`, COUNT(*) AS `count` FROM `item` {
$r = q(
"SELECT `contact`.`id`, `contact`.`name`, COUNT(*) AS `count` FROM `item`
INNER JOIN `contact` ON `item`.`contact-id` = `contact`.`id` INNER JOIN `contact` ON `item`.`contact-id` = `contact`.`id`
WHERE `item`.`uid` = %d AND `item`.`visible` AND NOT `item`.`deleted` AND `item`.`unseen` WHERE `item`.`uid` = %d AND `item`.`visible` AND NOT `item`.`deleted` AND `item`.`unseen`
AND `contact`.`network`= 'dfrn' AND (`contact`.`forum` OR `contact`.`prv`) AND `contact`.`network`= 'dfrn' AND (`contact`.`forum` OR `contact`.`prv`)
@ -194,5 +198,4 @@ class ForumManager {
return $r; return $r;
} }
} }

16
view/theme/vier/theme.php
View file

@ -10,6 +10,7 @@
*/ */
use Friendica\App; use Friendica\App;
use Friendica\Content\ForumManager;
use Friendica\Core\Config; use Friendica\Core\Config;
use Friendica\Core\PConfig; use Friendica\Core\PConfig;
use Friendica\Core\System; use Friendica\Core\System;
@ -210,9 +211,6 @@ function vier_community_info() {
//Community_Pages at right_aside //Community_Pages at right_aside
if ($show_pages && local_user()) { if ($show_pages && local_user()) {
require_once 'include/ForumManager.php';
if (x($_GET, 'cid') && intval($_GET['cid']) != 0) { if (x($_GET, 'cid') && intval($_GET['cid']) != 0) {
$cid = $_GET['cid']; $cid = $_GET['cid'];
} }
@ -220,16 +218,14 @@ function vier_community_info() {
//sort by last updated item //sort by last updated item
$lastitem = true; $lastitem = true;
$contacts = ForumManager::get_list($a->user['uid'],true,$lastitem, true); $contacts = ForumManager::getList($a->user['uid'], $lastitem, true, true);
$total = count($contacts); $total = count($contacts);
$visible_forums = 10; $visible_forums = 10;
if (count($contacts)) { if (count($contacts)) {
$id = 0; $id = 0;
foreach ($contacts as $contact) { foreach ($contacts as $contact) {
$selected = (($cid == $contact['id']) ? ' forum-selected' : ''); $selected = (($cid == $contact['id']) ? ' forum-selected' : '');
$entry = array( $entry = array(
@ -247,14 +243,16 @@ function vier_community_info() {
$tpl = get_markup_template('widget_forumlist_right.tpl'); $tpl = get_markup_template('widget_forumlist_right.tpl');
$page .= replace_macros($tpl, array( $page .= replace_macros(
$tpl,
array(
'$title' => t('Forums'), '$title' => t('Forums'),
'$forums' => $entries, '$forums' => $entries,
'$link_desc' => t('External link to forum'), '$link_desc' => t('External link to forum'),
'$total' => $total, '$total' => $total,
'$visible_forums' => $visible_forums, '$visible_forums' => $visible_forums,
'$showmore' => t('show more'), '$showmore' => t('show more'))
)); );
$aside['$page'] = $page; $aside['$page'] = $page;
} }