• Resolved Dharmaraj

    (@dharmarajiyer)


    Dear P?r:

    Thank you so much for this wonderful plugin!

    I’d like to download the history as a CSV file so that I can examine it in Excel. I’ve done this by writing a function that reads Simple History’s RSS feed.

    The problem is that the feed had only 10 items in it. To work around this, I changed the code in “simple-history/index.php” (version 1.3.6), line 444, from

    $arr_items = simple_history_get_items_array("items=10");

    to

    $arr_items = simple_history_get_items_array("items=all");

    Is there a way to do what I want without having to make this change to the plugin code? Perhaps the number of items in the RSS feed could be set by an option, or by a filter?

    Thanks again for the great plugin!
    Dharmaraj

    https://www.ads-software.com/plugins/simple-history/

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author P?r Thernstr?m

    (@eskapism)

    Sure thing.

    The just released 1.3.7 has a new filter called simple_history/rss_feed_args that you can use to modify the arguments used by the rss feed.

    Thanks for the optimizations!

    Please consider to also add the possibility to shut the rss feed completely down. I have to disable it after every update, because I don’t want anyone guessing the url see all usernames and actions.

    So, one option would be to simply add an option to the plugins options-page.

    Alternatively, I just added another filter:

    ...
    function output_rss() {
    
    	$rss_show = true;
    	$rss_show = apply_filters("simple_history/rss_feed_show", $rss_show);
    	if( !$rss_show ) wp_die( 'Nothing here.' );
    ...

    In action:

    // Don't display any rss items for simple history
    add_filter( 'simple_history/rss_feed_show', 'show' );
    function show( $args ) {
    	return false;
    }

    It would be really great if you included any of these options into the plugin – I think it’s a serious security flaw to have the feed public.

    Thanks again.

    Plugin Author P?r Thernstr?m

    (@eskapism)

    @te-online: ok, check the latest update 1.3.8, it has your suggestion. let me know if it works ok.

    Great!

    Thank you very much for your quick reaction, P?r. ??

    Thread Starter Dharmaraj

    (@dharmarajiyer)

    @p?r
    Thank you for the super-fast response! I’ll try it out.

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘More RSS feed items?’ is closed to new replies.