I love this mod it's a great idea and will give me a great deal of flexibility to offer deals... Thanks for the great support and a great idea...

So - in essence - It's not really "free"...

This is just plain old advertising SPAM.

Hello Shanecbell,

can we see all the templates demo before people paid for the annual membership

Package\templates\tmpl_uni\css\adwords_mod.css
crazy styles here, try to make it pretty (fix all spaces with tabs)
Now here or no tabs, or 2 spaces or tab.
Very wild style :)

Next,
Package\adsense_mod\ASMconfig.php and
Package\adsense_mod\ASMdisplay.php
naming convention is not met
look,
$s1 =
Ooh, recheck
http://www.boonex.com/unity/txt/extension_convention
again
2. Variable Declarations: Names
here clean sample how you should named variables

Next, please keep spacing between see more lines where need, keep your code clean and easy to read.

Next,
recheck formatting
again read code convention please,
I mean { } symbols
keep it at different lines

Next
Package\admin\adsense_mod\AdSenseModAdmin.php
much better than all previous files, but still same errors here

you can check good styles of developing just in sources of dolphin/orca
you can check also msdn.com site ;-)

That would mean that I am a dumb ass. Thanks. I have uploaded the correct version. Thank you! Please review as soon as you are able and let me know if I have made the cut.

Looks like the problem with hard coded access is still there

define('DB_HOST','localhost');
define('DB_NAME','lifeisno_Boonex');
define('DB_USER','lifeisno_isog');
define('DB_PASS','97138');

May be you forget to upload the updated package.

The Version: 1.6.0000 file that was re-uploaded after your request uses the code below to load the settings off the header.inc.php file:

require_once( '../inc/header.inc.php' );
define('DB_HOST',$db['host']);
define('DB_NAME',$db['db']);
define('DB_USER',$db['user']);
define('DB_PASS',$db['passwd']);
if (!mysql_connect(DB_HOST, DB_USER, DB_PASS)) die('Error connecting to mysql database. Check mysql db access information ...');
if (!mysql_select_db(DB_NAME)) die('Error connecting to mysql see more database. Check mysql db name ...');
$mysql = mysql_query("SELECT * FROM `AdSenseMod_settings`");
while ($info = @mysql_fetch_array($mysql)) { @define($info['setting_id'],$info['value']); }
if (COMPONENT_ONOFF != '1') die("This component is disabled.");

And the v1.6.0 file is the same one available for download on the updated zip file attached to this extension entry.

Some conventions is not met (as I mentioned above!)
Also you have the following hardcoded config:

define('DB_HOST','localhost');
define('DB_NAME','lifeisno_Boonex');
define('DB_USER','lifeisno_isog');
define('DB_PASS','97138');

I guess I don't follow:

require_once( '../inc/header.inc.php' );
define('DB_HOST',$db['host']);
define('DB_NAME',$db['db']);
define('DB_USER',$db['user']);
define('DB_PASS',$db['passwd']);
if (!mysql_connect(DB_HOST, DB_USER, DB_PASS)) die('Error connecting to mysql database. Check mysql db access information ...');
if (!mysql_select_db(DB_NAME)) die('Error connecting to mysql database. Check mysql db name ...');
$mysql = mysql_query("SELECT * FROM `AdSenseMod_settings`");
while see more ($info = @mysql_fetch_array($mysql)) { @define($info['setting_id'],$info['value']); }
if (COMPONENT_ONOFF != '1') die("This component is disabled.");


Where is the hard code as the $db['keywords'] variable import the boonex config into the adsense mod which bypassing hardcoding anything

Please explain for me, I am sure I am missing something. Thanks

Hard-coding in AWMconfig.php still exists. You should get the configuration data from Dolphin inc/header.inc.php file.

Alex,

We have done the specified changes, and updated the name of the mod.

naming convention is not met
place your files to separate directory
add instructions how to move css file if user have not uni template
get rig of hadcoded config in AWMconfig.php