<div class="gmail_quote">On Fri, Feb 18, 2011 at 12:49 PM, Robert McNamara <span dir="ltr"><<a href="mailto:robert.mcnamara@gmail.com">robert.mcnamara@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br><br><div class="gmail_quote"><div class="im">On Fri, Feb 18, 2011 at 9:45 AM, Eric Smith <span dir="ltr"><<a href="mailto:erictsmith@gmail.com" target="_blank">erictsmith@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div></div></div><div class="gmail_quote"><div><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><br><br></div></div></blockquote></div></div>
<div><br>I have created ticket 9597 <a href="http://code.mythtv.org/trac/ticket/9597" target="_blank">http://code.mythtv.org/trac/ticket/9597</a> with a
patch to support random fanart, coverart, and banners. It currently
only supports the front end mythvideo directory settings. At first
glance, the ability to fetch a random file from a storage group did not
seem obvious so I decided to stick with the old directory settings for
now. I will look further into that if the devs don't want to continue
to support the old settings.<br></div></div></blockquote><div><br></div></div><div>Hi Eric,</div><div><br></div><div>Thanks for the effort on the patch-- I've added my comments to the ticket. In order to get committed, it's going to need to support storage groups (and likely *only* storage groups is acceptable as we're likely to be SG only by .25). Also need support for all the artwork imagetypes, and to use variables consistent with everywhere else in the frontend (ie, we never refer to it as "bannerart"). Even after all of that is said and done, it will also need a stamp of approval from Stuart Morgan as well as the code is ultimately his.</div>
<div><br></div><div>Robert <br></div></div></blockquote><div><br><br>Thanks for the comments Robert. I assumed storage groups would likely be a requirement. I'll take another shot at it this weekend and see what I can come up with.<br>
</div></div>