Tvheadend

Hi Andy,
I will send you the full output tonight, but I am pretty sure that these were the only two <title> tags.

In this case (and probably in some other cases), if no language is provided, then the default grabber language should be applied ("fr" here). Shouldn't that be the case for all the "tv_grab_<language_code>" default XMLTV grabbers ?

Another approach would be to have an "other" or "default" lang code in TvHeadend, which would be used when the client (web for me) requests an unlisted language.

Of course if needed we will update the tv_grab_fr script to force this "lang" attribute.

Sorry for my mistake in your first name. I thought Adam but typed Andy. It would be so nice to allow the edition of the posts during a few minutes!

I agree that something better could probably be done to work around the XMLTV scripts not providing language codes for the default languages. And in your specific case sure you can assume that the first bit after tv_grab_** is the language code.

But what about tv_grab_uk../tv_grab_nz.. UK/NZ are not language codes they are country codes, sure I can assume they mean something (en in this case probably?) but really if a script is providing multi-lang support it should either explicitly define the default language somewhere or should provide a language for each string.

And those 2 are just the first 2 that come to mind immediately, I'm sure there are others.

However it's possible that I could assume that if no language code is provided TVH should use its own default language code (where configured). That would overcome the problem, but would require users to correctly set the config etc..

Adam

You are right: if no language code is provided, TVH should use its own default language code, if there is not any other <title> attribute for this language.
But let us wait and check tonight if there is any other place in the XMLTV file where the default language is indicated.
I can also act on the source and ask them to update the grabber so that it always fills the language attribute.

I just looked at the XMLTV DTD and I could not find any default language indication.
Another possibility, perhaps the cleanest, would be to configure this default language in the "EPG Grabber" tab.

We could also use the XmltvWorldDomination page to get a list of the default language values for the default XMLTV grabbers, which could be a constant in TvHeadend.

Here is the beginning of my XMLTV file. I don't post the end of the file because it only consists in <programme> elements.

<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE tv SYSTEM "xmltv.dtd">

<tv source-info-url="http://telepoche.guidetele.com/" source-data-url="http://te
lepoche.guidetele.com/" generator-info-name="XMLTV" generator-info-url="http://x
mltv.org/">
  <channel id="C1.telepoche.com">
    <display-name>TF1</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/tf1.gif" />
  </channel>
  <channel id="C2.telepoche.com">
    <display-name>France 2</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/france2.gif" />
  </channel>
  <channel id="C3.telepoche.com">
    <display-name>France 3</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/france3.gif" />
  </channel>
  <channel id="C4.telepoche.com">
    <display-name>Canal +</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/canalplus.gif" />
  </channel>
  <channel id="C5.telepoche.com">
    <display-name>Arte</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/arte.gif" />
  </channel>
  <channel id="C6.telepoche.com">
    <display-name>M6</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/m6_logo.gif" />
  </channel>
  <channel id="C7.telepoche.com">
    <display-name>France 5</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/france5.gif" />
  </channel>
  <channel id="C122.telepoche.com">
    <display-name>i-télé</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/i_tele.gif" />
  </channel>
  <channel id="C167.telepoche.com">
    <display-name>Direct 8</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/new-logo.png" />
  </channel>
  <channel id="C168.telepoche.com">
    <display-name>NT1</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/nt1.gif" />
  </channel>
  <channel id="C169.telepoche.com">
    <display-name>NRJ 12</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/nrj_12.gif" />
  </channel>
  <channel id="C170.telepoche.com">
    <display-name>Public Sénat - LCP AN</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/parlement.gif" />
  </channel>
  <channel id="C193.telepoche.com">
    <display-name>Direct Star</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/direct_star.jpg" />
  </channel>
  <channel id="C194.telepoche.com">
    <display-name>BFM TV</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/bfm_tv.jpg" />
  </channel>
  <channel id="C195.telepoche.com">
    <display-name>Gulli</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/gulli.png" />
  </channel>
  <channel id="C28.telepoche.com">
    <display-name>France 4</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/france4.gif" />
  </channel>
  <channel id="C38.telepoche.com">
    <display-name>W9</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/w9.png" />
  </channel>
  <channel id="C49.telepoche.com">
    <display-name>France Ô</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/franceo.gif" />
  </channel>
  <channel id="C9.telepoche.com">
    <display-name>TMC</display-name>
    <icon src="http://telepoche.guidetele.com/medias/chaines/new_tmc.png" />
  </channel>
  <programme start="20121028035000 +0100" stop="20121028044500 +0100" channel="C1.telepoche.com">
    <title>Patrimoine immatériel : Chef d'oeuvre de l'humanité</title>
    <sub-title>Jouer le kutiyattam avec le maître</sub-title>
    <credits>
      <director>Jérôme Segur</director>
    </credits>
    <date>2010</date>
    <category lang="fr">série/feuilleton</category>
    <category lang="fr">documentaire</category>
    <country lang="fr">France</country>
    <country>FR</country>
  </programme>
  <programme start="20121028044500 +0100" stop="20121028050000 +0100" channel="C1.telepoche.com">
    <title>Musique</title>
    <category lang="fr">musique</category>
    <category lang="fr">variétés</category>
    <country lang="fr">France</country>
    <country>FR</country>
    <video>
      <colour>yes</colour>
    </video>
  </programme>
  <programme start="20121028050000 +0100" stop="20121028052500 +0100" channel="C1.telepoche.com">
    <title>Histoires naturelles</title>
    <sub-title>Fées et gens en Brocéliande</sub-title>
    <category lang="fr">documentaire</category>
    <category lang="fr">découverte</category>
  </programme>
  [...]
</tv>

Yeah looks like a bad mix of using lang="" and not. While it may be possible to get the upstream devs to update the script and that will solve the problem, I think that it might also be worth my adding the mod I suggested.

I will look into doing this.

Adam