[ticket/351] Use version_helper and json version file for version check
B3P-351
This commit is contained in:
@@ -22,14 +22,9 @@ class version_check
|
||||
protected $config;
|
||||
|
||||
/**
|
||||
* @var string phpbb_root_path
|
||||
* @var \phpbb\version_helper $version_helper phpBB version helper
|
||||
*/
|
||||
protected $phpbb_root_path;
|
||||
|
||||
/**
|
||||
* @var string PHP file extension
|
||||
*/
|
||||
protected $php_ext;
|
||||
protected $version_helper;
|
||||
|
||||
/**
|
||||
* @var \phpbb\template\twig\twig
|
||||
@@ -41,125 +36,75 @@ class version_check
|
||||
*/
|
||||
protected $user;
|
||||
|
||||
/**
|
||||
* @var string Current version
|
||||
*/
|
||||
protected $current_version;
|
||||
|
||||
/**
|
||||
* Construct a version_check object
|
||||
*
|
||||
* @param array $version_data Version data
|
||||
* @param \phpbb\config\config $config phpBB config
|
||||
* @param string $phpbb_root_path phpBB root path
|
||||
* @param string $php_ext PHP file extension
|
||||
* @param \phpbb\version_helper $version_helper phpBB version helper
|
||||
* @param \phpbb\template\twig\twig $template phpBB template object
|
||||
* @param \phpbb\user $user phpBB user object
|
||||
*/
|
||||
public function __construct($version_data, $config, $phpbb_root_path, $php_ext, $template, $user)
|
||||
public function __construct($version_data, $config, $version_helper, $template, $user)
|
||||
{
|
||||
$this->version_data = $version_data;
|
||||
$this->config = $config;
|
||||
$this->phpbb_root_path = $phpbb_root_path;
|
||||
$this->php_ext = $php_ext;
|
||||
$this->version_helper = $version_helper;
|
||||
$this->template = $template;
|
||||
$this->user = $user;
|
||||
$this->current_version = $this->config[str_replace(' ', '', $this->version_data['version'])];
|
||||
}
|
||||
|
||||
/**
|
||||
* Check MOD version
|
||||
* Check MOD version and assign template variables for version info if not
|
||||
* returning current version
|
||||
*
|
||||
* @param bool $return_version Yes if current version should be returned
|
||||
* @return string Current version if $return_version is set to true
|
||||
*/
|
||||
public function check($return_version = false)
|
||||
{
|
||||
if (!function_exists('get_remote_file'))
|
||||
{
|
||||
include($this->phpbb_root_path . 'includes/functions_admin.' . $this->php_ext);
|
||||
}
|
||||
// Set file location
|
||||
$this->version_helper->set_file_location($this->version_data['file'][0], $this->version_data['file'][1], $this->version_data['file'][2]);
|
||||
// Set current version
|
||||
$this->version_helper->set_current_version($this->current_version);
|
||||
|
||||
// Fill with bogus data
|
||||
$this->get_empty_data($mod_version, $data);
|
||||
$this->version_helper->force_stability(($this->config['extension_force_unstable'] || !$this->version_helper->is_stable($this->current_version)) ? 'unstable' : null);
|
||||
|
||||
// Get version info from server
|
||||
$this->get_version_info($mod_version, $data);
|
||||
$updates = $this->version_helper->get_suggested_updates(true);
|
||||
|
||||
// remove spaces from the version in the mod file stored locally
|
||||
$version = $this->config[str_replace(' ', '', $this->version_data['version'])];
|
||||
// Return version if $return_version is set to true
|
||||
if ($return_version)
|
||||
{
|
||||
return $version;
|
||||
return $this->current_version;
|
||||
}
|
||||
|
||||
$version_compare = (version_compare($version, $mod_version, '<')) ? false : true;
|
||||
$version_up_to_date = empty($updates);
|
||||
|
||||
$this->template->assign_block_vars('mods', array(
|
||||
'ANNOUNCEMENT' => (string) $data['announcement'],
|
||||
$template_data = array(
|
||||
'AUTHOR' => $this->version_data['author'],
|
||||
'CURRENT_VERSION' => $version,
|
||||
'DESCRIPTION' => (string) $data['description'],
|
||||
'DOWNLOAD' => (string) $data['download'],
|
||||
'LATEST_VERSION' => $mod_version,
|
||||
'TITLE' => (string) $data['title'],
|
||||
|
||||
'UP_TO_DATE' => sprintf((!$version_compare) ? $this->user->lang['NOT_UP_TO_DATE'] : $this->user->lang['UP_TO_DATE'], $data['title']),
|
||||
|
||||
'S_UP_TO_DATE' => $version_compare,
|
||||
|
||||
'CURRENT_VERSION' => $this->current_version,
|
||||
'UP_TO_DATE' => sprintf((!$version_up_to_date) ? $this->user->lang['NOT_UP_TO_DATE'] : $this->user->lang['UP_TO_DATE'], $this->version_data['title']),
|
||||
'S_UP_TO_DATE' => $version_up_to_date,
|
||||
'U_AUTHOR' => 'http://www.phpbb.com/community/memberlist.php?mode=viewprofile&un=' . $this->version_data['author'],
|
||||
));
|
||||
}
|
||||
|
||||
/**
|
||||
* Fill variables with empty bogus data
|
||||
*
|
||||
* @param string $mod_version Mod version
|
||||
* @param array $data Array containing mod info
|
||||
*
|
||||
* @return null
|
||||
*/
|
||||
protected function get_empty_data(&$mod_version, &$data)
|
||||
{
|
||||
// Fill with bogus data
|
||||
$mod_version = $this->user->lang['NO_INFO'];
|
||||
$data = array(
|
||||
'title' => $this->version_data['title'],
|
||||
'description' => $this->user->lang['NO_INFO'],
|
||||
'download' => $this->user->lang['NO_INFO'],
|
||||
'announcement' => $this->user->lang['NO_INFO'],
|
||||
'TITLE' => (string) $this->version_data['title'],
|
||||
'LATEST_VERSION' => $this->current_version,
|
||||
);
|
||||
}
|
||||
|
||||
/**
|
||||
* Get version info from remote server
|
||||
*
|
||||
* @param string $mod_version Mod version
|
||||
* @param array $data Array containing mod info
|
||||
*
|
||||
* @return null
|
||||
*/
|
||||
protected function get_version_info(&$mod_version, &$data)
|
||||
{
|
||||
// Get current and latest version
|
||||
$errstr = '';
|
||||
$errno = 0;
|
||||
$var = $this->version_data;
|
||||
|
||||
$file = get_remote_file($this->version_data['file'][0], '/' . $this->version_data['file'][1], $this->version_data['file'][2], $errstr, $errno);
|
||||
|
||||
if ($file)
|
||||
if (!$version_up_to_date)
|
||||
{
|
||||
// let's not stop the page from loading if a mod author messed up their mod check file
|
||||
// also take care of one of the easiest ways to mess up an xml file: "&"
|
||||
$mod = @simplexml_load_string(str_replace('&', '&', $file));
|
||||
if (isset($mod->$var['tag']))
|
||||
{
|
||||
$row = $mod->$var['tag'];
|
||||
$mod_version = $row->mod_version->major . '.' . $row->mod_version->minor . '.' . $row->mod_version->revision . $row->mod_version->release;
|
||||
|
||||
$data = array(
|
||||
'title' => $row->title,
|
||||
'description' => $row->description,
|
||||
'download' => $row->download,
|
||||
'announcement' => $row->announcement,
|
||||
);
|
||||
}
|
||||
$updates = array_shift($updates);
|
||||
$template_data = array_merge($template_data, array(
|
||||
'ANNOUNCEMENT' => (string) $updates['announcement'],
|
||||
'DOWNLOAD' => (string) $updates['download'],
|
||||
'LATEST_VERSION' => $updates['current'],
|
||||
));
|
||||
}
|
||||
$this->template->assign_block_vars('mods', $template_data);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user