Needs review
Project:
Advanced Views RSS Feed
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
27 Feb 2015 at 02:03 UTC
Updated:
17 Feb 2022 at 21:25 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jmvidal commentedComment #2
jmvidal commentedComment #3
mauricio.tollin commentedI fixed it changing the pattern used to replace the urls, in views_rss.module > function views_rss_rewrite_relative_paths
$pattern = '/(src|href)=(\'|")' . preg_quote($base_path, '/') . '/';It works on simple relative paths, like src="/. But not works in paths with space.
And I not tested other cases.
Patch attached.
Comment #4
colling2 commentedLooks like it was the fix for issue 1960216 that broke this.
Comment #5
TGEink commentedI tried this patch, but still not working. Our URLs from this module come out:
http://thegreeneconomy.com/%3Ca%20href%3D%22http%3A//thegreeneconomy.com...Instead of
http://thegreeneconomy.com/content/defining-economics-and-secular-stagna...
It's a serious problem. Is there ANY way to fix this?
Comment #6
lily.yan commented#3 worked for me. thanks a lot.
Comment #7
nuncium commentedWith me, #3 method doesn't work that well.
On multilingual website it creates
www.mywebsite.com/enen/ instead of
www.mywebsite.com/en/
Any hints why?
Thanks!
Comment #8
liam morlandComment #9
liam morlandComment #10
marucci commentedBased on this solution, I created a module for those cannot edit their module
Comment #11
damienmckennaI think we need to add some minimal test coverage to make sure it works the way we expect.