Comments on: Smarter Hooks http://ptahdunbar.com/wordpress/smarter-hooks/ Web Craftsman, WordPress hacker and Entrepreneur. Tue, 16 Feb 2010 19:17:03 +0000 hourly 1 By: Ozh http://ptahdunbar.com/wordpress/smarter-hooks/#comment-4458 Ozh Thu, 02 Jul 2009 12:19:27 +0000 http://ptahdunbar.com/?p=763#comment-4458 Yeah, was going to bitch about this :) You'd need to be able to do something like do_atomic('footer', 'after') But, all in all, I'm not sure this makes better code: theme_before_footer() and theme_after_header() might be more readable than do_atomic('footer', 'before'). I like code when it looks like natural language, sort of. Yeah, was going to bitch about this :)
You’d need to be able to do something like do_atomic(‘footer’, ‘after’)

But, all in all, I’m not sure this makes better code: theme_before_footer() and theme_after_header() might be more readable than do_atomic(‘footer’, ‘before’). I like code when it looks like natural language, sort of.

]]> By: Ptah Dunbar http://ptahdunbar.com/wordpress/smarter-hooks/#comment-4457 Ptah Dunbar Thu, 02 Jul 2009 05:30:48 +0000 http://ptahdunbar.com/?p=763#comment-4457 Good catch. <code>$tag</code> was <code>$location</code> before i pasted this code on my blog. updated. I get what your saying about the container element. <code>do_atomic</code> could have a second parameter where if used, could surround the main hook location with a container div element or so. Good catch. $tag was $location before i pasted this code on my blog. updated.

I get what your saying about the container element. do_atomic could have a second parameter where if used, could surround the main hook location with a container div element or so.

]]> By: Dan Cole http://ptahdunbar.com/wordpress/smarter-hooks/#comment-4455 Dan Cole Thu, 02 Jul 2009 03:23:24 +0000 http://ptahdunbar.com/?p=763#comment-4455 I like the idea a lot, it does make things simpler, but I do have one thing against it. I think theme developers started using before and after because they wrapped a div tag around the element location, instead of going with a container or parent element hook and then attaching each child element with a priority. (Sorry for being vague) How would you suggest Thematic or Hybrid use this? Because they don't have three hooks in a row, but rather a div tag wrapping the middle one. Should this function also add the div too? Maybe your next post will give some insight into this. BTW, check out: <code>if ( !isset($tag) ) return;</code>, <b>$tag</b> should be <b>$location</b>. I like the idea a lot, it does make things simpler, but I do have one thing against it. I think theme developers started using before and after because they wrapped a div tag around the element location, instead of going with a container or parent element hook and then attaching each child element with a priority. (Sorry for being vague) How would you suggest Thematic or Hybrid use this? Because they don’t have three hooks in a row, but rather a div tag wrapping the middle one. Should this function also add the div too? Maybe your next post will give some insight into this.

BTW, check out: if ( !isset($tag) ) return;, $tag should be $location.

]]> By: Small Potato http://ptahdunbar.com/wordpress/smarter-hooks/#comment-4449 Small Potato Wed, 01 Jul 2009 17:14:50 +0000 http://ptahdunbar.com/?p=763#comment-4449 if you keep publishing posts like this, i might have to subscribe. i don't code anymore, but i'm trying to keep up with the codes to work my designs around them. if you keep publishing posts like this, i might have to subscribe. i don’t code anymore, but i’m trying to keep up with the codes to work my designs around them.

]]>