[ticket/13685] Add phpbb_dispatcher to fulltext search constructors

PHPBB3-13685
This commit is contained in:
brunoais 2015-03-11 17:46:42 +00:00
parent 049f584111
commit 1e45a05000
14 changed files with 50 additions and 18 deletions

View file

@ -40,7 +40,7 @@ if (!class_exists($search_type))
} }
$error = false; $error = false;
$search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher);
if ($error) if ($error)
{ {

View file

@ -632,7 +632,7 @@ class acp_main
{ {
$error = false; $error = false;
$search_type = $config['search_type']; $search_type = $config['search_type'];
$search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $phpbb_dispatcher, $user);
if (!$search->index_created()) if (!$search->index_created())
{ {

View file

@ -598,7 +598,7 @@ class acp_search
*/ */
function init_search($type, &$search, &$error) function init_search($type, &$search, &$error)
{ {
global $phpbb_root_path, $phpEx, $user, $auth, $config, $db; global $phpbb_root_path, $phpEx, $user, $auth, $config, $db, $phpbb_dispatcher;
if (!class_exists($type) || !method_exists($type, 'keyword_search')) if (!class_exists($type) || !method_exists($type, 'keyword_search'))
{ {
@ -607,7 +607,7 @@ class acp_search
} }
$error = false; $error = false;
$search = new $type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new $type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher);
return $error; return $error;
} }

View file

@ -920,7 +920,7 @@ function delete_posts($where_type, $where_ids, $auto_sync = true, $posted_sync =
} }
$error = false; $error = false;
$search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher);
if ($error) if ($error)
{ {

View file

@ -2209,7 +2209,7 @@ function submit_post($mode, $subject, $username, $topic_type, &$poll, &$data, $u
} }
$error = false; $error = false;
$search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher);
if ($error) if ($error)
{ {

View file

@ -1144,7 +1144,7 @@ function mcp_delete_post($post_ids, $is_soft = false, $soft_delete_reason = '',
function mcp_fork_topic($topic_ids) function mcp_fork_topic($topic_ids)
{ {
global $auth, $user, $db, $template, $config; global $auth, $user, $db, $template, $config;
global $phpEx, $phpbb_root_path; global $phpEx, $phpbb_root_path, $phpbb_dispatcher;
if (!phpbb_check_ids($topic_ids, TOPICS_TABLE, 'topic_id', array('m_'))) if (!phpbb_check_ids($topic_ids, TOPICS_TABLE, 'topic_id', array('m_')))
{ {
@ -1222,7 +1222,7 @@ function mcp_fork_topic($topic_ids)
} }
$error = false; $error = false;
$search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher);
$search_mode = 'post'; $search_mode = 'post';
if ($error) if ($error)

View file

@ -497,7 +497,7 @@ function change_poster(&$post_info, $userdata)
{ {
// We do some additional checks in the module to ensure it can actually be utilised // We do some additional checks in the module to ensure it can actually be utilised
$error = false; $error = false;
$search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher);
if (!$error && method_exists($search, 'destroy_cache')) if (!$error && method_exists($search, 'destroy_cache'))
{ {

View file

@ -619,7 +619,7 @@ class install_convert extends module
{ {
global $template, $user, $phpbb_root_path, $phpEx, $db, $lang, $config, $cache, $auth; global $template, $user, $phpbb_root_path, $phpEx, $db, $lang, $config, $cache, $auth;
global $convert, $convert_row, $message_parser, $skip_rows, $language; global $convert, $convert_row, $message_parser, $skip_rows, $language;
global $request, $phpbb_config_php_file; global $request, $phpbb_config_php_file, $phpbb_dispatcher;
extract($phpbb_config_php_file->get_all()); extract($phpbb_config_php_file->get_all());
@ -796,7 +796,7 @@ class install_convert extends module
} }
$error = false; $error = false;
$convert->fulltext_search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $convert->fulltext_search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $phpbb_dispatcher, $user);
if ($error) if ($error)
{ {

View file

@ -1459,7 +1459,7 @@ class install_install extends module
*/ */
function build_search_index($mode, $sub) function build_search_index($mode, $sub)
{ {
global $db, $lang, $phpbb_root_path, $phpEx, $config, $auth, $user; global $db, $lang, $phpbb_root_path, $phpbb_dispatcher, $phpEx, $config, $auth, $user;
// Obtain any submitted data // Obtain any submitted data
$data = $this->get_submitted_data(); $data = $this->get_submitted_data();
@ -1492,7 +1492,7 @@ class install_install extends module
set_config_count(null, null, null, $config); set_config_count(null, null, null, $config);
$error = false; $error = false;
$search = new \phpbb\search\fulltext_native($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new \phpbb\search\fulltext_native($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher);
$sql = 'SELECT post_id, post_subject, post_text, poster_id, forum_id $sql = 'SELECT post_id, post_subject, post_text, poster_id, forum_id
FROM ' . POSTS_TABLE; FROM ' . POSTS_TABLE;

View file

@ -42,6 +42,12 @@ class fulltext_mysql extends \phpbb\search\base
*/ */
protected $db; protected $db;
/**
* PhpBB event dispatcher object
* @var \phpbb\event\dispatcher_interface
*/
protected $phpbb_dispatcher;
/** /**
* User object * User object
* @var \phpbb\user * @var \phpbb\user
@ -79,11 +85,13 @@ class fulltext_mysql extends \phpbb\search\base
* @param \phpbb\config\config $config Config object * @param \phpbb\config\config $config Config object
* @param \phpbb\db\driver\driver_interface Database object * @param \phpbb\db\driver\driver_interface Database object
* @param \phpbb\user $user User object * @param \phpbb\user $user User object
* @param \phpbb\event\dispatcher_interface $phpbb_dispatcher Event dispatcher object
*/ */
public function __construct(&$error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user) public function __construct(&$error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher)
{ {
$this->config = $config; $this->config = $config;
$this->db = $db; $this->db = $db;
$this->phpbb_dispatcher = $phpbb_dispatcher;
$this->user = $user; $this->user = $user;
$this->word_length = array('min' => $this->config['fulltext_mysql_min_word_len'], 'max' => $this->config['fulltext_mysql_max_word_len']); $this->word_length = array('min' => $this->config['fulltext_mysql_min_word_len'], 'max' => $this->config['fulltext_mysql_max_word_len']);

View file

@ -86,6 +86,12 @@ class fulltext_native extends \phpbb\search\base
*/ */
protected $db; protected $db;
/**
* PhpBB event dispatcher object
* @var \phpbb\event\dispatcher_interface
*/
protected $phpbb_dispatcher;
/** /**
* User object * User object
* @var \phpbb\user * @var \phpbb\user
@ -96,13 +102,15 @@ class fulltext_native extends \phpbb\search\base
* Initialises the fulltext_native search backend with min/max word length and makes sure the UTF-8 normalizer is loaded * Initialises the fulltext_native search backend with min/max word length and makes sure the UTF-8 normalizer is loaded
* *
* @param boolean|string &$error is passed by reference and should either be set to false on success or an error message on failure * @param boolean|string &$error is passed by reference and should either be set to false on success or an error message on failure
* @param \phpbb\event\dispatcher_interface $phpbb_dispatcher Event dispatcher object
*/ */
public function __construct(&$error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user) public function __construct(&$error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher)
{ {
$this->phpbb_root_path = $phpbb_root_path; $this->phpbb_root_path = $phpbb_root_path;
$this->php_ext = $phpEx; $this->php_ext = $phpEx;
$this->config = $config; $this->config = $config;
$this->db = $db; $this->db = $db;
$this->phpbb_dispatcher = $phpbb_dispatcher;
$this->user = $user; $this->user = $user;
$this->word_length = array('min' => $this->config['fulltext_native_min_chars'], 'max' => $this->config['fulltext_native_max_chars']); $this->word_length = array('min' => $this->config['fulltext_native_min_chars'], 'max' => $this->config['fulltext_native_max_chars']);

View file

@ -55,6 +55,12 @@ class fulltext_postgres extends \phpbb\search\base
*/ */
protected $db; protected $db;
/**
* PhpBB event dispatcher object
* @var \phpbb\event\dispatcher_interface
*/
protected $phpbb_dispatcher;
/** /**
* User object * User object
* @var \phpbb\user * @var \phpbb\user
@ -92,11 +98,13 @@ class fulltext_postgres extends \phpbb\search\base
* @param \phpbb\config\config $config Config object * @param \phpbb\config\config $config Config object
* @param \phpbb\db\driver\driver_interface Database object * @param \phpbb\db\driver\driver_interface Database object
* @param \phpbb\user $user User object * @param \phpbb\user $user User object
* @param \phpbb\event\dispatcher_interface $phpbb_dispatcher Event dispatcher object
*/ */
public function __construct(&$error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user) public function __construct(&$error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher)
{ {
$this->config = $config; $this->config = $config;
$this->db = $db; $this->db = $db;
$this->phpbb_dispatcher = $phpbb_dispatcher;
$this->user = $user; $this->user = $user;
$this->word_length = array('min' => $this->config['fulltext_postgres_min_word_len'], 'max' => $this->config['fulltext_postgres_max_word_len']); $this->word_length = array('min' => $this->config['fulltext_postgres_min_word_len'], 'max' => $this->config['fulltext_postgres_max_word_len']);

View file

@ -95,6 +95,12 @@ class fulltext_sphinx
*/ */
protected $dbtype; protected $dbtype;
/**
* PhpBB event dispatcher object
* @var \phpbb\event\dispatcher_interface
*/
protected $phpbb_dispatcher;
/** /**
* User object * User object
* @var \phpbb\user * @var \phpbb\user
@ -125,12 +131,14 @@ class fulltext_sphinx
* @param \phpbb\config\config $config Config object * @param \phpbb\config\config $config Config object
* @param \phpbb\db\driver\driver_interface Database object * @param \phpbb\db\driver\driver_interface Database object
* @param \phpbb\user $user User object * @param \phpbb\user $user User object
* @param \phpbb\event\dispatcher_interface $phpbb_dispatcher Event dispatcher object
*/ */
public function __construct(&$error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user) public function __construct(&$error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher)
{ {
$this->phpbb_root_path = $phpbb_root_path; $this->phpbb_root_path = $phpbb_root_path;
$this->php_ext = $phpEx; $this->php_ext = $phpEx;
$this->config = $config; $this->config = $config;
$this->phpbb_dispatcher = $phpbb_dispatcher;
$this->user = $user; $this->user = $user;
$this->db = $db; $this->db = $db;
$this->auth = $auth; $this->auth = $auth;

View file

@ -274,7 +274,7 @@ if ($keywords || $author || $author_id || $search_id || $submit)
} }
// We do some additional checks in the module to ensure it can actually be utilised // We do some additional checks in the module to ensure it can actually be utilised
$error = false; $error = false;
$search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user); $search = new $search_type($error, $phpbb_root_path, $phpEx, $auth, $config, $db, $user, $phpbb_dispatcher);
if ($error) if ($error)
{ {