WordPress.org
Get WordPress

Make WordPress Core

Opened 19 years ago

Closed 17 years ago

#5789 closed defect (bug) (fixed)

feeds invalid / url not present -> prefer empty url instead of 'http://'

Reported by: neoxx Owned by:
Priority: normal Milestone: 2.8
Component: General Version: 2.0
Severity: normal Keywords: has-patch tested commit reporter-feedback dev-feedback
Cc: Focuses:

Description

greetz!

we got a validation error (check ​http://www.feedvalidator.org/) in our feeds, because logged in commenters without an url in their profile automatically hold '​http://'. - possible solutions: leave the field Website ($profileuser->user_url) empty, if no url is present. remove 'emtpy' urls from feeds.

berny

Attachments (2)

5789.diff​ (788 bytes ) - added by DD32 19 years ago.
5789.2.diff​ (1.4 KB ) - added by Denis-de-Bernardy 17 years ago.

Download all attachments as: .zip

Change History (10)

#1 @DD32
19 years ago

  • Milestone 2.6 → 2.0.12
  • Severity major → normal

Looking at the Comments feed: ​http://dd32.id.au/comments/feed/ i cant see any evidance of the issue.

It appears that under current trunk the URL the commentor gives never makes it to the feed.

I assume this is a 2.0-branch only issue?

#2 @DD32
19 years ago

  • Milestone 2.0.12 → 2.6

It was just pointed out by FFEMTcJ on IRC that it might not've been in the feed i was looking at.

Indeed, If i look at the Atom feed i see the problem.

@DD32
19 years ago

#3 @DD32
19 years ago

  • Keywords has-patch added
  • Milestone 2.6 → 2.5

#4 @DD32
18 years ago

Any movement? or wontfix worthy?

@Denis-de-Bernardy
17 years ago

#5 @Denis-de-Bernardy
17 years ago

  • Keywords tested dev-feedback added; feed profile user removed

updated the patch, but even without it, I failed to reproduce any of this. might be useful for old installs, though.

#6 @Denis-de-Bernardy
17 years ago

  • Milestone 2.9 → 2.8

#7 @Denis-de-Bernardy
17 years ago

  • Keywords commit reporter-feedback added

commit, invalid, wontfix?

#8 @westi
17 years ago

  • Resolution → fixed
  • Status new → closed

(In [11249]) Filter out blank urls for commentors. Fixes #5789.

Note: See TracTickets for help on using tickets.