Advertisement Management

[EXreaction]

Yes, I know the latest here is 1.0.4. It was submitted to mod validation almost a month ago.

What I gave you was instructions to fix it.

[bike749]

look like have to pay someone to install this for me.

[auggybendoggy]

Guys,
at the bottom of this forum is an ad is this the same mod their using?

Aug

[EXreaction]

Guys,
at the bottom of this forum is an ad is this the same mod their using?

Aug

No, but the same thing can be done.

[auggybendoggy]

Ok so what are they using for their ads?

but the real question for this is on installation: I've read and believe I'm ready to begin...
When it says to edit PROSILVER what if I don't use prosilver? I use Prosilver_se as my style theme.
Do I still need to edit prosilvers files or prosilver_se?

Aug

[Pardis]

Modification name: Advertisement Management
Author: EXreaction
Modification description: Adds an easy to use Advertisement Manager to a phpBB3 forum.
Modification version: 1.0.4
Tested on phpBB version: See below

Download file: Advertisement Management 1.0.4.zip
File size: 73030 Bytes

Modification overview page: View


Selected tags:

1. Category
   1. Add-Ons
2. Complexity
   1. Template Changes
   2. File Edits
3. Time
   1. ~ 5 Minutes
4. phpBB
   1. 3.0.2

Support for this modification needs to be asked within this topic. The phpBB Team is not responsible or required to give anyone support for this modification. By installing this MOD, you acknowledge that the phpBB Support Team or phpBB MODifications Team may not be able to provide support.

This MOD has only been tested by the phpBB MOD Team with the phpBB version listed in the topic. It may not work in any other versions of phpBB.

The attached mod in the first post is too old

The lastest release in lithiumstudios.org is 1.0.6 :

Code: Select all

http://www.lithiumstudios.org/forum/download/file.php?id=855&sid=1d41cc7f9be4750955c8602300c1b096

Please correct post

[RMcGirr83]

Once the 1.0.6 version has been validated, the download link will be updated to reflect this version.

When it says to edit PROSILVER what if I don't use prosilver? I use Prosilver_se as my style theme.
Do I still need to edit prosilvers files or prosilver_se?

Aug

You need to edit the html files for whichever style you are going to use. So to answer your question, yes you edit the Prosilver_se files and not the prosilver ones (Mod authors are required to include instructions in their mods for prosilver edits and some mod authors also provide instructions for subsilver2 which is usually found in a contrib folder in the mod download).

[Il Trova Inglese]

grathias! it's fantastic

[auggybendoggy]

Ok,
everything seems installed ok but may I ask what graphic size is this set up for? Perhaps there is no size limitation? Perhaps someone has a good size that works well they can offer as advice?

also,
Am I understanding this correctly? My guess is to make a HTML page and put all gracphics into a directory on the server. Then copy the html code into the box?

Aug

[auggybendoggy]

I've tried everything.

The edits look right, the ACP is good.

If I have NO code in the html code field will a empty banner show up?

I don't know what to put in the html code so I'm kinda stuck.

Any help would be appreciated.

Aug

[Paul]

MOD Updated to version 1.0.6
See first post for Download Link

[sysdev]

Excellent mod! Thank you for putting your time and effort into this.

I think I've found a bug (if it's not already a known issue):

In the ACP > Advertisement Management, edit an ad. In the "Ad Priority" field, enter "999" and submit the form. The following error is returned:

Code: Select all

General Error
SQL ERROR [ mysql4 ]

Out of range value adjusted for column 'ad_priority' at row 1 [1264]

SQL

UPDATE phpbb_ads SET ad_name = 'Ad number 1', ad_code = '[EDIT - ad code hidden]', ad_note = 'Ad number 1 note', ad_time = '1222012366', ad_time_end = 0, ad_views = 81, ad_clicks = 1, ad_priority = 999, ad_enabled = 1, all_forums = 1 WHERE ad_id = 7

BACKTRACE


FILE: includes/db/mysql.php
LINE: 158
CALL: dbal->sql_error()

FILE: includes/acp/acp_ads.php
LINE: 207
CALL: dbal_mysql->sql_query()

FILE: includes/functions_module.php
LINE: 471
CALL: acp_ads->main()

FILE: adm/index.php
LINE: 74
CALL: p_master->load_active()

Looks like a validation check is missing. It would also be useful to include an acceptable range in the field description, so that users know the acceptable values to enter into the field.

[EXreaction]

I see. I will put a check on that for the next version.

[sysdev]

After testing, it looks like the acceptable range of values for this field is "0" to "127". What is the algorithm for calculating the liklihood of the ad appearing in the given location? Would using a percentage be better?

Regardless, (since it's all relative to the other ads that are sharing the same space on the page,) it could also be useful to include some kind of report for each individual ad, (or for all enabled ads) showing the actual percentage of the liklihood that the ad would appear in each position that it could appear in. As more ads are enabled or disabled, the percentage shown in the report would change. That could be useful information for both administrators and advertisers. Just my 2 cents.

Again - thanks for a great mod!

[EXreaction]

To calculate the percentage of a chance of that advertisement appearing you'd need to add up the total of the ad priorities for what could be shown, then divide the what each has set for the priority by the total. So if say 5 different ads could be shown with 4 of them having 5 set for priority and 1 having 7, the chances for the 4 would be 18.5% and for the 1 at 7 would be 26%

Yes, I agree, it isn't too useful unless you understand how it works, but I can't think of a better way to do it right now...