diff options
| -rw-r--r-- | build/build.xml | 4 | ||||
| -rw-r--r-- | phpBB/includes/constants.php | 2 | ||||
| -rw-r--r-- | phpBB/includes/functions_content.php | 4 | ||||
| -rw-r--r-- | phpBB/includes/ucp/ucp_profile.php | 3 | ||||
| -rw-r--r-- | phpBB/install/database_update.php | 2 | ||||
| -rw-r--r-- | phpBB/install/install_update.php | 19 | ||||
| -rw-r--r-- | phpBB/install/schemas/schema_data.sql | 2 | ||||
| -rw-r--r-- | phpBB/styles/prosilver/template/posting_smilies.html | 4 | ||||
| -rw-r--r-- | phpBB/styles/subsilver2/template/posting_smilies.html | 4 | ||||
| -rw-r--r-- | phpBB/styles/subsilver2/template/ucp_profile_signature.html | 43 | ||||
| -rw-r--r-- | tests/test_framework/phpbb_functional_test_case.php | 1 | ||||
| -rw-r--r-- | tests/text_processing/generate_text_for_display_test.php | 38 | ||||
| -rw-r--r-- | travis/phpunit-mysql-travis.xml | 4 | ||||
| -rw-r--r-- | travis/phpunit-postgres-travis.xml | 1 | 
14 files changed, 103 insertions, 28 deletions
diff --git a/build/build.xml b/build/build.xml index db327ccc30..2285f32073 100644 --- a/build/build.xml +++ b/build/build.xml @@ -170,8 +170,8 @@  			checkreturn="true" />  		<!-- Install composer dependencies, if there are any. --> -		<exec dir="." -			command='php -r "\$j = json_decode(file_get_contents("composer.json")); echo isset(\$j->require);"' +		<exec dir="${dir}" +			command='php -r "\$j = json_decode(file_get_contents(\"composer.json\")); echo isset(\$j->require);"'  			checkreturn="true"  			outputProperty='composer-has-dependencies' />  		<if> diff --git a/phpBB/includes/constants.php b/phpBB/includes/constants.php index 09e1e50b8d..8d09fe4d9b 100644 --- a/phpBB/includes/constants.php +++ b/phpBB/includes/constants.php @@ -25,7 +25,7 @@ if (!defined('IN_PHPBB'))  */  // phpBB Version -define('PHPBB_VERSION', '3.0.12-RC2'); +define('PHPBB_VERSION', '3.0.13-dev');  // QA-related  // define('PHPBB_QA', 1); diff --git a/phpBB/includes/functions_content.php b/phpBB/includes/functions_content.php index b7650ecd6a..6213d2fd24 100644 --- a/phpBB/includes/functions_content.php +++ b/phpBB/includes/functions_content.php @@ -413,7 +413,7 @@ function generate_text_for_display($text, $uid, $bitfield, $flags)  {  	static $bbcode; -	if (!$text) +	if ($text === '')  	{  		return '';  	} @@ -459,7 +459,7 @@ function generate_text_for_storage(&$text, &$uid, &$bitfield, &$flags, $allow_bb  	$uid = $bitfield = '';  	$flags = (($allow_bbcode) ? OPTION_FLAG_BBCODE : 0) + (($allow_smilies) ? OPTION_FLAG_SMILIES : 0) + (($allow_urls) ? OPTION_FLAG_LINKS : 0); -	if (!$text) +	if ($text === '')  	{  		return;  	} diff --git a/phpBB/includes/ucp/ucp_profile.php b/phpBB/includes/ucp/ucp_profile.php index d35d13b6c1..847311058b 100644 --- a/phpBB/includes/ucp/ucp_profile.php +++ b/phpBB/includes/ucp/ucp_profile.php @@ -548,6 +548,9 @@ class ucp_profile  				// Build custom bbcodes array  				display_custom_bbcodes(); +				// Generate smiley listing +				generate_smilies('inline', 0); +  			break;  			case 'avatar': diff --git a/phpBB/install/database_update.php b/phpBB/install/database_update.php index 59f795e37e..6aca38d297 100644 --- a/phpBB/install/database_update.php +++ b/phpBB/install/database_update.php @@ -8,7 +8,7 @@  *  */ -define('UPDATES_TO_VERSION', '3.0.12-RC2'); +define('UPDATES_TO_VERSION', '3.0.13-dev');  // Enter any version to update from to test updates. The version within the db will not be updated.  define('DEBUG_FROM_VERSION', false); diff --git a/phpBB/install/install_update.php b/phpBB/install/install_update.php index c18a0fb4ec..2f3ee1c55a 100644 --- a/phpBB/install/install_update.php +++ b/phpBB/install/install_update.php @@ -58,7 +58,6 @@ class install_update extends module  	var $new_location;  	var $latest_version;  	var $current_version; -	var $unequal_version;  	var $update_to_version; @@ -76,7 +75,6 @@ class install_update extends module  		$this->tpl_name = 'install_update';  		$this->page_title = 'UPDATE_INSTALLATION'; -		$this->unequal_version = false;  		$this->old_location = $phpbb_root_path . 'install/update/old/';  		$this->new_location = $phpbb_root_path . 'install/update/new/'; @@ -195,8 +193,6 @@ class install_update extends module  		// Check if the update files are actually meant to update from the current version  		if ($this->current_version != $this->update_info['version']['from'])  		{ -			$this->unequal_version = true; -  			$template->assign_vars(array(  				'S_ERROR'	=> true,  				'ERROR_MSG'	=> sprintf($user->lang['INCOMPATIBLE_UPDATE_FILES'], $this->current_version, $this->update_info['version']['from'], $this->update_info['version']['to']), @@ -204,10 +200,8 @@ class install_update extends module  		}  		// Check if the update files stored are for the latest version... -		if ($this->latest_version != $this->update_info['version']['to']) +		if (version_compare(strtolower($this->latest_version), strtolower($this->update_info['version']['to']), '>'))  		{ -			$this->unequal_version = true; -  			$template->assign_vars(array(  				'S_WARNING'		=> true,  				'WARNING_MSG'	=> sprintf($user->lang['OLD_UPDATE_FILES'], $this->update_info['version']['from'], $this->update_info['version']['to'], $this->latest_version)) @@ -232,7 +226,14 @@ class install_update extends module  			}  			// What about the language file? Got it updated? -			if (in_array('language/en/install.' . $phpEx, $this->update_info['files'])) +			if (in_array('language/' . $language . '/install.' . $phpEx, $this->update_info['files'])) +			{ +				$lang = array(); +				include($this->new_location . 'language/' . $language . '/install.' . $phpEx); +				// this is the user's language.. just merge it +				$user->lang = array_merge($user->lang, $lang); +			} +			if ($language != 'en' && in_array('language/en/install.' . $phpEx, $this->update_info['files']))  			{  				$lang = array();  				include($this->new_location . 'language/en/install.' . $phpEx); @@ -287,7 +288,7 @@ class install_update extends module  				);  				// Print out version the update package updates to -				if ($this->unequal_version) +				if ($this->latest_version != $this->update_info['version']['to'])  				{  					$template->assign_var('PACKAGE_VERSION', $this->update_info['version']['to']);  				} diff --git a/phpBB/install/schemas/schema_data.sql b/phpBB/install/schemas/schema_data.sql index cfe3c0c4cb..e1b36e64a2 100644 --- a/phpBB/install/schemas/schema_data.sql +++ b/phpBB/install/schemas/schema_data.sql @@ -246,7 +246,7 @@ INSERT INTO phpbb_config (config_name, config_value) VALUES ('topics_per_page',  INSERT INTO phpbb_config (config_name, config_value) VALUES ('tpl_allow_php', '0');  INSERT INTO phpbb_config (config_name, config_value) VALUES ('upload_icons_path', 'images/upload_icons');  INSERT INTO phpbb_config (config_name, config_value) VALUES ('upload_path', 'files'); -INSERT INTO phpbb_config (config_name, config_value) VALUES ('version', '3.0.12-RC2'); +INSERT INTO phpbb_config (config_name, config_value) VALUES ('version', '3.0.13-dev');  INSERT INTO phpbb_config (config_name, config_value) VALUES ('warnings_expire_days', '90');  INSERT INTO phpbb_config (config_name, config_value) VALUES ('warnings_gc', '14400'); diff --git a/phpBB/styles/prosilver/template/posting_smilies.html b/phpBB/styles/prosilver/template/posting_smilies.html index 86ac24aa53..d45b185ed3 100644 --- a/phpBB/styles/prosilver/template/posting_smilies.html +++ b/phpBB/styles/prosilver/template/posting_smilies.html @@ -2,8 +2,8 @@  <script type="text/javascript">  // <![CDATA[ -	var form_name = 'postform'; -	var text_name = 'message'; +	var form_name = opener.form_name; +	var text_name = opener.text_name;  // ]]>  </script>  <script type="text/javascript" src="{T_SUPER_TEMPLATE_PATH}/editor.js"></script> diff --git a/phpBB/styles/subsilver2/template/posting_smilies.html b/phpBB/styles/subsilver2/template/posting_smilies.html index fcab578bd9..d2224887bc 100644 --- a/phpBB/styles/subsilver2/template/posting_smilies.html +++ b/phpBB/styles/subsilver2/template/posting_smilies.html @@ -2,8 +2,8 @@  <script type="text/javascript">  // <![CDATA[ -	var form_name = 'postform'; -	var text_name = 'message'; +	var form_name = opener.form_name; +	var text_name = opener.text_name;  // ]]>  </script>  <script type="text/javascript" src="{T_SUPER_TEMPLATE_PATH}/editor.js"></script> diff --git a/phpBB/styles/subsilver2/template/ucp_profile_signature.html b/phpBB/styles/subsilver2/template/ucp_profile_signature.html index a33726e166..54e8aaa723 100644 --- a/phpBB/styles/subsilver2/template/ucp_profile_signature.html +++ b/phpBB/styles/subsilver2/template/ucp_profile_signature.html @@ -1,13 +1,22 @@  <!-- INCLUDE ucp_header.html --> +<script type="text/javascript"> +// <![CDATA[ +	var form_name = 'ucp'; +	var text_name = 'signature'; +// ]]> +</script> +  <!-- DEFINE $S_SIGNATURE = 1 -->  <table class="tablebg" width="100%" cellspacing="1">  <tr>  	<th colspan="2">{L_TITLE}</th>  </tr> +<!-- IF not S_SMILIES_ALLOWED -->  <tr>  	<td colspan="2" class="row1">{L_SIGNATURE_EXPLAIN}</td>  </tr> +<!-- ENDIF -->  <!-- IF ERROR -->  	<tr> @@ -16,14 +25,32 @@  <!-- ENDIF -->  <tr> -	<td colspan="2" class="row2"> -		<script type="text/javascript"> -		// <![CDATA[ -			var form_name = 'ucp'; -			var text_name = 'signature'; -		// ]]> -		</script> -		 +	<!-- IF S_SMILIES_ALLOWED --> +	<td class="row1" width="22%" valign="top"> +		{L_SIGNATURE_EXPLAIN} +		<table width="100%" cellspacing="5" cellpadding="0" border="0" align="center"> +			<tr> +				<td class="gensmall" align="center"><b>{L_SMILIES}</b></td> +			</tr> +			<tr> +				<td align="center"> +				<!-- BEGIN smiley --> +					<a href="#" onclick="insert_text('{smiley.A_SMILEY_CODE}', true); return false;" style="line-height: 20px;"><img src="{smiley.SMILEY_IMG}" width="{smiley.SMILEY_WIDTH}" height="{smiley.SMILEY_HEIGHT}" alt="{smiley.SMILEY_CODE}" title="{smiley.SMILEY_DESC}" hspace="2" vspace="2" /></a> +				<!-- END smiley --> +				</td> +			</tr> +			<!-- IF S_SHOW_SMILEY_LINK --> +			<tr> +				<td align="center"><a class="nav" href="{U_MORE_SMILIES}" onclick="popup(this.href, 300, 350, '_phpbbsmilies'); return false;">{L_MORE_SMILIES}</a></td> +			</tr> +			<!-- ENDIF --> +		</table> +	</td> +	<td class="row2"> +	<!-- ELSE --> +	<td class="row2" colspan="2"> +	<!-- ENDIF --> +  		<table cellspacing="0" cellpadding="2" border="0" width="99%">  		<!-- INCLUDE posting_buttons.html -->  		<tr> diff --git a/tests/test_framework/phpbb_functional_test_case.php b/tests/test_framework/phpbb_functional_test_case.php index 7d8b4a3144..1ced462c76 100644 --- a/tests/test_framework/phpbb_functional_test_case.php +++ b/tests/test_framework/phpbb_functional_test_case.php @@ -577,6 +577,7 @@ class phpbb_functional_test_case extends phpbb_test_case  		// Any output before the doc type means there was an error  		$content = self::$client->getResponse()->getContent(); +		self::assertNotContains('[phpBB Debug]', $content);  		self::assertStringStartsWith('<!DOCTYPE', trim($content), 'Output found before DOCTYPE specification.');  	} diff --git a/tests/text_processing/generate_text_for_display_test.php b/tests/text_processing/generate_text_for_display_test.php new file mode 100644 index 0000000000..a157fe7d9a --- /dev/null +++ b/tests/text_processing/generate_text_for_display_test.php @@ -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)); +	} +} diff --git a/travis/phpunit-mysql-travis.xml b/travis/phpunit-mysql-travis.xml index 179a2a358e..e99f8d6cee 100644 --- a/travis/phpunit-mysql-travis.xml +++ b/travis/phpunit-mysql-travis.xml @@ -15,6 +15,10 @@  		<testsuite name="phpBB Test Suite">  			<directory suffix="_test.php">../tests/</directory>  			<exclude>tests/functional</exclude> +			<exclude>tests/lint_test.php</exclude> +		</testsuite> +		<testsuite name="phpBB Lint Test"> +			<file>tests/lint_test.php</file>  		</testsuite>  		<testsuite name="phpBB Functional Tests">  			<directory suffix="_test.php" phpVersion="5.3.19" phpVersionOperator=">=">../tests/functional</directory> diff --git a/travis/phpunit-postgres-travis.xml b/travis/phpunit-postgres-travis.xml index 12e20547ba..aa829f4d30 100644 --- a/travis/phpunit-postgres-travis.xml +++ b/travis/phpunit-postgres-travis.xml @@ -15,6 +15,7 @@  		<testsuite name="phpBB Test Suite">  			<directory suffix="_test.php">../tests/</directory>  			<exclude>tests/functional</exclude> +			<exclude>tests/lint_test.php</exclude>  		</testsuite>  		<testsuite name="phpBB Functional Tests">  			<directory suffix="_test.php" phpVersion="5.3.19" phpVersionOperator=">=">../tests/functional</directory>  | 
