#107: RSS invalid
-------------------------+--------------------------------------------------
Reporter: tbrooks | Owner: jcaffaro
Type: defect | Status: accepted
Priority: minor | Milestone:
Component: WebSearch | Version:
Keywords: RSS inspire |
-------------------------+--------------------------------------------------
Changes (by jcaffaro):
* owner: => jcaffaro
* status: new => accepted
Comment:
Some comments about the errors raised by the validator:
{{{
line 22, column 0: channel contains more than one opensearch:itemsPerPage
<opensearch:itemsPerPage>25</opensearch:itemsPerPage>
line 28, column 9: Undefined channel element: url
<url type="application/rss+xml" indexOffset="1"
rel="results" templ ...
^
In addition, interoperability with the widest range of feed readers could
be improved by implementing the following recommendations.
line 16, column 39: Email address is missing real name
<webMaster>[email protected]</webMaster>
^
line 18, column 95: Self reference doesn't match document location
... ln=en&p=find+a+brooks+and+a+mele" />
^
line 29, column 162: Unregistered link relationship: search
... scription+xml" title="Content Search" />
^
line 31, column 8: Avoid Text Input
<textInput>
}}}
* '''Line 22''': Fixed by a forthcoming patch.
* '''Line 28''': Fixed by a forthcoming patch.
* Next points are just recommendations from the validator:
* '''Line 16''': In that case we simply don't have the information to
display the name of the administrator. Shall it be added to invenio.conf?
Do we want to name a contact admin? Shall it be approximated from the
admin email (for eg. by splitting around dot before the @?) just for
compliance? Is it ''so'' important?
* '''Line 18''': False positive. As mentioned in the help, the
validator does not check for equivalence of the document (it apparently
just checks the URL to assess equivalence of the documents, and in this
case URL parameters have been reorded, but indeed the link points to the
same "document").
* '''Line 19''': in that case the ''search'' value for the ''rel''
attribute is made necessary to support the OpenSearch "protocol". Though
''search'' is currently not a valid relationship, there are efforts from
the OpenSearch community to get it accepted in the
[http://www.iana.org/assignments/link-relations/link-relations.xhtml IANA
list]. Given the number of sites already using it, the already built-in
support from major browsers for OpenSearch and the inclusion of this
relationship in the [http://dev.w3.org/html5/spec/links.html#link-type-
search HTML 5 draft], there are little chances that this point becomes a
problem.
* '''Line 31''': as <textInput> is part of the RSS specs, we should not
fear too much about that one. But it is true (and sad) that if there is
not support for that tag in any known reader, it might go away from the
feed at some point...
PS: I have tested with 4 different RSS readers, and none of them exhibited
any issue with this feed (even without fixing two first points, and it
worked even in Thunderbird, which Victor found to have some issues in the
past with the feed). It would be interesting to know which reader was
used, in order perform further tests and check if the remaining points
highlighted by the validator would still be problematic for this reader.
Given that two first points will be fixed, I would suggest to close this
task upon integration of the aforementioned patches.
--
Ticket URL: <http://www.indico-software.org/invenio/ticket/107#comment:1>
Invenio <http://invenio-software.org>
