RSS feed for comments

[mike]

Create an additional RSS feed for individual picture comments.

[dime]

This should be done, and removing current feed that appears in meta tags with cruft-free URLs turned on, on single-picture pages, which shows nothing but one picture, and only can create confusion and more used bandwith.

[sidtheduck]

If I find some time, I may be able to get to this for 1.0, but I'll put it under 1.1 for the time being. Should be easy enough to manage.

[sidtheduck]

Code to add picture comment support

[sidtheduck]

I added some quick code to the plog-rss.php file for single picture comment support. You can test out the patch to r569 that I've attached to this ticket. It fully validates for my test gallery rss feed.

Basically I'm looking for some feedback. Currently, it still shows the picture (just like a standard rss feed for an album or collection), but then adds the comments below it (in reverse order). If there are no comments, it just shows the single image by itself.

What I'm wondering is if we want to have the image in there or just the comments by themselves. If the image stays, do we want the description, etc. or just the image by itself. If there are no comments, should it display "No comments" much like regular front-end comments?

I also need to add code to not display the RSS feed link or button and not to display the RSS comments if comments are turned off by the user, but I wanted to get this out there to get feedback on how to better implement it.

[kimparsell]

Replying to sidtheduck:

I added some quick code to the plog-rss.php file for single picture comment support. You can test out the patch to r569 that I've attached to this ticket. It fully validates for my test gallery rss feed.

I checked and the comments feed validates on my test gallery as well.

Basically I'm looking for some feedback. Currently, it still shows the picture (just like a standard rss feed for an album or collection), but then adds the comments below it (in reverse order). If there are no comments, it just shows the single image by itself.

What I'm wondering is if we want to have the image in there or just the comments by themselves. If the image stays, do we want the description, etc. or just the image by itself. If there are no comments, should it display "No comments" much like regular front-end comments?

I would include the image in the feed, along with the description (if any). With regard to whether to display "no comments", my concern there is that we would display it initially before any comments had been added, but how will you remove it once a comment has been posted for that image? It could be confusing to see "no comments" and a list of comments directly under it.

I also need to add code to not display the RSS feed link or button and not to display the RSS comments if comments are turned off by the user, but I wanted to get this out there to get feedback on how to better implement it.

With this current patch, the rss image is not being displayed in the footer of the page, only in the address bar. If implemented, the rss feed image will need to show in the footer as well if comments are turned on. You will also need to add code to ensure that any comments in the moderation queue are not shown in the feed until they are approved.

[sidtheduck]

Replying to kimparsell:

I would include the image in the feed, along with the description (if any). With regard to whether to display "no comments", my concern there is that we would display it initially before any comments had been added, but how will you remove it once a comment has been posted for that image? It could be confusing to see "no comments" and a list of comments directly under it.

Replying to dime:

... which shows nothing but one picture, and only can create confusion and more used bandwith.

I was mostly bringing up the "No comments" or no image in the feed if there are no comments because of dime's post above saying that it could cause confusion and waste bandwidth. I would agree with the confusion part if there are no comments and it is just a single image posted. Should I have it only display the image if comments are present and if not, just post a blank RSS feed?

Replying to kimparsell:

With this current patch, the rss image is not being displayed in the footer of the page, only in the address bar. If implemented, the rss feed image will need to show in the footer as well if comments are turned on. You will also need to add code to ensure that any comments in the moderation queue are not shown in the feed until they are approved.

Yep, this is what I was talking about. I need to add the code to show the rss feed button and link if comments are turned on as well as put safeguards in the rss feed code itself to only display if comments are turned on (in case someone has a direct link to the rss feed even after comments have been turned off).

Comments in the moderation queue ahould already taken care of with the current code as well as it uses the same comment loop as the front-end gallery.

[sidtheduck]

The RSS code is fixed for pictures with no comments. All that's left is the RSS feed link and RSS feed button.

[kimparsell]

In rev590, mages with no comments are not showing the RSS link in the footer of the page, nor in the address bar, so I am unable to subscribe to them. Images that do have at least 1 approved comment show the RSS link in the footer and address bar.

Are we going to try and get this included in the next release?

[sidtheduck]

What else needs to be implemented / included? What you described is exactly the code I included to make it work properly (AFAIK). Is there a different functionality you were looking for, kimparsell?

[kimparsell]

Then I guess it's working as it is intended to: images with at least 1 comment will show the feed button at the bottom of the page and in the address bar. Images with no comments will show neither of them.

I was under the assumption that you could subscribe to the comments for an image regardless of whether it already had a comment or not. After re-reading the conversation above, I see that my assumption was incorrect.

I think we can close this ticket now.