Drupal 6 module review

brmassa - September 20, 2008 - 16:00
Project:Openads
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:active
Description

Guys,

me and Chris Nutting (OpenX developer) did some revamping on the D6's version of this. There are some changes on this code:
* SEVERAL performance enhancements
* Admin page interface cleaner and much more simple
* Only Javascript method available (OpenX dev team reported that almost all users use JS)
* JS code only added when the page really have a ad
* Roles feature is gone since its possible to set the exactly same filter on block settings.

Note: we opted for naming the module as really "openx" instead the "openads". You might install/test it without uninstalling the current module.

regards,

massa

AttachmentSize
openx-d6.zip11.33 KB

#1

jaydub - December 16, 2008 - 12:06

Out of curiousity what are the performance enhancements?

Dropping the other invocation methods is a mistake I think since OpenX supplies them why not support them? Removing functionality that is already in place makes no sense. The business I work for uses the XML-RPC method which is certainly faster than Javascript even with SPC (mentioned by the OpenX people in their own forum).

Was the roles feature dropped because of this:

http://forum.openx.org/index.php?showtopic=503421470

 
 

Drupal is a registered trademark of Dries Buytaert.