composer/iTop
6
0
Fork 0
mirror of https://github.com/Combodo/iTop.git synced 2026-02-13 07:24:13 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fixed regression due to forcing camel case names in operation methods, now allow both camel case and snake case

Browse Source
This commit is contained in:
Eric Espie
2023-05-30 15:25:00 +02:00
parent 173960e717
commit 2c4c69fcfc
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
sources/Application/TwigBase/Controller/Controller.php
View File

@@ -168,17 +168,19 @@ abstract class Controller extends AbstractController
$this->CheckAccess();
$this->m_sOperation = utils::ReadParam('operation', $this->m_sDefaultOperation);
$sMethodName = 'Operation'.utils::ToCamelCase($this->m_sOperation);
$oKPI = new ExecutionKPI();
$oKPI->ComputeAndReport('Starting operation '.$this->m_sOperation);
if (method_exists($this, $sMethodName))
{
$this->$sMethodName();
if ($this->CallOperation(utils::ToCamelCase($this->m_sOperation))) {
return;
}
else
{
$this->DisplayBadRequest();
// Fallback to unchanged names for compatibility
if ($this->CallOperation($this->m_sOperation)) {
return;
}
$this->DisplayBadRequest();
}
catch (Exception $e)
{