[fix/code_sniffer] Fix code_sniffer complaints

This commit is contained in:
Marc Alexander
2013-11-03 15:31:38 +01:00
parent 404fad65b5
commit 2d87dc449c
7 changed files with 18 additions and 18 deletions

View File

@@ -9,7 +9,7 @@
if (!defined('IN_PHPBB') && !defined('UMIL_AUTO') && !defined('IN_INSTALL'))
{
exit;
exit;
}
// Get portal config
@@ -175,10 +175,10 @@ function phpbb_fetch_posts($module_id, $forum_from, $permissions, $number_of_pos
$topic_order = ($config['board3_news_show_last_' . $module_id]) ? 't.topic_last_post_time DESC' : 't.topic_time DESC' ;
break;
default:
$topic_type = $str_where = $user_link = $post_link = '';
$topic_order = 't.topic_time DESC';
// maybe use trigger_error here, as this shouldn't happen
default:
$topic_type = $str_where = $user_link = $post_link = '';
$topic_order = 't.topic_time DESC';
// maybe use trigger_error here, as this shouldn't happen
}
if ($type == 'announcements' && $global_f < 1)
@@ -447,10 +447,10 @@ function generate_portal_pagination($base_url, $num_items, $per_page, $start_ite
$anker = '#n';
break;
default:
// this shouldn't happend @todo: use trigger_error()
$pagination_type = 'ap';
$anker = '#a';
default:
// this shouldn't happend @todo: use trigger_error()
$pagination_type = 'ap';
$anker = '#a';
}
// Make sure $per_page is a valid value
@@ -609,13 +609,13 @@ function get_portal_tracking_info($fetch_news)
}
$db->sql_freeresult($result);
// @todo: do not use $current_forum here as this is already used by the outside foreach
// @todo: do not use $current_forum here as this is already used by the outside foreach
foreach($forum_ids as $current_forum)
{
$user_lastmark[$current_forum] = (isset($mark_time[$current_forum])) ? $mark_time[$current_forum] : $user->data['user_lastmark'];
}
// @todo: also check if $user_lastmark has been defined for this specific forum_id
// @todo: also check if $user_lastmark has been defined for this specific forum_id
foreach ($topic_ids as $topic_id)
{
$last_read[$topic_id] = (!isset($last_read[$topic_id]) || $user_lastmark[$rev_forum_ids[$topic_id]] > $last_read[$topic_id]) ? $user_lastmark[$rev_forum_ids[$topic_id]] : $last_read[$topic_id];

View File

@@ -26,8 +26,8 @@ function column_num_string($column)
return 'top';
case 5:
return 'bottom';
default:
return 0;
default:
return 0;
}
}

View File

@@ -9,7 +9,7 @@
if (!defined('IN_PHPBB'))
{
exit;
exit;
}
/**