productiondate in xml

[Knaudel]

hallo,

habe Probleme mit dem Argument
<productiondate>X </productiondate>
in der xml-Datei

laut movieinfo.cpp handelt es sich um das Jahr ab 1900
mit 50 < X < 200

bis 1950 -> X=50 -> 1950 richtig!
1951 -> X=51 -> 3851 ?
2006 -> X=106 -> 3906 ?

mach ich einen Fehler oder stimmt hier was nicht?
besten Dank an alle

[seife]

Ich verstehe nicht, was du uns sagen willst :-)

Was willst du denn erreichen?

[Knaudel]

Hallo seife,

ich möchte im TS-Filmarchiv mit ? das Produktionsjahr angezeigt bekommen.

also z.B. USA/GB 2006

dazu gebe ich in der xml-Datei unter
<productioncountry>USA/GB</productioncountry> und
<productiondate>106</productiondate> ein.

bekomme dann aber USA/GB 3906 angezeigt.

besten Dank

[seife]

Ah, jetzt, ja! Eine Insel! :-)

Das Problem ist, daß das einmal in CMovieInfo::loadMovieInfo gemacht wird, aber später in CMovieInfo::showMovieInfo praktisch nochmal, nur ohne Abfrage. Ein (schmutziger) workaround ist, das so zu machen:

Code: Alles auswählen

Index: gui/movieinfo.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/movieinfo.cpp,v
retrieving revision 1.9
diff -u -p -r1.9 movieinfo.cpp
--- a/gui/movieinfo.cpp 7 Sep 2007 00:46:27 -0000       1.9
+++ b/gui/movieinfo.cpp 9 Nov 2007 12:18:41 -0000
@@ -442,8 +442,11 @@ void CMovieInfo::showMovieInfo(MI_MOVIE_
         {
                print_buffer += "\n";
                print_buffer += movie_info.productionCountry;
-               print_buffer += " ";
-               snprintf(date_char, 12,"%4d",movie_info.productionDate + 1900);
+               print_buffer += " ";
+               if (movie_info.productionDate > 50 && movie_info.productionDate < 200)
+                       snprintf(date_char, 12,"%4d",movie_info.productionDate + 1900);
+               else
+                       snprintf(date_char, 12,"%4d",movie_info.productionDate);
                print_buffer += date_char;
      }

Wo kommt denn das productionDate her? Kommt das aus dem EPG oder muss man das händisch eintragen? Wenn es aus dem EPG kommt, dann sollte man das so lösen, daß es von Anfang an im gewünschten Format vorliegt (also als absolutes Jahr, "2001") und gar nie anders verwendet wird. Dann würde in der loadMovieInfo() die Konversion für die Abwärtskompatibilität drinbleiben, und in der showMovieInfo() würde man das "+1900" einfach ganz rausnehmen.
Ich habe mir das jetzt aber nicht so genau angeschaut. Movieplayer/browser/... machen schon alleine wegen dem gräßlichen Codingstyle immer extremste Schmerzen beim Betrachten :-)

[seife]

Ich habe es mir (trotz der Schmerzen :-) nochmal angeschaut. Das productionDate muss man ja sowieso von Hand eingeben, das kommt nie aus dem EPG. Ich würde also in der CMovieInfo::showMovieInfo einfach das +1900 wegmachen.

[Knaudel]

Hallo seife,

hoffe die Schmerzen lassen langsam wieder nach

habe es auch so verstanden das man <productiondate> von Hand eingeben muss,

wusste nur nicht ob es am Code liegt oder an mir

besten Dank

[Mucki]

Mit der Änderung die seife vorgeschlagen hat funktioniert's.

... in der CMovieInfo::showMovieInfo einfach das +1900 wegmachen.

Könnte jemand die movieinfo.cpp im CSV dahingehend ändern?

[Gaucho316]

Wenn das hier jemand im CVS fixt, sollte er auch die Kommentare in den Zeilen 584 und 980 anpassen. Dort steht nämlich "years since 1900". Das stimmt dann ja nicht mehr.

[Gaucho316]

Ich habe die angesprochenen Änderungen mal in ein Diff gepackt und hoffe, dass mal wieder jemand Lust und Zeit hat, diesen und einige andere Patches ins CVS einzuchecken.

Link entfernt, da Patch im CVS