CMS MADE SIMPLE FORGE

CGMyOrders

 

[#8760] orderlisttemplate, invoicetemplate and useridtemplate params are not taken into account

avatar
Created By: Toff. (toff)
Date Submitted: Mon Dec 17 19:53:46 -0500 2012

Assigned To: Robert Campbell (calguy1000)
Version: 1.0
CMSMS Version: 1.11.2.1
Severity: Major
Resolution: None
State: Open
Summary:
orderlisttemplate, invoicetemplate and useridtemplate params are not taken into account
Detailed Description:
Hi Robert,

As always, a real treat to use your modules!
Thanks heaps.

Just noticed that the module's actions are not using the  template params such
as orderlisttemplate="".

I haven't checked whether this was already fixed in SVN, so don't mind me if
that was the case.


For everyone else, here is a quick temporary fix:


In action.default.php:


1. Comment the following lines at the end of the file (~line 126)

//debug_display($the_template);
//echo $this->ProcessTemplateFromDatabase('orderlist_'.$the_template);


2. Add the following just after:

if( isset($params['orderlisttemplate'] ) )
  {
    $the_template = $params['orderlisttemplate'];
  }
echo $this->ProcessTemplateFromDatabase('orderlist_'.$the_template);



The same would need to be done for the other actions (view invoice & user_id)


action.view_invoice.php:


- add
if( isset($params['invoicetemplate'] ) )
  {
    $the_template = $params['invoicetemplate'];
  }

- before:
echo $orders_mod->ProcessTemplateFromDatabase('invoice_'.$the_template);




action.userid.php:

- add
if( isset($params['useridtemplate'] ) )
  {
    $userid_template = $params['useridtemplate'];
  }

- before:
echo $this->ProcessTemplateFromDatabase('useridform_'.$userid_template);


History

Comments
avatar
Date: 2013-08-21 06:16
Posted By: Jonathan Schmid (Foaly*)

Also, need to take pass them through from default action.

Diff file: http://jonathanschmid.de/ext/BR8760.diff
      
Updates

Updated: 2012-12-17 20:07
summary: orderlisttemplate param is not taken into account => orderlisttemplate, invoicetemplate and useridtemplate params are not taken into account

Updated: 2012-12-17 20:06
description: Hi Robert, As always, a real treat to use your modules! Thanks heaps. Just noticed that the module's actions are not using the template params such as orderlisttemplate="". I haven't checked whether this was already fixed in SVN, so don't mind => Hi Robert, As always, a real treat to use your modules! Thanks heaps. Just noticed that the module's actions are not using the template params such as orderlisttemplate="". I haven't checked whether this was already fixed in SVN, so don't mind

Updated: 2012-12-17 20:05
description: Hi Robert, As always, a real treat to use your modules! Thanks heaps. Just noticed that the module's actions are not using the template params such as orderlisttemplate="". I haven't checked whether this was already fixed in SVN, so don't mind => Hi Robert, As always, a real treat to use your modules! Thanks heaps. Just noticed that the module's actions are not using the template params such as orderlisttemplate="". I haven't checked whether this was already fixed in SVN, so don't mind
resolution_id: => 5