According to http://tools.ietf.org/html/rfc4287#section-4.2.1 the atom:author tag is a Person construct. It can (as http://tools.ietf.org/html/rfc4287#section-3.2) has 3 children. Something like this is valid:

        <author>
                <name>John Doe</name>
                <email>johndoe@example.com</email>
        </author>

(both in feed source or each entry).

Currently Aggregator does not parse author tag, nor its children.

A consequence of this bug is if we have this:

        <summary>some data...</summary>
        <author>
                <name>John Doe</name>
                <email>johndoe@example.com</email>
        </author>

Then after the parsing, $feed->items[0]['summary'] contains erronous data:

        some data...
                John Doe</name>
                johndoe@example.com

It is because the parser does not process these tags.

Comments

jcisio’s picture

Status: Active » Needs work
StatusFileSize
new2.15 KB

This is the first try. I should work, however I don't understand why 'author' is NULL. As I don't have more time for this, I just upload this patch and fix my project with a workaround.

Note: this patch however fixes the 'summary' field.

jcisio’s picture

Issue summary: View changes

add a consequence

Crisz’s picture

Issue summary: View changes
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: aggregator.patch, failed testing.

ParisLiakos’s picture

Version: 8.0.x-dev » 7.x-dev

i think now we switched parsing to zend, this should be fixed

jcisio’s picture

StatusFileSize
new1.21 KB
new3.49 KB

Ok then patches for Drupal 7.

jcisio’s picture

Status: Needs work » Needs review

The last submitted patch, 5: 1369408-aggregator-author-5-test-only.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 5: 1369408-aggregator-author-5.patch, failed testing.

Status: Needs work » Needs review
jcisio’s picture

It's weird that we has tests failed but only in RemoveFeedItemTestCase. I have different fails locally, so retest to check.

Status: Needs review » Needs work

The last submitted patch, 5: 1369408-aggregator-author-5.patch, failed testing.

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.