backdrop-contrib / aggregator

A port of the Drupal 7 Aggregator module to Backdrop.
GNU General Public License v2.0
1 stars 2 forks source link

XML views render the site's HTML as well as the XML #11

Open sillygwailo opened 2 years ago

sillygwailo commented 2 years ago

Without another example to show it, my site's aggregator demonstrates this bug: See https://richard.eriksson.ca/aggregator/opml and https://richard.eriksson.ca/aggregator/rss as examples.

sillygwailo commented 2 years ago

This may be as simple as adding the Backdrop equivalent of drupal_exit(); (which is to say, backdrop_exit();) at the end of the theme functions. I have that in the code for my personal site, but I'm going to give it time to marinate before pushing to the module.

See https://drupal.stackexchange.com/questions/4631/how-do-i-return-the-output-from-a-callback-function-unthemed

laryn commented 2 years ago

This may be a comparable issue in case it's helpful to look over:

(If memory serves, adding a delivery callback prevents Layouts from adding markup to the output.)