Hi,
I would like to hear your opinion on this matter, let me explain better what I mean.
Right now, when language negotiation in enabled, the links of the feed items are prefixed with the current language, which for drupal means the current interface language.
This could have sense for page output, but since feeds don't have any User Interface, I think that links in feeds should really be prefixed by the actual content language.
To achieve that I made the changes in the attached patch, what do you think about this? I am still making up my mind, but I am almost convinced that my observation makes a lot of sense.
There is still the case of language neutral content which deserves attention, I would say that the default language would be a good choice as a prefix of language neutral feed item link. Could that be handled by the url() function?
Thanks,
Antonio
| Comment | File | Size | Author |
|---|---|---|---|
| feeds_link_use_content_language_prefix_v0.patch | 1.31 KB | ao2 |
Comments
Comment #1
dpearcefl commentedAre you interested in pursuing this in Drupal v8? A change like this won't make it into D6 or D7.
Sounds reasonable though.
Comment #2
ao2 commentedWell, the patch attached in #1 does not work on feed displays from Views, it just affects the old rss.xml generation, but I guess we can move the question to 8.x-dev, just to see what people think about the idea.
I don't think people would care much about this issue tho, it was just an observation about rss.xml on my very first experiments with Drupal.
Maybe the path prefix can even be dropped altogether in feed links so language negotiation happens only when the actual browser opens the link?
Comment #3
dpearcefl commentedComment #4
xjm(Merging "node system" and "node.module" components for 8.x; disregard.)
Comment #18
smustgrave commentedThank you for sharing your idea for improving Drupal.
We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
Comment #19
acbramley commented~14 years since any activity on this one, I think it's safe to close out.