clef/clef-wordpress

Use `site_url` instead of `home_url` to configure logout hook

Closed this issue · 2 comments

Is there any reason not to do this here?

This would allow us to support wordpress configurations where wordpress is installed not in the apache root, i.e. if wordpress was at http://example.com/wordpress instead of just http://example.com.

(Ignore this if I'm muddying instead of clarifying...)

I think the answer is no. But I'm not clear how switching will make a difference. For which of the two non-root-install cases is Clef logout currently unsupported?

  1. running a root install but loading core files from a subdirectory — i.e., blog still loads from example.com
  2. running a subdirectory install — i.e., blog only loads from example.com/wordpress

I thought that it was already working for both.

home_url() => designed to link any front-end link, like pages, posts, archives.
site_url() => designed to link real files urls like wp-login.php, admin-ajax.php, admin-post.php