mirror of
https://github.com/phpbb/phpbb.git
synced 2025-07-17 15:11:40 +02:00
[ticket/11620] Minor indentation changes and comment clarity
PHPBB3-11620
This commit is contained in:
@@ -23,11 +23,11 @@ class phpbb_session_check_ban_test extends phpbb_session_test_case
|
|||||||
{
|
{
|
||||||
return array(
|
return array(
|
||||||
array('All false values, should not be banned',
|
array('All false values, should not be banned',
|
||||||
false, false, false, false, /* ?: */ false),
|
false, false, false, false, /* should be banned? -> */ false),
|
||||||
array('Matching values in the database, should be banned',
|
array('Matching values in the database, should be banned',
|
||||||
4, '127.0.0.1', 'bar@example.org', true, /* ?: */ true),
|
4, '127.0.0.1', 'bar@example.org', true, /* should be banned? -> */ true),
|
||||||
array('IP Banned, should be banned',
|
array('IP Banned, should be banned',
|
||||||
false, '127.1.1.1', false, false, /* ?: */ true),
|
false, '127.1.1.1', false, false, /* should be banned? -> */ true),
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -38,7 +38,7 @@ class phpbb_session_check_ban_test extends phpbb_session_test_case
|
|||||||
// Change the global cache object for this test because
|
// Change the global cache object for this test because
|
||||||
// the mock cache object does not hit the database as is
|
// the mock cache object does not hit the database as is
|
||||||
// needed for this test.
|
// needed for this test.
|
||||||
global $cache, $config, $phpbb_root_path, $php_ext;
|
global $cache, $config, $phpbb_root_path, $php_ext;
|
||||||
$cache = new phpbb_cache_service(
|
$cache = new phpbb_cache_service(
|
||||||
new phpbb_cache_driver_file(),
|
new phpbb_cache_driver_file(),
|
||||||
$config,
|
$config,
|
||||||
@@ -49,8 +49,7 @@ class phpbb_session_check_ban_test extends phpbb_session_test_case
|
|||||||
|
|
||||||
try
|
try
|
||||||
{
|
{
|
||||||
$is_banned =
|
$is_banned = $session->check_ban($user_id, $user_ips, $user_email, $return);
|
||||||
$session->check_ban($user_id, $user_ips, $user_email, $return);
|
|
||||||
} catch (PHPUnit_Framework_Error_Notice $e)
|
} catch (PHPUnit_Framework_Error_Notice $e)
|
||||||
{
|
{
|
||||||
// User error was triggered, user must have been banned
|
// User error was triggered, user must have been banned
|
||||||
|
@@ -9,7 +9,6 @@
|
|||||||
|
|
||||||
require_once dirname(__FILE__) . '/../test_framework/phpbb_session_test_case.php';
|
require_once dirname(__FILE__) . '/../test_framework/phpbb_session_test_case.php';
|
||||||
|
|
||||||
|
|
||||||
class phpbb_session_check_isvalid_test extends phpbb_session_test_case
|
class phpbb_session_check_isvalid_test extends phpbb_session_test_case
|
||||||
{
|
{
|
||||||
public function getDataSet()
|
public function getDataSet()
|
||||||
|
Reference in New Issue
Block a user