Update in the documentation of Changing Action Links
-
https://www.jfarthing.com/development/theme-my-login/changing-action-links/
The example needs a ; in the end of the following line in order not to break the site immediately upon activation:
$url = 'YOUR REGISTRATION URL HERE'
For a second improvement, the documentation could change that this only changes where the action links point to, and that to have there be a proper page where the new link points to something else needs to be done, and a link to that documentation (which I’m trying to find :)).
For a third improvement, I’d make the example use curly braces “{}” with the if. The WordPress coding standards don’t insist on this, but many others do due to the amount of time wasted on hunting bugs that were caused by editing code that didn’t originally contain these. Even experts fall for that sometimes, and the people using copy-paste solutions from the documentation are mainly not experts, so two curly braces in a copy-paste can save a lot of headache around the world.
- The topic ‘Update in the documentation of Changing Action Links’ is closed to new replies.