[wp-trac] [WordPress Trac] #32096: Twenty Fifteen: author-bio.php clashes with "bio" username
WordPress Trac
noreply at wordpress.org
Tue Mar 19 16:41:17 UTC 2019
#32096: Twenty Fifteen: author-bio.php clashes with "bio" username
-------------------------------------------------+-------------------------
Reporter: lordlod | Owner: (none)
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: Awaiting
| Review
Component: Bundled Theme | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch dev-feedback bulk- | Focuses:
reopened |
-------------------------------------------------+-------------------------
Changes (by ianbelanger):
* keywords: has-patch, dev-feedback, bulk-reopened => has-patch dev-
feedback bulk-reopened
* version: 4.0 => trunk
Comment:
IMO, this issue breaks a page which is available for anyone to view and
can be crawled by the SE bots. So even though it might be an "edge" case,
it is still something that should be fixed. I'm not sure that I agree that
the current fix is that impactful either. It only effects template loading
for this particular issue. Also, I'm not sure that there is a better way
to handle it.
All that being said, I tested the most recent patch and it failed to
apply, so I updated the patch against trunk. While doing so, I noticed
another issue with the previous fix. If the author named `bio` did not
have any published posts, then the previous fix did not work, so I updated
the patch to check against the authors post number.
32096.4.diff addresses all of these things:
* Refreshed patch against current trunk
* Better targeted function names to their particular theme
* Updated @since for both themes
* Added conditional check for authors published post number
I understand that there are differing opinions on whether this should be
fixed, so if there are any objections to fixing this, please state your
case. Otherwise, in the interest of closing out old tickets, I will
recommend that this be committed into the bundled themes.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/32096#comment:17>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list