Closed (fixed)
Project:
Text Link Ads Integration
Version:
4.7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Nov 2006 at 19:45 UTC
Updated:
21 Feb 2007 at 19:01 UTC
Hye guys. Great module. Thanks.
But one thing is missing (in another release): it just gives an option for 8 ad rows but it's possible to have 10 ads on page now. It's line #101 at current release:
$row_options = drupal_map_assoc(array(1, 2, 3, 4, 8));
but should be:
$row_options = drupal_map_assoc(array(1, 2, 3, 4, 8, 10));
Comments
Comment #1
mcurry commentedI was going to submit a patch for this, but ... there are no CVS files available for download via CVS? If you go to the CVS repo via the web, it shows all the project files as dead. What gives?
Comment #2
mcurry commenteddead files: http://cvs.drupal.org/viewcvs/drupal/contributions/modules/textlinkads/
Comment #3
kbahey commentedCommitted to 4.7.
Comment #4
(not verified) commented