I propose a new feature to the module. It is to make the "Advertisements" content type fields available when creating a view with the "node" base table, so you don't need to create a view of type Advertisement" to display ad-related info. The patch doesn't break the previous way, it just adds another implementation. It simplifies the views creation process, as #985302: relationship to node or cck? shows the present way is a bit hard to grasp.

CommentFileSizeAuthor
#1 ad_views_declaration_views_data.patch10.91 KB_tychris
ad_views_declaration_views_data.patch10.91 KB_tychris

Comments

_tychris’s picture

_tychris commented
StatusFileSize
new ad_views_declaration_views_data.patch10.91 KB

Sorry, I remove developement comments.

samhassell’s picture

samhassell commented
Status: Needs review » Needs work

Hey I got this patch working but theres a line at the top that says the 

'path' => drupal_get_path('module', 'ad_views')  . '/handlers',

but the handlers are all in the root directory.

changing that line to:

'path' => drupal_get_path('module', 'ad_views'),

fixes the problem.

pindaman’s picture

pindaman commented

Great patch thanks.
I have patched into ad_views-6.x-1.x-dev

if you want it take it from here.