Bug 1384735 - RSS validation error "Missing entry element: updated"
Summary: RSS validation error "Missing entry element: updated"
Keywords:
Status: CLOSED DUPLICATE of bug 1379149
Alias: None
Product: Fedora
Classification: Fedora
Component: python-pelican
Version: 25
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Matthias Runge
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1379149
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-10-14 02:51 UTC by Viorel Tabara
Modified: 2016-10-14 11:46 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-10-14 11:46:01 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Viorel Tabara 2016-10-14 02:51:41 UTC
Description of problem:

The atom feed file:

    feeds/all.atom.xml

is missing the <updated> element under the <entry> section.


Version-Release number of selected component (if applicable):

python2-pelican-3.6.3-6.fc25.noarch 


How reproducible:

Always.


Steps to Reproduce:

1. Build the sample content:

      $ git clone https://github.com/getpelican/pelican
      $ cd pelican/samples
      $ pelican -s pelican.conf.py -t ../pelican/themes/notmyidea/ content

2. Inspect the atom feeds file 'feeds/all.atom.xml'.


Actual results:

The one and only <updated> element is located just before the first <entry>:

   omiday ~/work/omiday/pelican/samples $ xmllint --format output/feeds/all.atom.xml | egrep "^[[:space:]]*<(|/)(updated|entry)>"
   <updated>2013-11-17T23:29:00+01:00</updated>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>
   <entry>
   </entry>


Expected results:

Every <entry> section should have an <updated> element like so:

   omiday ~/work/omiday/pelican/samples $ xmllint --format output/feeds/all.atom.xml | egrep "^[[:space:]]*<(|/)(updated|entry)>"
   <updated>2013-11-17T23:29:00+01:00</updated>
   <entry>
      <updated>2012-11-30T00:00:00+01:00</updated>
   </entry>
   <entry>
      <updated>2012-03-02T14:01:01+01:00</updated>
   </entry>
   <entry>
      <updated>2012-02-29T00:00:00+01:00</updated>
   </entry>
   <entry>
      <updated>2012-02-29T00:00:00+01:00</updated>
   </entry>
   <entry>
      <updated>2011-02-17T00:00:00+01:00</updated>
   </entry>
   <entry>
      <updated>2011-02-17T00:00:00+01:00</updated>
   </entry>
   <entry>
      <updated>2011-02-17T00:00:00+01:00</updated>
   </entry>
   <entry>
      <updated>2013-11-17T23:29:00+01:00</updated>
   </entry>
   <entry>
      <updated>2010-10-20T10:14:00+02:00</updated>
   </entry>
   <entry>
      <updated>2010-10-15T20:30:00+02:00</updated>
   </entry>
   <entry>
      <updated>2010-03-14T00:00:00+01:00</updated>
   </entry>


Additional info:

The atom feed file is correctly generated under the following virtualenv:

   $ pip list
   beautifulsoup4 (4.5.1)
   blinker (1.4)
   bs4 (0.0.1)
   docutils (0.12)
   feedgenerator (1.9)
   Jinja2 (2.8)
   MarkupSafe (0.23)
   pelican (3.6.3)
   pip (8.1.2)
   Pygments (2.1.3)
   python-dateutil (2.5.3)
   pytz (2016.7)
   setuptools (28.3.0)
   six (1.10.0)
   Unidecode (0.4.19)
   wheel (0.30.0a0)

Since the review for the updated 'feedgenerator' is in works at bug 1379149 I'm 
filing is as a dependency.

Comment 1 Matthias Runge 2016-10-14 11:46:01 UTC
Thanks for the detailed report. To me, it looks like a duplicate of 1379149.

I wonder why this was not uncovered earlier though.

As a remark, I like the solution in upstream Django better than forking/renaming/removing not required bits from Django itself. iirc, the fix is included in Django-1.10, which is available in rawhide.

*** This bug has been marked as a duplicate of bug 1379149 ***


Note You need to log in before you can comment on or make changes to this bug.