Initial conversion was performed using the deadwood project, with some manual cleanup. The entire code base needs to be audited, and functionality needs to be broken out into smaller include files where applicable.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | ad_text.zip | 2.5 KB | sersim |
| #6 | ad_text.module.patch | 818 bytes | sersim |
Comments
Comment #1
alexandreracine commentedYou told me to report that here :)
---
I just installed the ad module with simply ad and ad_text.
I fill the fields, clic submit and get the "You must specify a valid Destination URL." error. The problem is that, this field is not there.
Good day to you.
---
Reference (duplicate) http://drupal.org/node/339724
Comment #2
alexandreracine commentedYou told me to report this here :
When clicking on publish-ad (?q=node/add/ad) you'll see the publish menu with the ad item, and with the submenus of ad, like in my case see below:
# Advertisement
* image advertisement (?q=node/add/ad/image)
* text advertisement (?q=node/add/ad/text)
Clicking on the submenus gives this error : "Fatal error: require_once() [function.require]: Failed opening required 'sites/all/modules/ad/node.pages.inc' (include_path='.:/usr/share/php:/usr/share/pear') in /var/www/drupal/montrealsalsa/includes/menu.inc on line 346"
Reference (duplicate) http://drupal.org/node/339725
Comment #3
xaler commentedSame problem over here..
In HTML Ads, first I am able to save, and then edit so I can add HTML for the Ad.
In Text Ads, I am not able to save because of this error.
Fabian
Comment #4
sersim commentedTry the patches I've posted here: http://drupal.org/node/294029#comment-1126143
Comment #5
sersim commentedThis patch fixes the following issue: "You must specify a valid Destination URL" message appears even if the URL is valid.
I've posted the zip file for people which are not familiar with patches. It contains the updated ad_text.module which you should place into sites/all/modules/ad/text
Comment #6
sersim commentedComment #7
jeremy commentedThis module is now functional in my testing, please test again with the latest code from CVS.