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 |
Jump to:
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
| Attachment | Size |
|---|---|
| openx-d6.zip | 11.33 KB |

#1
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