The comments area of the blog is not good for posting any code, it get corrupted

Anyway, first question is: What is not working?

title = MODULE_BOXES_ADSENSE_TITLE;
$this->description = MODULE_BOXES_ADSENSE_DESCRIPTION;

if ( defined(‘MODULE_BOXES_ADSENSE_STATUS’) ) {
$this->sort_order = MODULE_BOXES_ADSENSE_SORT_ORDER;
$this->enabled = (MODULE_BOXES_ADSENSE_STATUS == ‘True’);

$this->group = ((MODULE_BOXES_ADSENSE_CONTENT_PLACEMENT == ‘Left Column’) ? ‘boxes_column_left’ : ‘boxes_column_right’);
}
}

function dataF() {

$data = ”;
$data .= MODULE_BOXES_ADSENSE_CODE;
$data .=”;

return $data;
}

function execute() {
global $oscTemplate;

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

function isEnabled() {
return $this->enabled;
}

function check() {
return defined(‘MODULE_BOXES_ADSENSE_STATUS’);
}

function install() {
tep_db_query(“insert into ” . TABLE_CONFIGURATION . ” (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) values (‘Enable adsense Module’, ‘MODULE_BOXES_ADSENSE_STATUS’, ‘True’, ‘Do you want to add the module to your shop?’, ’6′, ’1′, ‘tep_cfg_select_option(array(\’True\’, \’False\’), ‘, now())”);
tep_db_query(“insert into ” . TABLE_CONFIGURATION . ” (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) values (‘Content Placement’, ‘MODULE_BOXES_ADSENSE_CONTENT_PLACEMENT’, ‘Left Column’, ‘Should the module be loaded in the left or right column?’, ’6′, ’1′, ‘tep_cfg_select_option(array(\’Left Column\’, \’Right Column\’), ‘, now())”);
tep_db_query(“insert into ” . TABLE_CONFIGURATION . ” (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, date_added) values (‘Sort Order’, ‘MODULE_BOXES_ADSENSE_SORT_ORDER’, ’0′, ‘Sort order of display. Lowest is displayed first.’, ’6′, ’0′, now())”);
tep_db_query( “insert into ” . TABLE_CONFIGURATION . ” (configuration_title, configuration_key, configuration_value, configuration_description, configuration_group_id, sort_order, set_function, date_added) values ( ‘adSense code’, ‘MODULE_BOXES_ADSENSE_CODE’, ”, ‘Paste the adSense code into here’, ’6′, ’4′, ‘tep_draw_textarea_field(\’configuration[MODULE_BOXES_ADSENSE_CODE]\’, false, 35, 20, ‘, now())” );
}

function remove() {
tep_db_query(“delete from ” . TABLE_CONFIGURATION . ” where configuration_key in (‘” . implode(“‘, ‘”, $this->keys()) . “‘)”);
}

function keys() {
return array(‘MODULE_BOXES_ADSENSE_STATUS’, ‘MODULE_BOXES_ADSENSE_CONTENT_PLACEMENT’, ‘MODULE_BOXES_ADSENSE_SORT_ORDER’, ‘MODULE_BOXES_ADSENSE_CODE’);
}
}
?>

If you can’t see the box at all when you click on “install” on the top right corner in admin (under modules>boxes) then I would suspect that you missed some point at the replacements, in the example above from “information” to “adsense”

Most possible the issue is in the constants, whatever is in CAPITAL_LETTERS

What can be wrong?

Thanks

Doesn’t it make the lines shorter and more readable if I don’t use the operator “||” and by storing $PHP_SELF into a variable? Is there any side-effect the way I did it?

This comes because Mini Template System “re-execute” the boxes depending on your template settings. That is the reason why “data” has been separated from “execution”

If you want to add any conditions, best is you do this in function “dataF()” You could by the way merge all of your conditions into one using “or” that look like this “||”.

Something like this: if (condition1 || condition2 || condition3); return;

Not sure also why you store $PHP_SELF into a variable and don’t use it as is?