diff options
| -rw-r--r-- | build/build.xml | 2 | ||||
| -rw-r--r-- | phpBB/docs/CHANGELOG.html | 57 | ||||
| -rw-r--r-- | phpBB/includes/constants.php | 2 | ||||
| -rw-r--r-- | phpBB/install/convertors/convert_phpbb20.php | 2 | ||||
| -rwxr-xr-x | phpBB/install/phpbbcli.php | 2 | ||||
| -rw-r--r-- | phpBB/install/schemas/schema_data.sql | 2 | ||||
| -rw-r--r-- | phpBB/phpbb/db/migration/data/v32x/v323rc1.php | 36 | ||||
| -rw-r--r-- | phpBB/phpbb/db/migration/tool/module.php | 9 | ||||
| -rw-r--r-- | phpBB/styles/prosilver/style.cfg | 4 | ||||
| -rw-r--r-- | tests/dbal/migrator_tool_module_test.php | 161 | 
10 files changed, 264 insertions, 13 deletions
diff --git a/build/build.xml b/build/build.xml index 1491b95b48..f85bebe70e 100644 --- a/build/build.xml +++ b/build/build.xml @@ -2,7 +2,7 @@  <project name="phpBB" description="The phpBB forum software" default="all" basedir="../">  	<!-- a few settings for the build --> -	<property name="newversion" value="3.2.3-dev" /> +	<property name="newversion" value="3.2.3-RC1" />  	<property name="prevversion" value="3.2.2" />  	<property name="olderversions" value="3.0.14, 3.1.0, 3.1.1, 3.1.2, 3.1.3, 3.1.4, 3.1.5, 3.1.6, 3.1.7, 3.1.7-pl1, 3.1.8, 3.1.9, 3.1.10, 3.1.11, 3.1.12, 3.2.0-a1, 3.2.0-a2, 3.2.0-b1, 3.2.0-b2, 3.2.0-RC1, 3.2.0-RC2, 3.2.0, 3.2.1" />  	<!-- no configuration should be needed beyond this point --> diff --git a/phpBB/docs/CHANGELOG.html b/phpBB/docs/CHANGELOG.html index 13272e154c..eee4f3efcc 100644 --- a/phpBB/docs/CHANGELOG.html +++ b/phpBB/docs/CHANGELOG.html @@ -50,6 +50,7 @@  <ol>  	<li><a href="#changelog">Changelog</a>  	<ul> +		<li><a href="#v322">Changes since 3.2.2</a></li>  		<li><a href="#v321">Changes since 3.2.1</a></li>  		<li><a href="#v320">Changes since 3.2.0</a></li>  		<li><a href="#v320rc1">Changes since 3.2.0-RC1</a></li> @@ -128,6 +129,62 @@  		<div class="inner">  		<div class="content"> +			<a name="v322"></a><h3>Changes since 3.2.2</h3> +			<h4>Bug</h4> +			<ul> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-14936">PHPBB3-14936</a>] - Missing language variable INST_ERR_DB</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15491">PHPBB3-15491</a>] - Outdated linkes in installer support page</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15492">PHPBB3-15492</a>] - Permissions role combobox does not work in RTL</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15500">PHPBB3-15500</a>] - Docs outdated for new PHP 5.4.7 requirement</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15502">PHPBB3-15502</a>] - Errors in migrations in 3.2.2 release</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15506">PHPBB3-15506</a>] - Previewing new post empties attachment list of all but first attachment</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15512">PHPBB3-15512</a>] - Avoid reparsing non-existent polls</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15513">PHPBB3-15513</a>] - Signature edit in acp gives error</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15520">PHPBB3-15520</a>] - DbDriver->sql_build_query cant cope with sub-selects</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15522">PHPBB3-15522</a>] - Allow multiple color palettes per page</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15523">PHPBB3-15523</a>] - AdBlocker may cause JS error when using CookieConsent</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15525">PHPBB3-15525</a>] - composer.json License is Invalid/Deprecated</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15526">PHPBB3-15526</a>] - Cast bbcode ID to integer</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15527">PHPBB3-15527</a>] - Cannot interpret the BBCode definition</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15532">PHPBB3-15532</a>] - Update pull request template</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15533">PHPBB3-15533</a>] - Typo in viewtopic_topic_tools.html</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15558">PHPBB3-15558</a>] - phpbb\report\report_handler_post.php</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15559">PHPBB3-15559</a>] - phpbb\report\report_handler_pm.php:56</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15595">PHPBB3-15595</a>] - Migration Module Exists Tool Broken</li> +			</ul> +			<h4>Improvement</h4> +			<ul> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-12579">PHPBB3-12579</a>] - Add BUTTON_ language strings for post & PM buttons</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15495">PHPBB3-15495</a>] - Use transactions for queries in move_forum</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15499">PHPBB3-15499</a>] - Drop HHVM support</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15510">PHPBB3-15510</a>] - Link Orphan attachments in ACP>General to Orphaned attachments page </li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15514">PHPBB3-15514</a>] - Improve accessibility by adding vital info from explanation to a title</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15518">PHPBB3-15518</a>] - Do not attempt to accurately determine whether posters can read private messages in viewtopic</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15528">PHPBB3-15528</a>] - Display the version of the installed styles in acp</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15529">PHPBB3-15529</a>] - Color groups in ACP</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15531">PHPBB3-15531</a>] - Log malformed BBCodes</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15534">PHPBB3-15534</a>] - Outdated ACP extensions database link for phpBB 3.2</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15535">PHPBB3-15535</a>] - Add S_FIRST_POST to postrow on viewtopic</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15537">PHPBB3-15537</a>] - Add events core.search_(native|mysql|postgres|sphinx)_index_before</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15547">PHPBB3-15547</a>] - Add file object to event core.avatar_driver_upload_move_file_before</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15561">PHPBB3-15561</a>] - Add core events for adding columns to MySQL and Postgres search backends</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15568">PHPBB3-15568</a>] - Update depencies to latest versions</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15569">PHPBB3-15569</a>] - Adjust update instructions to suggest file replacement method</li> +			</ul> +			<h4>New Feature</h4> +			<ul> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15398">PHPBB3-15398</a>] - Add event to oauth login after ID check</li> +			</ul> +			<h4>Security Issue</h4> +			<ul> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15570">PHPBB3-15570</a>] - Extension version check is restricted to TLS 1.0</li> +			</ul> +			<h4>Task</h4> +			<ul> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15466">PHPBB3-15466</a>] - Move Nils in CREDITS.txt</li> +				<li>[<a href="http://tracker.phpbb.com/browse/PHPBB3-15516">PHPBB3-15516</a>] - Add instructions on running UI tests</li> +			</ul> +  			<a name="v321"></a><h3>Changes since 3.2.1</h3>  			<h4>Security Issue</h4>  			<ul> diff --git a/phpBB/includes/constants.php b/phpBB/includes/constants.php index 7eeb36595d..d94d418450 100644 --- a/phpBB/includes/constants.php +++ b/phpBB/includes/constants.php @@ -28,7 +28,7 @@ if (!defined('IN_PHPBB'))  */  // phpBB Version -@define('PHPBB_VERSION', '3.2.3-dev'); +@define('PHPBB_VERSION', '3.2.3-RC1');  // QA-related  // define('PHPBB_QA', 1); diff --git a/phpBB/install/convertors/convert_phpbb20.php b/phpBB/install/convertors/convert_phpbb20.php index a453e5f7ff..7dfb73be7e 100644 --- a/phpBB/install/convertors/convert_phpbb20.php +++ b/phpBB/install/convertors/convert_phpbb20.php @@ -38,7 +38,7 @@ $dbms = $phpbb_config_php_file->convert_30_dbms_to_31($dbms);  $convertor_data = array(  	'forum_name'	=> 'phpBB 2.0.x',  	'version'		=> '1.0.3', -	'phpbb_version'	=> '3.2.2', +	'phpbb_version'	=> '3.2.3',  	'author'		=> '<a href="https://www.phpbb.com/">phpBB Limited</a>',  	'dbms'			=> $dbms,  	'dbhost'		=> $dbhost, diff --git a/phpBB/install/phpbbcli.php b/phpBB/install/phpbbcli.php index 17b8444aa5..611e4ec53b 100755 --- a/phpBB/install/phpbbcli.php +++ b/phpBB/install/phpbbcli.php @@ -23,7 +23,7 @@ if (php_sapi_name() !== 'cli')  define('IN_PHPBB', true);  define('IN_INSTALL', true);  define('PHPBB_ENVIRONMENT', 'production'); -define('PHPBB_VERSION', '3.2.2-RC1'); +define('PHPBB_VERSION', '3.2.3-RC1');  $phpbb_root_path = __DIR__ . '/../';  $phpEx = substr(strrchr(__FILE__, '.'), 1); diff --git a/phpBB/install/schemas/schema_data.sql b/phpBB/install/schemas/schema_data.sql index 4dfe5e94aa..e7eedfb46c 100644 --- a/phpBB/install/schemas/schema_data.sql +++ b/phpBB/install/schemas/schema_data.sql @@ -280,7 +280,7 @@ 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 ('use_system_cron', '0'); -INSERT INTO phpbb_config (config_name, config_value) VALUES ('version', '3.2.3-dev'); +INSERT INTO phpbb_config (config_name, config_value) VALUES ('version', '3.2.3-RC1');  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/phpbb/db/migration/data/v32x/v323rc1.php b/phpBB/phpbb/db/migration/data/v32x/v323rc1.php new file mode 100644 index 0000000000..0ff20d5074 --- /dev/null +++ b/phpBB/phpbb/db/migration/data/v32x/v323rc1.php @@ -0,0 +1,36 @@ +<?php +/** +* +* This file is part of the phpBB Forum Software package. +* +* @copyright (c) phpBB Limited <https://www.phpbb.com> +* @license GNU General Public License, version 2 (GPL-2.0) +* +* For full copyright and license information, please see +* the docs/CREDITS.txt file. +* +*/ + +namespace phpbb\db\migration\data\v32x; + +class v323rc1 extends \phpbb\db\migration\migration +{ +	public function effectively_installed() +	{ +		return phpbb_version_compare($this->config['version'], '3.2.3-RC1', '>='); +	} + +	static public function depends_on() +	{ +		return array( +			'\phpbb\db\migration\data\v32x\enable_accurate_pm_button', +		); +	} + +	public function update_data() +	{ +		return array( +			array('config.update', array('version', '3.2.3-RC1')), +		); +	} +} diff --git a/phpBB/phpbb/db/migration/tool/module.php b/phpBB/phpbb/db/migration/tool/module.php index 7d2720c861..e24c78e228 100644 --- a/phpBB/phpbb/db/migration/tool/module.php +++ b/phpBB/phpbb/db/migration/tool/module.php @@ -86,7 +86,8 @@ class module implements \phpbb\db\migration\tool\tool_interface  	* 		check for to see if it exists  	* @param bool $lazy Checks lazily if the module exists. Returns true if it exists in at  	*       least one given parent. -	* @return bool true if module exists in *all* given parents, false if not +	* @return bool true if module exists in *all* given parents, false if not in any given parent; +	 *      true if ignoring parent check and module exists class wide, false if not found at all.  	*/  	public function exists($class, $parent, $module, $lazy = false)  	{ @@ -110,6 +111,10 @@ class module implements \phpbb\db\migration\tool\tool_interface  				$parent_sqls[] = 'AND parent_id = ' . (int) $parent_id;  			}  		} +		else +		{ +			$parent_sqls[] = ''; +		}  		foreach ($parent_sqls as $parent_sql)  		{ @@ -126,7 +131,7 @@ class module implements \phpbb\db\migration\tool\tool_interface  			{  				return false;  			} -			else if ($lazy && $module_id) +			if ($lazy && $module_id)  			{  				return true;  			} diff --git a/phpBB/styles/prosilver/style.cfg b/phpBB/styles/prosilver/style.cfg index 00d07fee6c..ac2359dd99 100644 --- a/phpBB/styles/prosilver/style.cfg +++ b/phpBB/styles/prosilver/style.cfg @@ -21,8 +21,8 @@  # General Information about this style  name = prosilver  copyright = © phpBB Limited, 2007 -style_version = 3.2.2 -phpbb_version = 3.2.2 +style_version = 3.2.3 +phpbb_version = 3.2.3  # Defining a different template bitfield  # template_bitfield = //g= diff --git a/tests/dbal/migrator_tool_module_test.php b/tests/dbal/migrator_tool_module_test.php index c625b93ded..0f3febb24d 100644 --- a/tests/dbal/migrator_tool_module_test.php +++ b/tests/dbal/migrator_tool_module_test.php @@ -52,11 +52,39 @@ class phpbb_dbal_migrator_tool_module_test extends phpbb_database_test_case  			array(  				'',  				'ACP_CAT', +				false,  				true,  			),  			array(  				0,  				'ACP_CAT', +				false, +				true, +			), +			array( +				false, +				'ACP_CAT', +				false, +				true, +			), + +			// Test the existing category lazily +			array( +				'', +				'ACP_CAT', +				true, +				true, +			), +			array( +				0, +				'ACP_CAT', +				true, +				true, +			), +			array( +				false, +				'ACP_CAT', +				true,  				true,  			), @@ -65,16 +93,39 @@ class phpbb_dbal_migrator_tool_module_test extends phpbb_database_test_case  				'',  				'ACP_MODULE',  				false, +				false, +			), +			array( +				false, +				'ACP_MODULE', +				false, +				true, +			), +			array( +				'ACP_CAT', +				'ACP_MODULE', +				false, +				true, +			), + +			// Test the existing module lazily +			array( +				'', +				'ACP_MODULE', +				true, +				false,  			),  			array(  				false,  				'ACP_MODULE',  				true, +				true,  			),  			array(  				'ACP_CAT',  				'ACP_MODULE',  				true, +				true,  			),  			// Test for non-existant modules @@ -82,11 +133,39 @@ class phpbb_dbal_migrator_tool_module_test extends phpbb_database_test_case  				'',  				'ACP_NON_EXISTANT_CAT',  				false, +				false, +			), +			array( +				false, +				'ACP_NON_EXISTANT_CAT', +				false, +				false,  			),  			array(  				'ACP_CAT',  				'ACP_NON_EXISTANT_MODULE',  				false, +				false, +			), + +			// Test for non-existant modules lazily +			array( +				'', +				'ACP_NON_EXISTANT_CAT', +				true, +				false, +			), +			array( +				false, +				'ACP_NON_EXISTANT_CAT', +				true, +				false, +			), +			array( +				'ACP_CAT', +				'ACP_NON_EXISTANT_MODULE', +				true, +				false,  			),  		);  	} @@ -94,9 +173,9 @@ class phpbb_dbal_migrator_tool_module_test extends phpbb_database_test_case  	/**  	* @dataProvider exists_data_acp  	*/ -	public function test_exists_acp($parent, $module, $expected) +	public function test_exists_acp($parent, $module, $lazy, $expected)  	{ -		$this->assertEquals($expected, $this->tool->exists('acp', $parent, $module)); +		$this->assertEquals($expected, $this->tool->exists('acp', $parent, $module, $lazy));  	}  	public function exists_data_ucp() @@ -106,11 +185,39 @@ class phpbb_dbal_migrator_tool_module_test extends phpbb_database_test_case  			array(  				'',  				'UCP_MAIN_CAT', +				false,  				true,  			),  			array(  				0,  				'UCP_MAIN_CAT', +				false, +				true, +			), +			array( +				false, +				'UCP_MAIN_CAT', +				false, +				true, +			), + +			// Test the existing category lazily +			array( +				'', +				'UCP_MAIN_CAT', +				true, +				true, +			), +			array( +				0, +				'UCP_MAIN_CAT', +				true, +				true, +			), +			array( +				false, +				'UCP_MAIN_CAT', +				true,  				true,  			), @@ -119,20 +226,50 @@ class phpbb_dbal_migrator_tool_module_test extends phpbb_database_test_case  				'',  				'UCP_SUBCATEGORY',  				false, +				false,  			),  			array(  				false,  				'UCP_SUBCATEGORY', +				false,  				true,  			),  			array(  				'UCP_MAIN_CAT',  				'UCP_SUBCATEGORY', +				false,  				true,  			),  			array(  				'UCP_SUBCATEGORY',  				'UCP_MODULE', +				false, +				true, +			), + +			// Test the existing module lazily +			array( +				'', +				'UCP_SUBCATEGORY', +				true, +				false, +			), +			array( +				false, +				'UCP_SUBCATEGORY', +				true, +				true, +			), +			array( +				'UCP_MAIN_CAT', +				'UCP_SUBCATEGORY', +				true, +				true, +			), +			array( +				'UCP_SUBCATEGORY', +				'UCP_MODULE', +				true,  				true,  			), @@ -141,10 +278,26 @@ class phpbb_dbal_migrator_tool_module_test extends phpbb_database_test_case  				'',  				'UCP_NON_EXISTANT_CAT',  				false, +				false, +			), +			array( +				'UCP_MAIN_CAT', +				'UCP_NON_EXISTANT_MODULE', +				false, +				false, +			), + +			// Test for non-existant modules lazily +			array( +				'', +				'UCP_NON_EXISTANT_CAT', +				true, +				false,  			),  			array(  				'UCP_MAIN_CAT',  				'UCP_NON_EXISTANT_MODULE', +				true,  				false,  			),  		); @@ -153,9 +306,9 @@ class phpbb_dbal_migrator_tool_module_test extends phpbb_database_test_case  	/**  	* @dataProvider exists_data_ucp  	*/ -	public function test_exists_ucp($parent, $module, $expected) +	public function test_exists_ucp($parent, $module, $lazy, $expected)  	{ -		$this->assertEquals($expected, $this->tool->exists('ucp', $parent, $module)); +		$this->assertEquals($expected, $this->tool->exists('ucp', $parent, $module, $lazy));  	}  	public function test_add()  | 
