Needs review
Project:
Openads
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Jan 2011 at 06:28 UTC
Updated:
8 Apr 2011 at 14:40 UTC
Подготовил версию для 7.x. В основу взял 6.x-1.0.
Тестировалась с OpenX 2.8.7, возможны глюки с другими версиями.
--------
Ready version 7.x. The basis took 6.x-1.0.
Tested for the OpenX 2.8.7, there may be bugs with other versions.
(Sorry for my bad english)
| Comment | File | Size | Author |
|---|---|---|---|
| openads-7.x-1.x-dev.tar_.gz | 7.28 KB | Anonym |
Comments
Comment #1
andypostJS code seem strange
.info file files-section should contain only list of module's files whick contain classes
uninstall code - suppose
db_query("DELETE FROM {variable} WHERE name LIKE 'openads_%'");is enough (only one query to DB).Please use coder module to clean-up code style
Comment #2
Anonym commentedandypost: Do you have other version OpenADS for 7.x?
Comment #3
jashughes commentedThis version does not have Publisher ID as an admin setting like the 6.x module version. Is there a way to specify the publisher ID for which to pull ads - even I have to hard code it? Thanks, Jason