jsumners / jsumners/feedparser

atom ids are identified as links

Open
#416 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

auto-migrated
Dominant language
Python
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

What steps will reproduce the problem?

>>> import feedparser
>>> atomfd = 
feedparser.parse('http://bldgblog.blogspot.com/feeds/posts/default')
>>> atomfd.entries[0].guid
u'tag:blogger.com,1999:blog-8663346.post-6589476380961282518'
>>> atomfd.entries[0].guidislink     
True
>>> rssfeed = 
feedparser.parse('http://bldgblog.blogspot.com/feeds/posts/default?alt=rss')
>>> rssfeed.entries[0].guid
u'tag:blogger.com,1999:blog-8663346.post-6589476380961282518'
>>> rssfeed.entries[0].guidislink
False

What is the expected output? What do you see instead?

I expect guidislink to return False for the atom feed.

What version of the product are you using? On what operating system?

Python 2.7.5 (default, Oct 15 2013, 11:31:09) 
[GCC 4.2.1 20070719 ] on openbsd5
>>> feedparser.__version__
'5.1.3'

Original issue reported on code.google.com by ted.unan...@gmail.com on 3 Nov 2013 at 7:39

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the reported difference with feedparser.parse using the Atom and RSS feed URLs shown, then inspect how guid and guidislink are derived. Done means the Atom example reports guidislink as False while the RSS behavior remains unchanged; add or update a regression test for both cases if the project’s test location can be identified.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.