[theme-reviewers] get_option('home') bloginfo('url') andhome_url()

[Philip M. Hofer (Frumph)]

its echo home_url() and echo site_url() btw, because they are returned info while bloginfo('url') bloginfo('wpurl') are already echo'd

  ----- Original Message ----- 
  From: yulian yordanov 
  To: theme-reviewers at lists.wordpress.org 
  Sent: Wednesday, September 15, 2010 10:03 AM
  Subject: Re: [theme-reviewers] get_option('home') bloginfo('url') andhome_url()


  ooOK :) Thanks


  On Wed, Sep 15, 2010 at 7:53 PM, Andrew Nacin <wp at andrewnacin.com> wrote:

    2010/9/15 Edward Caissie <edward.caissie at gmail.com>

      So I would look at using home_url() as a recommendation over get_option("home") as well, not required ... but definitely good practice.


    I would strongly recommend that avoiding get_option('home') should be *required*. If a theme does it, tell them not to. I'm not concerned about rejecting said theme as much as making it crystal clear that while home_url() is recommended over both get_bloginfo('url') and get_option('home'), that there are different reasons for each.


    get_bloginfo('url') simply calls home_url(). They can be treated as synonymous and the guidelines might as well say "home_url() or get_bloginfo('url')", without stating a preference. (Various other get_bloginfo() calls are synonymous with independent functions.)


    But the same cannot be said for get_option('home'). The issue here being that get_option('home') simply cannot properly support SSL situations. Thus get_option('home') is a bad idea.

    _______________________________________________
    theme-reviewers mailing list
    theme-reviewers at lists.wordpress.org
    http://lists.wordpress.org/mailman/listinfo/theme-reviewers





  -- 
  Юлиян Йорданов

  Yulian Yordanov
  post-scriptum.info




------------------------------------------------------------------------------


  _______________________________________________
  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/20100915/8e9ddf10/attachment.htm>