makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-27 21:58:52 +00:00
Code Issues Projects Releases Packages Wiki Activity

[ticket/14257] Use migrations instead of cron job for some reparsers

Browse source 
PHPBB3-14257
This commit is contained in:
Oliver Schramm 2015-10-25 02:07:40 +01:00
parent 8b0f8d7b3c
commit 33500fd372
2 changed files with 87 additions and 52 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

52
phpBB/config/default/container/services_cron.yml
View file

@ -147,58 +147,6 @@ services:
tags:
- { name: cron.task }
cron.task.text_reparser.contact_admin_info:
class: phpbb\cron\task\text_reparser\reparser
arguments:
- @config
- @config_text
- @text_reparser.lock
- @text_reparser_collection
calls:
- [set_name, [cron.task.text_reparser.contact_admin_info]]
- [set_reparser, [text_reparser.contact_admin_info]]
tags:
- { name: cron.task }
cron.task.text_reparser.forum_description:
class: phpbb\cron\task\text_reparser\reparser
arguments:
- @config
- @config_text
- @text_reparser.lock
- @text_reparser_collection
calls:
- [set_name, [cron.task.text_reparser.forum_description]]
- [set_reparser, [text_reparser.forum_description]]
tags:
- { name: cron.task }
cron.task.text_reparser.forum_rules:
class: phpbb\cron\task\text_reparser\reparser
arguments:
- @config
- @config_text
- @text_reparser.lock
- @text_reparser_collection
calls:
- [set_name, [cron.task.text_reparser.forum_rules]]
- [set_reparser, [text_reparser.forum_rules]]
tags:
- { name: cron.task }
cron.task.text_reparser.group_description:
class: phpbb\cron\task\text_reparser\reparser
arguments:
- @config
- @config_text
- @text_reparser.lock
- @text_reparser_collection
calls:
- [set_name, [cron.task.text_reparser.group_description]]
- [set_reparser, [text_reparser.group_description]]
tags:
- { name: cron.task }
cron.task.text_reparser.pm_text:
class: phpbb\cron\task\text_reparser\reparser
arguments:

87
phpBB/phpbb/db/migration/data/v320/reparse_fast.php Normal file
View file

@ -0,0 +1,87 @@
<?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\v320;
class reparse_fast extends \phpbb\db\migration\container_aware_migration
{
static public function depends_on()
{
return array('\phpbb\db\migration\data\v310\contact_admin_form');
}
public function effectively_installed()
{
return isset($this->config['reparse_lock']);
}
public function update_data()
{
return array(
array('config.add', array('reparse_lock', 0, true)),
array('custom', array(array($this, 'reparse'))),
);
}
public function reparse($resume_data)
{
// Somtimes a cron job is too much
$limit = 200;
$fast_reparsers = array(
'text_reparser.contact_admin_info',
'text_reparser.forum_description',
'text_reparser.forum_rules',
'text_reparser.group_description',
);
if (!is_array($resume_data))
{
$resume_data = array(
'reparser' => 0,
'current' => $this->container->get($fast_reparsers[0])->get_max_id(),
);
}
$fast_reparsers_size = sizeof($fast_reparsers);
$processed_records = 0;
while ($processed_records < $limit && $resume_data['reparser'] < $fast_reparsers_size)
{
$reparser = $this->container->get($fast_reparsers[$resume_data['reparser']]);
// New reparser
if ($resume_data['current'] === 0)
{
$resume_data['current'] = $reparser->get_max_id();
}
$start = max(1, $resume_data['current'] + 1 - ($limit - $processed_records));
$end = max(1, $resume_data['current']);
$reparser->reparse_range($start, $end);
$processed_records = $end - $start + 1;
$resume_data['current'] = $start - 1;
if ($start === 1)
{
$resume_data['reparser']++;
}
}
if ($resume_data['reparser'] === $fast_reparsers_size)
{
return true;
}
return $resume_data;
}
}