[theme-reviewers] get_template_part vs locate_template

[Chip Bennett]

So, I found the code you're talking
about<http://themes.svn.wordpress.org/wiredrive-classic/1.0.1/css/custom-css.php>
.

The correct way to do this:

1) Wrap that output in a function, e.g.:

function wiredrive-classic_custom_css() {
     include( get_template_directory() . '/css/custom-css.php' );
}


2) Enqueue the function (increase priority as needed):

add_action( 'wp_print_styles', 'wiredrive-classic_custom_css', 11 );


And that's it.

Chip

On Fri, Apr 22, 2011 at 11:34 AM, Curtis McHale <curtis at curtismchale.ca>wrote:

> I'm reviewing a theme and I've found uses of get_template_part where I
> believe that locate_template is the proper item. Specifically the code below
> loads the custom stylesheet.
>
> <?php get_template_part('css/custom-css')?>
>
> It was my understanding that in this case we should use:
>
> <?php locate_template( array( '/css/custom-css.css' ), 1); ?>
>
> and that get_template part was to call a file in the same directory like
> loop-main.php. Is there established best practice and if so what is it? Is
> the use above an okay way to call a custom stylesheet in to the theme? If
> not what way is currently advised?
>
> Curtis McHale
> PH: 604.751.3482
> http://www.curtismchale.ca
> Linkedin: http://www.linkedin.com/in/curtismchale
> Twitter: twitter.com/curtismchale
>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110422/d6f583c3/attachment.htm>