Expand support for other fields
-
I’ve tried parsing an atom field with this plugin & it failed. When I looked at the code there were no support for pubDate & category fields.
(Also in my dev environment I got an error on $content variable which was not set to empty string. Adding$content = '';
beforeforeach
solved the issue.)Here’s what I’ve added to the FetchFeedPageable_call function to extend the support. If you could incorporate these in future plugin revisions that would be great.
case "pubDate": $date=$item->get_date(); $content .= '<div class="ItemDate">'. esc_html($date) .'</div>'; break;
and
case "category": $category = $item->get_category(); $content .= '<div class="ItemCategory">'.$category->get_label() .'</div>'; }
I would also recommend removing the hard
from the results, it’s best practices and folks can add the spacing/separators they need via CSS.Hope this helps.
https://www.ads-software.com/extend/plugins/fetch-feed-shortcode-pageable/
- The topic ‘Expand support for other fields’ is closed to new replies.