Opened 15 years ago
Closed 9 years ago
#12480 closed defect (bug) (duplicate)
Moderated comments don't show blank links
Reported by: | donncha | Owned by: | chriscct7 |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 3.0 |
Component: | Comments | Keywords: | has-patch dev-feedback has-unit-tests |
Focuses: | Cc: |
Description
Comments caught in moderation won't show the blank links inserted into them unless you edit them.
A comment with html code like <a href='http://example.com/'></a> will not show that link on the moderation page.
Attachments (4)
Change History (15)
#2
@
15 years ago
- Component changed from General to Comments
- Milestone changed from Unassigned to Future Release
#4
@
15 years ago
- Cc bmbalex@… added
- Keywords has-patch added
If the links are empty (or contain only blanks spaces, tabs...) then inside of them it is inserted a blank string
but it needs some performing because I don't know how to detect
also I am not sure if I gave right priority to new filter in add_filter function
@
15 years ago
Replaces empty, whitespace or nbsp anchor text with the sanitized URL. Improves regex to be less greedy for non href attributes
#5
@
15 years ago
- Milestone changed from 3.0 to 3.1
This isn't a regression, and looks like it could use some unit tests or at least some more testing. Moving to 3.1.
#6
@
14 years ago
I believe there are other special entities related to blank space too, like      
I wasn't able to find now but I've already seen entities that generate different sized spaces, it had even a grath with sizes.
|". |
#7
@
14 years ago
- Keywords comments removed
- Milestone changed from Awaiting Triage to Future Release
#8
@
10 years ago
- Keywords dev-feedback added
Looks good to go. Verified still a "bug", and patch looks good. Core devs might want to review intent
#9
@
9 years ago
- Keywords needs-unit-tests added
- Owner set to chriscct7
- Status changed from new to assigned
@
9 years ago
Refresh for WordPress 4.4 with addition of unit tests + better WordPress coding standards support
Maybe Slashdot-style show a URL's domain in parentheses?