fossasia/engelsystem

View on GitHub
includes/controller/user_news_controller.php

Summary

Maintainability
C
1 day
Test Coverage
<?php
function user_news_comments_title() {
return _("News comments");
}
 
function news_title() {
return _("News");
}
 
function meetings_title() {
return _("Meetings");
}
 
Function `user_meetings` has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.
function user_meetings() {
global $DISPLAY_NEWS, $privileges, $user;
 
$html = '<div class="col-md-12"><h1>' . meetings_title() . '</h1>' . msg();
 
if (isset($_REQUEST['page']) && preg_match("/^[0-9]{1,}$/", $_REQUEST['page']))
$page = $_REQUEST['page'];
else
$page = 0;
 
$news = select_news_treffen($page, $DISPLAY_NEWS);
foreach ($news as $entry)
$html .= display_news($entry);
 
$dis_rows = ceil(select_news() / $DISPLAY_NEWS);
$html .= '<div class="text-center">' . '<ul class="pagination">';
Similar blocks of code found in 2 locations. Consider refactoring.
for ($i = 0; $i < $dis_rows; $i ++) {
if (isset($_REQUEST['page']) && $i == $_REQUEST['page'])
$html .= '<li class="active">';
elseif (! isset($_REQUEST['page']) && $i == 0)
$html .= '<li class="active">';
else
$html .= '<li>';
$html .= '<a href="' . page_link_to("user_meetings") . '&page=' . $i . '">' . ($i + 1) . '</a></li>';
}
$html .= '</ul></div></div>';
 
return $html;
}
 
function display_news($news) {
global $privileges, $p;
 
$html = '';
$html .= '<div class="panel' . ($news['Treffen'] == 1 ? ' panel-info' : ' panel-default') . '">';
$html .= '<div class="panel-heading">';
$html .= '<h3 class="panel-title">' . ($news['Treffen'] == 1 ? '[Meeting] ' : '') . ReplaceSmilies($news['Betreff']) . '</h3>';
$html .= '</div>';
$html .= '<div class="panel-body">' . ReplaceSmilies(nl2br($news['Text'])) . '</div>';
 
$html .= '<div class="panel-footer text-muted">';
if (in_array("admin_news", $privileges))
$html .= '<div class="pull-right">' . button_glyph(page_link_to("admin_news") . '&action=edit&id=' . $news['ID'], 'edit', 'btn-xs') . '</div>';
 
$html .= '<span class="glyphicon glyphicon-time"></span> ' . date("Y-m-d H:i", $news['Datum']) . '&emsp;';
 
$user_source = User($news['UID']);
if ($user_source === false)
engelsystem_error(_("Unable to load user."));
 
$html .= User_Nick_render($user_source);
if ($p != "news_comments")
$html .= '&emsp;<a href="' . page_link_to("news_comments") . '&nid=' . $news['ID'] . '"><span class="glyphicon glyphicon-comment"></span> ' . _("Comments") . ' &raquo;</a> <span class="badge">' . sql_num_query("SELECT * FROM `NewsComments` WHERE `Refid`='" . sql_escape($news['ID']) . "'") . '</span>';
$html .= '</div>';
$html .= '</div>';
return $html;
}
 
Method `user_news_comments` has 34 lines of code (exceeds 25 allowed). Consider refactoring.
Function `user_news_comments` has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.
function user_news_comments() {
global $user;
 
$html = '<div class="col-md-12"><h1>' . user_news_comments_title() . '</h1>';
if (isset($_REQUEST["nid"]) && preg_match("/^[0-9]{1,}$/", $_REQUEST['nid']) && count_news_by_id($_REQUEST['nid']) > 0) {
$nid = $_REQUEST["nid"];
list($news) = select_news_by_id($nid);
if (isset($_REQUEST["text"])) {
$text = preg_replace("/([^\p{L}\p{P}\p{Z}\p{N}\n]{1,})/ui", '', strip_tags($_REQUEST['text']));
insert_news($nid, $text, $user["UID"]);
engelsystem_log("Created news_comment: " . $text);
$html .= success(_("Entry saved."), true);
}
 
$html .= display_news($news);
 
$comments = select_newscomments_by_id($nid);
foreach ($comments as $comment) {
$user_source = User($comment['UID']);
if ($user_source === false)
engelsystem_error(_("Unable to load user."));
 
$html .= '<div class="panel panel-default">';
$html .= '<div class="panel-body">' . nl2br($comment['Text']) . '</div>';
$html .= '<div class="panel-footer text-muted">';
$html .= '<span class="glyphicon glyphicon-time"></span> ' . $comment['Datum'] . '&emsp;';
$html .= User_Nick_render($user_source);
$html .= '</div>';
$html .= '</div>';
}
 
$html .= '<hr /><h2>' . _("New Comment:") . '</h2>';
$html .= form(array(
form_textarea('text', _("Message"), ''),
form_submit('submit', _("Save"))
), page_link_to('news_comments') . '&nid=' . $news['ID']);
 
} else {
$html .= _("Invalid request.");
}
 
return $html . '</div>';
}
 
Function `user_news` has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.
Method `user_news` has 40 lines of code (exceeds 25 allowed). Consider refactoring.
function user_news() {
global $DISPLAY_NEWS, $privileges, $user;
 
$html = '<div class="col-md-12"><h1>' . news_title() . '</h1>' . msg();
 
if (isset($_POST["text"]) && isset($_POST["betreff"]) && in_array("admin_news", $privileges)) {
if (! isset($_POST["treffen"]) || ! in_array("admin_news", $privileges))
$_POST["treffen"] = 0;
insert_news_val($_POST["betreff"], $_POST["text"], $user['UID'], $_POST["treffen"]);
engelsystem_log("Created news: " . $_POST["betreff"] . ", treffen: " . $_POST["treffen"]);
success(_("Entry saved."));
redirect(page_link_to('news'));
}
 
if (isset($_REQUEST['page']) && preg_match("/^[0-9]{1,}$/", $_REQUEST['page']))
$page = $_REQUEST['page'];
else
$page = 0;
 
$news = select_news_by_date($page, $DISPLAY_NEWS);
foreach ($news as $entry)
$html .= display_news($entry);
 
$dis_rows = ceil(select_news() / $DISPLAY_NEWS);
$html .= '<div class="text-center">' . '<ul class="pagination">';
Similar blocks of code found in 2 locations. Consider refactoring.
for ($i = 0; $i < $dis_rows; $i ++) {
if (isset($_REQUEST['page']) && $i == $_REQUEST['page'])
$html .= '<li class="active">';
elseif (! isset($_REQUEST['page']) && $i == 0)
$html .= '<li class="active">';
else
$html .= '<li>';
$html .= '<a href="' . page_link_to("news") . '&page=' . $i . '">' . ($i + 1) . '</a></li>';
}
$html .= '</ul></div>';
 
if (in_array("admin_news", $privileges)) {
$html .= '<hr />';
$html .= '<h2>' . _("Create news:") . '</h2>';
 
$html .= form(array(
form_text('betreff', _("Subject"), ''),
form_textarea('text', _("Message"), ''),
form_checkbox('treffen', _("Meeting"), false, 1),
form_submit('submit', _("Save"))
));
}
return $html . '</div>';
}
?>