Setting the iTunes Category to a field with an ampersand outputs additional "amp;" text after the HTML entity.
For example, using "Religion & Spirituality/Christianity" text as the category outputs the following XML.
<itunes:category text="Religion &amp;amp; Spirituality"><itunes:category text="Christianity"/></itunes:category>
It would seem that the first category should be "Religion & Spirituality" in the feed output, without the extra "amp;amp;" appended.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 3029877-2.patch | 1.01 KB | rodrigoaguilera |
Comments
Comment #2
rodrigoaguileraI experienced the same.
The text is already htmlencoded and the podcast module encodes it again and the the render system for the attributes.
Here is my suggestions for changing the code
Comment #3
geoffreyr commented+1 for the patch in #2.
Comment #4
shane birley commentedPatch applied for testing.
Comment #5
shane birley commentedI haven't run into any problems so far. Thumbs up!
Comment #6
shane birley commentedHowever, I will mention that this issue is making it difficult to test this fully as the
itunes:categoryfield isn't being rendered properly.Comment #7
fozzieblue commentedI really appreciate the podcast module. I've applied three other patches successfully, but the multiple top-level categories patch is failing.
I'm using:
This is the first time I've added a patch on my own. I hope posting here was the correct next move.
Comment #8
shane birley commented@fozzieblue,
Can you take a peek at this issue? Essentially, I would like to confirm that you have copied the theme files from the module's templates folder to the active theme's directory?
Comment #9
fozzieblue commented@shane-birley
Thanks for replying! Yes, I've copied all 5 of the templates from the podcast module templates folder into our custom theme's templates > views folder (themes > custom > npl > templates > views). I did that before I added the three patches that were successful.
Comment #11
rodrigoaguileraApplied the patch at #2
Thank you for testing!