Merge branch 'develop-olympus' into develop

* develop-olympus:
  [ticket/11762] Added call to test class's parent::setUp().
  [ticket/11762] Fixed test's filename
  [ticket/11762] Use the === operator to distinguish "0" from ""
This commit is contained in:
Joas Schilling 2013-08-03 15:37:01 +02:00
commit 78623bf2ff
2 changed files with 40 additions and 2 deletions

View file

@ -413,7 +413,7 @@ function generate_text_for_display($text, $uid, $bitfield, $flags, $censor_text
static $bbcode; static $bbcode;
global $phpbb_dispatcher; global $phpbb_dispatcher;
if (!$text) if ($text === '')
{ {
return ''; return '';
} }
@ -505,7 +505,7 @@ function generate_text_for_storage(&$text, &$uid, &$bitfield, &$flags, $allow_bb
$uid = $bitfield = ''; $uid = $bitfield = '';
$flags = (($allow_bbcode) ? OPTION_FLAG_BBCODE : 0) + (($allow_smilies) ? OPTION_FLAG_SMILIES : 0) + (($allow_urls) ? OPTION_FLAG_LINKS : 0); $flags = (($allow_bbcode) ? OPTION_FLAG_BBCODE : 0) + (($allow_smilies) ? OPTION_FLAG_SMILIES : 0) + (($allow_urls) ? OPTION_FLAG_LINKS : 0);
if (!$text) if ($text === '')
{ {
return; return;
} }

View file

@ -0,0 +1,38 @@
<?php
/**
*
* @package testing
* @copyright (c) 2013 phpBB Group
* @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
*
*/
require_once dirname(__FILE__) . '/../../phpBB/includes/functions.php';
require_once dirname(__FILE__) . '/../../phpBB/includes/functions_content.php';
require_once dirname(__FILE__) . '/../mock/user.php';
require_once dirname(__FILE__) . '/../mock/cache.php';
class phpbb_text_processing_generate_text_for_display_test extends phpbb_test_case
{
public function setUp()
{
global $cache, $user;
parent::setUp();
$cache = new phpbb_mock_cache;
$user = new phpbb_mock_user;
$user->optionset('viewcensors', false);
}
public function test_empty_string()
{
$this->assertSame('', generate_text_for_display('', '', '', 0));
}
public function test_zero_string()
{
$this->assertSame('0', generate_text_for_display('0', '', '', 0));
}
}