Couple questions about banner vs. txt and banner size

[webdzr]

Im still testing before I go live with selling ads
and there are a few questions that I have.

If I set an ad location to allow banner images,
banner ads must use the full adspace is checked and set the adspace width and height,

Allow text links not being checked,

the purchacer of adspace is still able to create a textlink ad.

The same thing if Allow text links is checked and banner ad is not
they can still create a banner ad.

Also specifing the banner height and width seem to have no effect, the purchacer
can still upload any size image.

Now im testing in sandbox mode so im not sure if that has any effect on things
or is the script acting properly?


Also is there a way to have the Advertise tab in the menu that shows only for
members that are logged in?

[SMFHacks]

Will look into the top part thursday night.

For advertise button in SMF 2.0 you would need to modify

Sources/Subs.php
Find

Code: [Select]

// the [Ad Seller Pro] button
'adpro' => array(
'title' => $txt['txt_seller_admin_tab'],
'href' => $scripturl . '?action=adpro;sa=buyadspace',
'show' => ($modSettings['seller_show_advetise'] == 1 ? true : false),
'icon' => '',
),

Change to

Code: [Select]

// the [Ad Seller Pro] button
'adpro' => array(
'title' => $txt['txt_seller_admin_tab'],
'href' => $scripturl . '?action=adpro;sa=buyadspace',
'show' => ($modSettings['seller_show_advetise'] == 1 && !$user_info['is_guest']) ? true : false),
'icon' => '',
),



[webdzr]

Ok Thanks

[webdzr]

Will look into the top part thursday night.

For advertise button in SMF 2.0 you would need to modify

Sources/Subs.php
Find

Code: [Select]

// the [Ad Seller Pro] button
'adpro' => array(
'title' => $txt['txt_seller_admin_tab'],
'href' => $scripturl . '?action=adpro;sa=buyadspace',
'show' => ($modSettings['seller_show_advetise'] == 1 ? true : false),
'icon' => '',
),

Change to

Code: [Select]

// the [Ad Seller Pro] button
'adpro' => array(
'title' => $txt['txt_seller_admin_tab'],
'href' => $scripturl . '?action=adpro;sa=buyadspace',
'show' => ($modSettings['seller_show_advetise'] == 1 && !$user_info['is_guest']) ? true : false),
'icon' => '',
),



Just for reference if anybody else wants to use the above code there is an extra  )  right before  ? true : false), that
breaks the code.

It should look like this:

Code: [Select]

// the [Ad Seller Pro] button
'adpro' => array(
'title' => $txt['txt_seller_admin_tab'],
'href' => $scripturl . '?action=adpro;sa=buyadspace',
'show' => ($modSettings['seller_show_advetise'] == 1 && !$user_info['is_guest'] ? true : false),
'icon' => '',
),

Works perfect now!

[SMFHacks]

Posted an update that should fix checking for ad types and banner size

1.0.11
!Fixed issue with when buying ads that checking was not working for allowed ad types textlinks/banners.
!Fixed checking for banner width and height for buyer adspace

[webdzr]

Thank you so much for that update. It really helped me a lot

There is one thing though, i think the banner ad must use full adspace function isn't working.

[SMFHacks]

Will look into the fullspace issue for banners.

[amwebby]

Sorry to necro but someone has just posted a text ad and it is 100% width. Is there any way to restrict that and the amount of text they can put in the body?

[SMFHacks]

At the moment no but sounds like a good setting request.

[amwebby]

Then please consider it one. Probably the same size as banner ads. Not sure how you determine the body text restriction from that but I guess looking at how Google AdSense does it would be a good start