Add filter to allow customising 'You must be logged in to...' template notice

[gawain]

Usability use case(s):

• Hyperlinking text to login page
• Adding HTML social login buttons
• etc...

[gawain]

If this looks like a worthwhile effort, I am happy to go through and add these filters in more locations where it makes sense. I just wanted to get a feel of whether this was a viable addition to the code base in the first place.

[alexvorn2]

why print and not echo?

[gawain]

echo returns an error:

PHP Parse error:  syntax error, unexpected 'echo' (T_ECHO)

[netweb]

Related #1883

[johnjamesjacoby]

Would rather keep filters out of templates, and build a proper set of error handler/message callbacks.

[johnjamesjacoby]

Replying to gawain:
If all you want to do is change strings, you can use custom po/mo's to do this without filters.

​http://codex.buddypress.org/developer/extending-buddypress/customizing-labels-messages-and-urls/

I don't like the idea of filtering strings, since they're already ran through several sets of functions and filters.

I'm going to close this, since keeping the strings in the templates is the most flexible solution.

[gawain]

No problem, thanks for looking at it jjj.

I do wonder though if (only) that one could be treated as a special case? My thinking here is adding 3rd party login functionality via a plugin that generates the HTML from a function call. An example ​here...

[johnjamesjacoby]

Would rather not introduce something temporary or incomplete. If you want to link to the login_url() (which has been requested before) you could patch this to include a link to login?

[johnjamesjacoby]

Circling back, it's really too bad this isn't easier for folks.

I can't imagine an easy solution, considering the range of strings available to us.

Maybe filter gettext and check the original $text?

[netweb]

Replying to johnjamesjacoby:

Would rather not introduce something temporary or incomplete. If you want to link to the login_url() (which has been requested before) you could patch this to include a link to login?

This was just added to w.org recently: ​https://meta.trac.wordpress.org/changeset/4747

<?php printf( __( 'You must be <a href="%s">logged in</a> to reply to this review.', 'wporg-forums' ), wp_login_url() ); ?>

3rd party social login plugins typically extend the default login form I think... The above would cover that would it not?

cc @johnjamesjacoby