Julian/phpbb_board3-portal_tlw
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[ticket/506] Only check styles/all folder for module images and fix tests

Browse Source 
B3P-506
This commit is contained in:
Marc Alexander
2015-03-15 13:33:10 +01:00
parent f8de6be004
commit 08a4d74937
2 changed files with 13 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
includes/functions.php
View File

@@ -427,36 +427,24 @@ function get_portal_tracking_info($fetch_news)
*/ */
function check_file_src($value, $key, $module_id, $force_error = true) function check_file_src($value, $key, $module_id, $force_error = true)
{ {
global $db, $phpbb_root_path, $phpEx, $user; global $phpbb_admin_path, $portal_root_path, $phpEx, $user;
$error = ''; $error = '';
// We check if the chosen file is present in all active styles // We check if the chosen file is present in the styles/all/ folder
$sql = 'SELECT style_path if (!file_exists($portal_root_path . 'styles/all/theme/images/portal/' . $value))
FROM ' . STYLES_TABLE . '
WHERE style_active = 1';
$result = $db->sql_query($sql);
while ($row = $db->sql_fetchrow($result))
{ {
if (!file_exists($phpbb_root_path . 'styles/' . $row['style_path'] . '/theme/images/portal/' . $value) && $error .= $user->lang['B3P_FILE_NOT_FOUND'] . ': styles/all/theme/images/portal/' . $value . '<br />';
!file_exists($phpbb_root_path . 'ext/board3/portal/styles/' . $row['style_path'] . '/theme/images/portal/' . $value))
{
$error .= $user->lang['B3P_FILE_NOT_FOUND'] . ': styles/' . $row['style_path'] . '/theme/images/portal/' . $value . '<br />';
}
} }
$db->sql_freeresult($result);
if (!empty($error)) if (!empty($error))
{ {
if ($force_error) if ($force_error)
{ {
trigger_error($error . adm_back_link(append_sid("{$phpbb_root_path}adm/index.$phpEx", 'i=\board3\portal\acp\portal_module&amp;mode=config&amp;module_id=' . $module_id)), E_USER_WARNING); trigger_error($error . adm_back_link(append_sid("{$phpbb_admin_path}index.$phpEx", 'i=\board3\portal\acp\portal_module&amp;mode=config&amp;module_id=' . $module_id)), E_USER_WARNING);
}
else
{
return $error;
} }
return $error;
} }
else else
{ {

8
tests/unit/functions/check_file_src_test.php
View File

@@ -15,10 +15,14 @@ class phpbb_functions_check_file_src_test extends \board3\portal\tests\testframe
{ {
public function setUp() public function setUp()
{ {
global $phpbb_root_path, $portal_root_path;
parent::setUp(); parent::setUp();
global $user; global $user;
$user = new phpbb_mock_user(); $user = new phpbb_mock_user();
$portal_root_path = $phpbb_root_path . 'ext/board3/portal/';
} }
public function getDataSet() public function getDataSet()
@@ -29,7 +33,7 @@ class phpbb_functions_check_file_src_test extends \board3\portal\tests\testframe
public function test_check_file_src() public function test_check_file_src()
{ {
$this->assertFalse(check_file_src('portal_attach.png', '', 15, false)); $this->assertFalse(check_file_src('portal_attach.png', '', 15, false));
$this->assertEquals(': styles/prosilver/theme/images/portal/portal_foobar.png<br />', check_file_src('portal_foobar.png', '', 15, false)); $this->assertEquals(': styles/all/theme/images/portal/portal_foobar.png<br />', check_file_src('portal_foobar.png', '', 15, false));
} }
public function test_check_file_src_error() public function test_check_file_src_error()
@@ -38,6 +42,6 @@ class phpbb_functions_check_file_src_test extends \board3\portal\tests\testframe
$phpbb_dispatcher = new phpbb_mock_event_dispatcher; $phpbb_dispatcher = new phpbb_mock_event_dispatcher;
$this->setExpectedTriggerError(E_USER_WARNING); $this->setExpectedTriggerError(E_USER_WARNING);
$this->assertEquals(': styles/prosilver/theme/images/portal/portal_foobar.png<br />', check_file_src('portal_foobar.png', '', 15)); $this->assertEquals(': styles/all/theme/images/portal/portal_foobar.png<br />', check_file_src('portal_foobar.png', '', 15));
} }
} }