Textpattern CMS support forum
You are not logged in. Register | Login | Help
- Topics: Active | Unanswered
Re: Combine comments and articles in a single stream
jdueck wrote #283375:
I had to install and enable XSL on my server.
I should have mentioned it, sorry.
<txp:recent_comments break="" sort=''FIELD(discussid, <txp:variable name="dscids" />) DESC'' limit=''<txp:variable name="dscount" />'' form="comments_form_with_timestamps" />...…I get empty output. But if I manually enter the values like this:
<txp:recent_comments break="" sort=''FIELD(discussid, 313,311,310,309) DESC'' limit=''4'' form="comments_form_with_timestamps" />...I get the expected output. It’s as if
recent_commentsobjects to the use oftxp:variablewithin its attribute values.
I guess it’s a quotes affair, try with one single quote. It’s a bit strange that your second example works, however.
Edit: sorry, I goofed, it must be {dscount?} (and not {dscount}) inside the first etc_query tag.
Last edited by etc (2014-08-31 10:37:57)
Offline
Re: Combine comments and articles in a single stream
Thanks, I had already found and fixed the dscount? typo when I noticed the value was being surrounded by tags in the output. I think the issue was ultimately Textpattern’s pickiness about how to do quotes for attribute values when nesting tags.
Anyhow, it’s working now! I’m experimenting with typographic treatment at thelocalyarn.com/blogtest (link will expire at some point). My next step will probably be to do the same thing for my RSS feed. (I’ve been talking about the reasons behind doing all this on Twitter.) Many thanks for showing me so far down the correct path, I would never have found the solution on my own.
Offline
Re: Combine comments and articles in a single stream
Interesting sidenote, I am really enjoying the flexibility of etc_query. As part of this, I found I could not use the txp:author_email tag within my comments form in this context (it returned empty output). This was needed to slightly alter a few things if the current comment was one of mine, so I used etc_query to roll my own:
<txp:etc_query data='SELECT email FROM `txp_users` WHERE name=
(SELECT AuthorID FROM `textpattern` WHERE ID=
(SELECT parentid FROM `txp_discuss` WHERE discussid=<txp:comment_id />
))' name="etc_author_email" />Offline
Re: Combine comments and articles in a single stream
jdueck wrote #283383:
Interesting sidenote, I am really enjoying the flexibility of
etc_query. As part of this, I found I could not use thetxp:author_emailtag within my comments form in this context (it returned empty output).
Glad you enjoy it. I think you can gain one db query, since article ids are populated by <txp:recent_comments />:
<txp:etc_query data='SELECT email FROM `txp_users` WHERE name=
(SELECT AuthorID FROM `textpattern` WHERE ID=<txp:article_id />)' name="etc_author_email" />Offline
Re: Combine comments and articles in a single stream
I’m implementing this in my RSS feed, and have overcome several small obstacles but am up against one last one related to etc_query that I can’t seem to figure out.
You can see my efforts so far at http://thelocalyarn.com/feed (temporary). You can see that the entry elements are being sorted by the values contained in their child published elements.
My problem is that etc_query appears to be stripping <![CDATA[ from my items when processing the XSLT. My article form for the RSS feed is as follows:
<entry>
<author>
<name><txp:author link="0" /></name>
</author>
<published><txp:posted format="iso8601" /></published>
<updated><txp:posted format="iso8601" /></updated>
<title type="html"><txp:title no_widow="0" /></title>
<link rel="alternate" type="text/html" href="<txp:permlink />" />
<id>tag:thelocalyarn.com,<txp:posted format="%Y-%m-%d" />:<txp:etc_query data='SELECT uid FROM `textpattern` WHERE ID=<txp:article_id />'>{uid?}</txp:etc_query></id>
<content type="html"><![CDATA[
<txp:body />
]]></content>
<summary type="html"><![CDATA[
<txp:excerpt />
]]></summary>
</entry>But as you can see the feed that gets served has the opening CDATA tag stripped out, and the now-useless closing CDATA tag is escaped. After doing some extra-credit homework I thought perhaps I could get it to work by adding cdata-section-elements (or output-cdata-elements) to the XSLT:
<txp:etc_query
data='<body>
<txp:if_variable name="artids" value=""><txp:else />
<txp:article_custom id=''<txp:variable name="artids" />'' form="rss_article" />
</txp:if_variable>
<txp:if_variable name="dscids" value=""><txp:else />
<txp:recent_comments break="" sort=''FIELD(discussid, <txp:variable name="dscids" />) DESC'' limit=''<txp:variable name="dscount" />'' form="rss_comment" />
</txp:if_variable>
</body>'>
<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
<xsl:output method="html" encoding="utf-8" omit-xml-declaration="yes" cdata-section-elements="content summary" />
<xsl:template match="body">
<xsl:for-each select="entry">
<xsl:sort select="published" order="descending" />
<xsl:copy-of select="." />
</xsl:for-each>
</xsl:template>
</xsl:stylesheet>
</txp:etc_query>…but this has no effect. Is there a way to get etc_query to leave my cdata alone? (I have verified that it’s etc_query by manually generating the same output using txp:article_custom.)
PS. it is also apparently changing <link href="..." /> to <link href="..."> (note lack of trailing slash) but it doesn’t appear that this will affect the feed’s validation.
Offline
Offline
Re: Combine comments and articles in a single stream
Ah, that must be it, thanks. Although now instead of passing the <![CDATA[ through unmodified, it actually strips out the cdata tags and escapes all the HTML inside the element, which is another way of getting there (‘there’ meaning valid XML).
Inspired by your suggestion I also changed xsl:output to method="xml" which fixed the link tag markup that actually was causing me all kinds of other problems I didn’t realize.
Last edited by jdueck (2014-09-21 23:51:25)
Offline
Offline
Re: Combine comments and articles in a single stream
etc wrote #284018:
I haven’t played much with xsl, but everything seems to work fine with
cdata-section-elements="content summary"(thanks for info): test. Might it be something else that intervenes before the feed is served?
Nope that actually does work. I had removed cdata-section-elements while debugging; putting it back in did the trick, in combination with the other changes.
Offline