CMS MADE SIMPLE FORGE

CGEcommerceBase

 
Back Back to List

[#11843] {cgecomm_company_address assign=foo} does not work

avatar
Created By: Rolf (rolf1)
Date Submitted: Sat Jun 23 07:26:41 -0400 2018

Assigned To:
Version: 1.8.9
CMSMS Version: 2.2.9.1
Severity: None
Resolution: Fixed
State: Open
Summary:
{cgecomm_company_address assign=foo} does not work
Detailed Description:
Notice: Undefined variable: my_addr in
/home/public_html/modules/CGEcommerceBase/lib/class.smarty_plugins.php on line
120

line 120:
$smarty->assign($params['assign'],$my_addr);

should be

$smarty->assign($params['assign'],$addr);

Thanks Rolf


History

Comments
avatar 
Date: 2019-02-18 08:43
Posted By: Rolf (rolf1)

Not yet fixed in release 1.8.9 and core 2.2.9.1
avatar 
Date: 2020-01-10 05:56
Posted By: Deleted User (deleteduser_11200)

Verified bug not yet fixed in 1.8.10.

I applied a patch manually.

"class.smarty_plugins.php" line 120 of 204 --58%-- col 5


114     public static function cgecomm_company_address($params,&$smarty)
115     {
116         $addr = \cg_ecomm::get_company_address();
117         if( !$addr ) $addr = new \cg_ecomm_company_address;
118 
119         if( isset($params['assign']) ) {
120             $smarty->assign($params['assign'],$my_addr);
121             return;
122         }
123         return $addr;
124     }
125
avatar 
Date: 2020-01-23 03:32
Posted By: Robert Campbell (calguy1000)

fixed in svn
Updates

Updated: 2020-01-23 03:32
resolution_id: 5 => 7

Updated: 2019-02-18 08:43
version_id: 31296 => 31423
resolution_id: => 5
cmsms_version_id: 31369 => 31462