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.

CommentFileSizeAuthor
#6 ad_text.zip2.5 KBsersim
#6 ad_text.module.patch818 bytessersim

Comments

alexandreracine’s picture

You 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

alexandreracine’s picture

You 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

xaler’s picture

Same 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

sersim’s picture

Try the patches I've posted here: http://drupal.org/node/294029#comment-1126143

sersim’s picture

This 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

sersim’s picture

StatusFileSize
new818 bytes
new2.5 KB
jeremy’s picture

Status: Active » Fixed

This module is now functional in my testing, please test again with the latest code from CVS.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.