I can see that you have changed the way the plugin returns the output if you have the variable $display, set to false.
This version always formats the display according to the template set in the options, and always returns a html string as the output.
I want to display most_viewed post differently on my page and therefore I looped throw the output myself accordingly.
Now I can’t anymore, and I can’t specify different templates for different places on my site. This change breaks my page…
Why have you moved the if( ! $display ) return $most_viewed down to after the templating? Maybe you can let me specify the variable $views_options[‘most_viewed_template’] as a input when I call the function get_most_viewed() or maybe you can let me specify a “plz just return the database result”-flag. A last suggestion is that you put a filter or a action in there, so that I can use the output before the template.
Thx WpGuru