mirror/php-phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-05-06 15:45:34 +02:00
Code Issues Releases Wiki Activity

[ticket/11871] Fallback to int ID of the module, when creating links

Browse Source 
Firefox is currently unable to correctly copy a urlencoded \ so users will be
unable to post links to modules. However we can still fallback to the id
instead of the name, so we do that in this case.

PHPBB3-11871
This commit is contained in:
Joas Schilling 2013-10-02 22:37:16 +02:00
parent e0adb15074
commit 2ab3ee7d67
1 changed files with 27 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
phpBB/includes/functions_module.php
View File

@ -519,7 +519,7 @@ class p_master
} }
// Not being able to overwrite ;) // Not being able to overwrite ;)
$this->module->u_action = append_sid("{$phpbb_admin_path}index.$phpEx", "i={$this->p_name}") . (($icat) ? '&icat=' . $icat : '') . "&mode={$this->p_mode}"; $this->module->u_action = append_sid("{$phpbb_admin_path}index.$phpEx", 'i=' . $this->get_module_identifier($this->p_name, $this->p_id)) . (($icat) ? '&icat=' . $icat : '') . "&mode={$this->p_mode}";
} }
else else
{ {
@ -551,7 +551,7 @@ class p_master
$this->module->u_action = $phpbb_root_path . (($user->page['page_dir']) ? $user->page['page_dir'] . '/' : '') . $user->page['page_name']; $this->module->u_action = $phpbb_root_path . (($user->page['page_dir']) ? $user->page['page_dir'] . '/' : '') . $user->page['page_name'];
} }
$this->module->u_action = append_sid($this->module->u_action, "i={$this->p_name}") . (($icat) ? '&icat=' . $icat : '') . "&mode={$this->p_mode}"; $this->module->u_action = append_sid($this->module->u_action, 'i=' . $this->get_module_identifier($this->p_name, $this->p_id)) . (($icat) ? '&icat=' . $icat : '') . "&mode={$this->p_mode}";
} }
// Add url_extra parameter to u_action url // Add url_extra parameter to u_action url
@ -799,12 +799,12 @@ class p_master
// if the item has a name use it, else use its id // if the item has a name use it, else use its id
if (empty($item_ary['name'])) if (empty($item_ary['name']))
{ {
$u_title .= $item_ary['id']; $u_title .= $item_ary['id'];
} }
else else
{ {
// if the category has a name, then use it. // if the category has a name, then use it.
$u_title .= $item_ary['name']; $u_title .= $this->get_module_identifier($item_ary['name'], $item_ary['id']);
} }
// If the item is not a category append the mode // If the item is not a category append the mode
if (!$item_ary['cat']) if (!$item_ary['cat'])
@ -982,6 +982,29 @@ class p_master
return substr($basename, strlen($this->p_class) + 1); return substr($basename, strlen($this->p_class) + 1);
} }
/**
* If the basename contains a \ we dont use that for the URL.
*
* Firefox is currently unable to correctly copy a urlencoded \
* so users will be unable to post links to modules.
* However we can still fallback to the id instead of the name,
* so we do that in this case.
*
* @param string $basename Basename of the module
* @param int $id Id of the module
* @return mixed Identifier that should be used for
* module link creation
*/
protected function get_module_identifier($basename, $id)
{
if (strpos($basename, '\\') === false)
{
return $basename;
}
return $id;
}
/** /**
* Checks whether the given module basename is a correct class name * Checks whether the given module basename is a correct class name
* *