[mythtv-users] Recommendation page in mythweb

[Anthony Giggins]

On 9 October 2013 03:53, Karl Newman <siliconfiend at gmail.com> wrote:

> On Tue, Oct 8, 2013 at 10:23 AM, Karl Newman <siliconfiend at gmail.com>wrote:
>
>> Thanks to Rich Freeman's work in Gentoo I was able to easily upgrade to
>> 0.27 last night after holding out on 0.26. So far, so good (except for
>> having to disable the fanart for readability and kid-friendliness reasons).
>> I noticed the new "Recommended Programs" option in mythweb and thought I'd
>> check it out. So I set up an account on
>> http://myth-recommendations.aws.af.cm/ (*) and entered the resulting API
>> key. However, it throws a SQL error when I try to access the "Recommended
>> Programs" page.
>>
>> Backtrace here: http://pastebin.com/wxY8zy6H  The suspicious part is the
>> SELECT statement, where there's a AND () just before the ORDER BY clause.
>> In other words, there's an AND with an empty condition.
>>
>> Here's the commit which adds this "Recommended Programs" feature to
>> mythweb:
>> https://github.com/MythTV/mythweb/commit/15080f2e2f80ab78fe5982405829f0ff735b337e
>>
>> Has anyone tried this feature yet? I'll dig into the code a little bit
>> and see if I can find what's causing that empty AND condition.
>>
>> * That website has a couple broken links. A few examples--the "Register"
>> and "Genres" links in the header are broken until you add a trailing slash.
>>
>> Thanks,
>>
>> Karl
>>
>
> I solved this. The problem was that I had not rated any shows so my
> recommendations were empty. This could be handled more gracefully, though.
> I put a patch here: http://code.mythtv.org/trac/ticket/11898
>
> Karl
>
> How did you add recommendations? I've tried adding them from mythweb
recorded and via http://myth-recommendations.aws.af.cm/
and I still get the same error as your backtrace.

Cheers,

Anthony
<http://myth-recommendations.aws.af.cm/>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mythtv.org/pipermail/mythtv-users/attachments/20131010/98dc5740/attachment.html>