Download & Extend
Ad Flash » Issues

Port to Drupal 6

Posted by Manuel Garcia on January 27, 2009 at 11:26am
Project:Ad Flash
Version:5.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Not sure what the status is on this, will the d6 version of ad be incorporating this addon module, or will this module be ported to Drupal 6 itself?

Login or register to post comments

Comments

#1

Posted by artis.bajars on February 4, 2009 at 10:08am
Status:active» needs review

Here's my attempt at porting ad_flash for Drupal 6, in case anyone else needs this

Changes made:

  • Added hook_theme registration
  • Removed ad_flash_menu() hook
  • Made minor adjustments to database queries in ad_flash_display_ad() and ad_flash_global_settings_submit()
  • Changed adapi implementation at case:"type"
  • Fixed clickTAG output in theme functions to get redirects working

Tested with Drupal 6.9 and latest dev release of ad.module (2009-Jan-30).
The .js files from http://www.adobe.com/devnet/activecontent/articles/devletter.html are not included in the attachment. Follow the installation instructions to add those. Feel free to test and comment.

AttachmentSize
ad_flash.zip 15.08 KB

#2

Posted by chriscohen on February 4, 2009 at 10:12am

Many thanks indeed for this. I intend to test this soon with a view to creating a snapshot for D6.

#3

Posted by Jānis Bebrītis on February 4, 2009 at 12:53pm

this is truly great work, converting module for active release.

#4

Posted by Manuel Garcia on February 4, 2009 at 1:06pm

Many thanks artis.bajars!! I'll stay tuned for the 6.x-1.x-dev for testing

#5

Posted by OFF on February 27, 2009 at 8:23pm

Thanks!

Module work great for me.

Ad 6.x-1.1

Drupal 6.10

#6

Posted by Pandelon on April 8, 2009 at 2:31pm

Unfortunately dosen't work for me.

" Unable to locate flash" - when a preview the ad.

Ad 6.x-1.1

Drupal 6.10

#7

Posted by introfini on April 14, 2009 at 11:08pm

it stoped working in the last beta6...

introfini

#8

Posted by nrauni on April 16, 2009 at 6:35am

It didn't work to me..

I'm using the ad

; $Id: ad.info,v 1.1.2.2.2.1 2008/08/11 21:15:26 jeremy Exp $

with the drupal 6.10.

UPDATE: I needed to change the function called adaccess by ad_permission.

my the banner didn't show the banner, looking the source code it show:

<div class="content">
<script type='text/javascript' src='http://localhost/test/sites/all/modules/ad/serve.php?q=1&t=0&u=node/79'></script>  </div>
</div>

accessing http://localhost/test/sites/all/modules/ad/serve.php?q=1&t=0&u=node/79

It show disconfigured codes, like this

.....\x3c/script\x3e\n ');

I tryed to change the html.. but...

Do you have any ideia?

thanks

#9

Posted by introfini on April 18, 2009 at 1:30am

working version in the attachment

introfini

AttachmentSize
ad_flash.zip 16.33 KB

#10

Posted by chriscohen on April 20, 2009 at 9:42am

I've put together a Drupal 6 dev version using introfini's attachment above. Thanks for all the effort on this.

#11

Posted by Alex Andrascu on January 7, 2010 at 4:09pm
Status:needs review» closed (fixed)
nobody click here