[Phpbb-checkins] r9927 - in /branches/phpBB-3_0_0/phpBB: docs/CHANGELOG.html includes/functions_posting.php

[Meik S. <acy...@ph...>]

Author: acydburn
Date: Wed Aug  5 14:01:03 2009
New Revision: 9927

Log:
submit_post() now accepts force_approved_state key passed to $data to indicate new posts being approved (true) or unapproved (false).

Modified:
    branches/phpBB-3_0_0/phpBB/docs/CHANGELOG.html
    branches/phpBB-3_0_0/phpBB/includes/functions_posting.php

Modified: branches/phpBB-3_0_0/phpBB/docs/CHANGELOG.html
==============================================================================
*** branches/phpBB-3_0_0/phpBB/docs/CHANGELOG.html (original)
--- branches/phpBB-3_0_0/phpBB/docs/CHANGELOG.html Wed Aug  5 14:01:03 2009
***************
*** 188,193 ****
--- 188,194 ----
  		<li>[Fix] Adjust build_url() to not prepend $phpbb_root_path if path returned from redirect() is an URL. This fixes redirect issues with some installations and bridges. (Bug #47535)</li>
  		<li>[Fix] Do not mark global announcements as read if all topics in a forum become read (Bug #15729).</li>
  		<li>[Fix] Fix general error while registration, through undefined variable $config in validate_referer (Bug #49035 - Patch by wjvriend)</li>
+ 		<li>[Change] submit_post() now accepts force_approved_state key passed to $data to indicate new posts being approved (true) or unapproved (false).</li>
  		<li>[Change] Change the data format of the default file ACM to be more secure from tampering and have better performance.</li>
  		<li>[Change] Add index on log_time to the log table to prevent slowdown on boards with many log entries. (Bug #44665 - Patch by bantu)</li>
  		<li>[Change] Template engine now permits to a limited extent variable includes.</li>

Modified: branches/phpBB-3_0_0/phpBB/includes/functions_posting.php
==============================================================================
*** branches/phpBB-3_0_0/phpBB/includes/functions_posting.php (original)
--- branches/phpBB-3_0_0/phpBB/includes/functions_posting.php Wed Aug  5 14:01:03 2009
***************
*** 1674,1687 ****
--- 1674,1695 ----
  	}
  
  	// This variable indicates if the user is able to post or put into the queue - it is used later for all code decisions regarding approval
+ 	// The variable name should be $post_approved, because it indicates if the post is approved or not
  	$post_approval = 1;
  
  	// Check the permissions for post approval. Moderators are not affected.
  	if (!$auth->acl_get('f_noapprove', $data['forum_id']) && !$auth->acl_get('m_approve', $data['forum_id']))
  	{
+ 		// Post not approved, but in queue
  		$post_approval = 0;
  	}
  
+ 	// Mods are able to force approved/unapproved posts. True means the post is approved, false the post is unapproved
+ 	if ($data['force_approved_state'])
+ 	{
+ 		$post_approval = ($data['force_approved_state']) ? 1 : 0;
+ 	}
+ 
  	// Start the transaction here
  	$db->sql_transaction('begin');