CUNY Academic Commons

Alright so I did this assuming that we only wanted this extra message to come up if someone was coming from the "Account already activated" error screen, but it'd be easy to change it so that it came up for all failed login attempts.

Here's the changeset: https://github.com/cuny-academic-commons/cac/commit/bfdb172be20433800fdded0b061e13e1e46d2a04

What I did was add a GET variable to the redirect_to URL in the login form on that error page, and tested for it by hooking into the login_errors filter called in wp_login.php. It's a kind of hack-y solution, but the only one I could think of without messing with Wordpress Core. Let me know if I should change it, either to appear on all failed logins (in which case I think we'd have to change the language), or to also appear when someone fails to login from the form on the successful activation page (which I also think would warrant a change in the language).

Thanks for this work, Daniel! If it wouldn't be too much trouble, I would say that it would be great to add the following line to the failed login screen, regardless of where people are coming from:

"If you are unable to log in and have trouble using the "Forgot Password?" link above, please contact the CUNY Academic Commons team at commons@gc.cuny.edu for assistance"

Again, sorry for the delay on this one. Now the message shows up on every failed login attempt, not just ones from the account activation page. Here's the changeset: https://github.com/cuny-academic-commons/cac/commit/a59d1102424dd1c3b60f0304ef45e49a3dc8639f

I made the changes on the 1.8.x branch again - let me know if I should also add them to 1.7.x

Thanks!

Thank you, Daniel

I've made some minor changes to Daniel's commits and merged them into 1.7.x branch:
https://github.com/cuny-academic-commons/cac/commit/19e19e5d84e89dd0e8560a7ecd32f553ddc68328

Marking this as resolved.

Thanks Daniel!