[mythtv] Mythfilldatabase & xmltv,

[Laurent ARNAL]

Daniel Kristjansson a écrit :
> On Fri, 2009-05-08 at 13:33 +0200, Laurent ARNAL wrote:
>   
>> Hi Everybody,
>> =
>>     
>
>   
>> I always find that configuring TV Card in MyhTV is difficult for non US =
>>     
>
>   
>> citizen.
>> What I'm usually do is:
>> =
>>     
>
>   
>>    1/ Do a first frequency scan to find the channel.
>>    2/ Go to mythweb, and fill the xmltvid column by hand for each entry =
>>     
>
>   
>> in Channel table.
>>    3/ Launch mythfilldatabase to fill missing informations (icons, =
>>     
>
>   
>> programs, etc...).
>> =
>>     
>
>   
>> Perhaps miss something somewhere, but what hurts is Step 2 because you =
>>     
>
>   
>> don't have a link between channel frequency and xmltv informations.
>> The only link is the channel name that could be the same, or near the sam=
>>     
> e.
>   
>> =
>>     
>
>   
>> What I propose is to modify void ChannelData::handleChannels.
>> The goal is when we are handling a channel coming from the xmltv parser, =
>>     
>
>   
>> to verify if we don't already have an entry in the Channel table without =
>>     
>
>   
>> xmltvid but with the same name.
>> If so, we can do an update of this entry, instead of doing an insert.
>> =
>>     
>
>   
>> I Create a path for this purpose, see ticket #6531.
>>     
>
> You didn't miss anything step 2 is the part that hurts everywhere.
> Fuzzy matching the channel names should help a good deal.
>
> PS I understand there is a language barrier, but if you could make the
> small changes needed to the code in patch to make it look more like the
> existing code that would help it get applied more quickly. Otherwise
> the person applying the patch needs to make it adhere to the coding
> conventions before it can be applied.
>
> -- Daniel
>
> _______________________________________________
> mythtv-dev mailing list
> mythtv-dev at mythtv.org
> http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev
>   
Will review the patch tomorrow about coding standard.
Already remove the tab, and review the spacing.
Need to re-read the coding rule on the wiki, because I'm not sure what I 
miss :)

Laurent.