Hello,
I needed to place this in a panel pane rather then inside a block and the block module is off. I figured I would go ahead and share with anyone that might want to use! Will post the patch.

Comments

evanjenkins’s picture

StatusFileSize
new1.95 KB

Here is the patch!

bleen’s picture

Issue tags: +Needs tests
+++ b/plugins/content_types/dfp_pane.incundefined
@@ -0,0 +1,62 @@
+  ¶
...
+  ¶

white spaces

... is there a way we can write a simpletest for this? If you add "test_dependencies[] = panels" then testbot will know to include panels when running tests

tom friedhof’s picture

StatusFileSize
new2.11 KB

Here is the same patch from #1 with the exception of explicitly declaring what plugins this module implements. I was getting Fatal Errors when ctools read the contexts plugins into memory because it was looking for a Class in a module not enabled on the site I'm working on. (Context module)

The context integration should probably be broken out into a separate module, since that module isn't necessarily needed to use this module, especially with this panels integration. That's a topic for another discussion.

bleen’s picture

Issue summary: View changes

If anyone has used the patch in #3 and can post an "it works" review that would be great. Other than some minor white space issues, I'd be willing to bring this in...

  • bleen18 committed 7b641cb on 7.x-1.x authored by evanjenkins
    Issue #2012092 by evanjenkins, tom friedhof: Custom Content Type - (...
bleen’s picture

Fixed (except I changed the of the form element to 'dfp_ads' instead of 'cehck_ads'

bleen’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.