CMS 1.08 bug in admin/addtemplate.php

A place to discuss the testing process in beta cycles or against SVN for the CMS Made Simple CORE package.
Locked
ajlaird_au

CMS 1.08 bug in admin/addtemplate.php

Post by ajlaird_au »

When reporting an error in inserting a template, the error message for an insert failure includes the query used to check for duplicate template names. This is misleading as the query is totally irrelevant to the error at that point.

if ($result)
{
#Perform the addtemplate_post callback
foreach($gCms->modules as $key=>$value)
{
if ($gCms->modules[$key]['installed'] == true &&
$gCms->modules[$key]['active'] == true)
{
$gCms->modules[$key]['object']->AddTemplatePost($newtemplate);
}
}

Events::SendEvent('Core', 'AddTemplatePost', array('template' => &$newtemplate));

audit($newtemplate->id, $template, 'Added Template');
redirect("listtemplates.php");
return;
}
else
{
$error .= "
  • ".lang('errorinsertingtemplate')."$query
  • ";
    }
    Locked

    Return to “[locked] Quality Assurance”