Comments on: Separating data from execution in osCommerce boxes

By: Alexandra

Alexandra — Mon, 17 Mar 2014 22:39:07 +0000

Ok, I figured it out. :)
This part should be below the curly braces
return $data;
}
}
}
function execute() {
global $oscTemplate;

By: Alexandra

Alexandra — Mon, 17 Mar 2014 22:00:04 +0000

Hi George,
I followed your instructions and was able to separate data from the bm_information box and output the data into another file.
I am trying to separate data now for bm_currencies. I get a Fatal error: Call to undefined method bm_currencies::execute()

This is what I have done. Any idea as to what I have done wrong? Thank you in advance.

function dataF() {
global $PHP_SELF, $currencies, $HTTP_GET_VARS, $request_type, $currency;

if (substr(basename($PHP_SELF), 0, 8) != ‘checkout’) {
if (isset($currencies) && is_object($currencies) && (count($currencies->currencies) > 1)) {
reset($currencies->currencies);
$currencies_array = array();
while (list($key, $value) = each($currencies->currencies)) {
$currencies_array[] = array(‘id’ => $key, ‘text’ => $value['title']);
}

$hidden_get_variables = ”;
reset($HTTP_GET_VARS);
while (list($key, $value) = each($HTTP_GET_VARS)) {
if ( is_string($value) && ($key != ‘currency’) && ($key != tep_session_name()) && ($key != ‘x’) && ($key != ‘y’) ) {
$hidden_get_variables .= tep_draw_hidden_field($key, $value);
}
}

$data = ” .
‘ ‘ . MODULE_BOXES_CURRENCIES_BOX_TITLE . ” .
‘ ‘ .
‘ ‘ . tep_draw_form(‘currencies’, tep_href_link(basename($PHP_SELF), ”, $request_type, false), ‘get’) .
‘ ‘ . tep_draw_pull_down_menu(‘currency’, $currencies_array, $currency, ‘onchange=”this.form.submit();” style=”width: 100%”‘) . $hidden_get_variables . tep_hide_session_id() . ” .
‘ ‘ .
”;

return $data;
}
function execute() {
global $oscTemplate;
$oscTemplate->addBlock($this->dataF(), $this->group);
}
}
}

By: Zeeshan

Zeeshan — Thu, 20 Jun 2013 08:06:44 +0000

Very Useful topic.
Thanks for writing this.

By: Ludwig Arcache

Ludwig Arcache — Sat, 04 Aug 2012 11:13:44 +0000

never mind, its done… thank you for the information, sir!!

By: Ludwig Arcache

Ludwig Arcache — Sat, 04 Aug 2012 10:21:27 +0000

Hello, well thank you for this posting this article.
i understood very well how to split the execute function. but i want to know:

-should i leave the rest of the code ? (function isEnabled(), function check(), ect…)

-after editing the file (bm_manufacturer_info), should i upload and replace the existing file?

By: Rick

Rick — Thu, 05 Jul 2012 00:49:15 +0000

Thank you very much, I understand better now.

By: multimixer

multimixer — Wed, 04 Jul 2012 06:19:05 +0000

There is no difference for box bm_categories.php, its the same process like for any other boxes: You slit the function execute() into 2 functions, the one is dataF() that return the output, the other execute() that add the box to a “group”

Concrete, in bm_categories.php you have

function execute() {
global $SID, $oscTemplate;

if ((USE_CACHE == ‘true’) && empty($SID)) {
$output = tep_cache_categories_box();
} else {
$output = $this->getData();
}

$oscTemplate->addBlock($output, $this->group);
}

This you can replace with following

function dataF() {
global $SID;

if ((USE_CACHE == ‘true’) && empty($SID)) {
$output = tep_cache_categories_box();
} else {
$output = $this->getData();
}

return $output;
}

function execute() {
global $oscTemplate;

$oscTemplate->addBlock($this->dataF(), $this->group);
}

By: Rick

Rick — Tue, 03 Jul 2012 21:38:15 +0000

This all makes great sense and thank you for posting this tutorial.

However I am confused where to split the function in the bm_categories.php file so that I can call it where I would like to?

Thanks for any help in advance.

By: Steve

Steve — Wed, 04 Jan 2012 11:27:19 +0000

Yep it was point 1 that solved it, I had the code correct in the data A call, it just didnt show when I posted it.

thanks alot

By: multimixer

multimixer — Tue, 03 Jan 2012 13:23:24 +0000

Hi Steve

From a first sight I think that you need to do following to get it working

1) Add all “globals” that you have now in function execute() to function dataA(), except $oscTemplate that should stay in function execute()

2) Add to the page the complete piece of code like it is done for the shopping cart example. Simply “dataA()” says nothing

3) make sure you have the manufacturer info box installed in admin. It has not to be enabled, but it need to be active

Hope it helps