Thema: kaffeine epg patch2

[amin]

Keine Ahnung wie der patch heissen soll.
Ich zeige am besten mit Bild.


hxxp://paste.uni.cc/13819

[spoke1]

Sieht schick aus. Magst Du noch ein kurzes Howto für Newbies im Thread drunterlegen? Das wär ne richtig runde Sache. 

[amin]

Howto ? Was meinst du genau ?

//Edit
Wenn um patchen geht:
cd kaffeine/src/input/dvb/
patch -p0 < follow.diff

Hier noch kleine patch. Denke so wird die Liste etwas schneller gebaut.

Code: [Auswählen]

--- kevents.cpp~        2007-03-21 14:00:22.000000000 +0100
+++ kevents.cpp 2007-03-21 14:00:40.000000000 +0100
@@ -304,6 +304,8 @@
                                sids.clear();
                                for ( j=0; j<(int)tlist->events.count();
                                        desc = tlist->events.at(j);
+                                       if ( desc->title.isEmpty() )
+                                               continue;
                                        //if ( mode==1 && desc->tid>0x4f
                                        if ( sids.contains( desc->sid )
                                                continue;
@@ -333,8 +335,6 @@
                                        begin = KGlobal::locale()->forma
                                        duration = desc->duration.toStri
                                        title = desc->title;
-                                       if ( title.isEmpty() )
-                                               continue;


[mtron]

danke, sehr nützlich  habe wiedermal ein binary package für ubuntu vom aktuellen SVN - Snapshot von kaffeine 0.8.4 (revision 644978) mit deinem EPG patch gebaut.

Wird Zeit, dass du dich selbst auf kaffeine-devel registrierst 

[amin]

Wird Zeit, dass du dich selbst auf kaffeine-devel registrierst 

Lieber nicht mit meinem English.

[spoke1]

@ amin

Genau das mit dem Patchen war gemeint.

Das Englisch Problem kenn ich irgentwo her.  Man nutzt das einfach nach der Schule zu wenig. Daher gilt auch für mich: my english makes me nobody so fast after. 

[mtron]

Hallo amin!

Danke für deinen letzten patch, sehr nützlich. Einige Kleinigkeiten die mir noch aufgefallen sind und verbessert werden könnten:

- Vielleicht wäre es sinnvoller, wenn man die Funktion bei änlichen Events einsetzen könnte, eg. einen rmb Menü eintrag, oder einen Button der dann die folgenden Episoden listet

- Eine Intelligente Suche wäre besser, wie z.B. "SIMPSONS: title" "simpsons : a title" ...

Wäre schön wenn du diese Anregungen aufgreifst.

Cheers
mtron

[mtron]

Und nochwas: ich weiss nicht ob du die kaffeine-useres liste mitliesst, dort kam vor einigen Tagen folgender Bugreport rein:

Am Samstag, 31. März 2007 08:00 schrieb Peter Imhoff:
> Hi,
>
> maybe this problem is already known.
> But I noticed that the third entry for the 'current channel' list
> is often (always?) missing.
> Two entries are always shown when you switch from one channel to another. It seems to me
> as there might be an 'off by one'-error when adding
> the rest of the list, e.g. starting with 3 instead of 2?...
> Hope it helps...
>
> Kind regards,
> Peter

Also das wichtigste nochmals auf deutsch:

Die Population der EPG - Eventliste des momentanen Kanals scheint sich um einen Eintrag zu verschlucken, und zwar wird der dritte Eintrag anscheinend übergangen (ich kanns' leider nicht verifizieren, weil ich grad auf Osterurlaub weit weg von meinem Kaffeine bin )

Laut dem Bug - Reporter sind die ersten beiden Einträge ( Current & Next) - wie sie im OSD gezeigt werden - ok, aber es wird der dritte Eintrag verschluckt.

Wie gesagt, ich kann den Fehler momentan leider nicht reproduzieren, aber bin ab Donnerstag wieder daheim, dann kann ich mehr Info anbieten.

Cheers
mtron

[amin]

In der Tat , es wird der 3. EPG-Eintrag aus der EPG-Liste nicht gezeigt. 
Komische Fehler.
Nur das hat auch nichts mit meiner Änderungen zu tun.
Hier der Beweis.
 

Es Fehlt "16:10 Ein Mann für eine Saison (Fever Pitch)"
ps. Ich glaube , ich darf die kaffeine 0.8.2 bei mir nicht mehr löschen, sonst werde ich in beweis Not

[amin]

- Eine Intelligente Suche wäre besser, wie z.B. "SIMPSONS: title" "simpsons : a title" ...

Willst du nur nach Titel suchen, oder auch in EPG beschreibung ?

[amin]

Hier patch2+patch3 mit Suchfunktion.(ist noch beta)
 

ps. Keine Ahnung ob die Englische Menu-Beschriftung  so bleiben kann.

//EDIT
Patch nicht mehr gültig, neue ist unten.

[amin]

und hier noch kleine patch für dvbpanel.

http://pastebin.ca/455682

[mtron]

Amin, was soll ich sagen - genial, genau das hatte ich mir vorgestellt!  Danke.

Wird sogleich gebaut und getestet!

Edit: funktioniert wunderbar. die GUI - Umsetzung der epg suche gefällt mir sehr gut. Eigentlich könnte man jetzt die "next showings" in blau (unter dem EPG Text) ja wieder weglassen.

epg search: beim Text - Eingabefeld  "KEvents::epgSearch()" (Zeile 229) wäre noch gut, wenn das drücken der Eingabetaste dieselbe Funktion hat wie der EPG - Search Button (natürlich nur wenn was geschrieben wurde)

dvbpanel: ist auch eine gute idee. Was noch wirklich nützlich wäre ist im rechtsklick - menü neben "select icon" einen zweiter drop down  eintrag aufzunehmen (Edit Channel), der direkt auf die Kanaleigenschaften öffnet.  So hätte man eine schnelle Methode APIDS ect. zu ändern.

[amin]

Ich habe noch ein Fehler gefunden:
<link gelöscht>

Ja, ich denke auch, dass die "next showings" jetzt weg können.

Edit Channel ist gute idee.

ps. Wegen 3. Eintrag in EPG. Heute kann ich es nicht reproduzieren.

//EDIT
Ok, hier ist patch ohne "next showings".
http://pastebin.ca/456153

Was die Edit Channel angeht, muss ich schauen, wie es zu lösen ist.
Weil edit Funktion "private" ist.

[mtron]

Was noch wirklich nützlich wäre ist im rechtsklick - menü neben "select icon" einen zweiter drop down  eintrag aufzunehmen (Edit Channel), der direkt auf die Kanaleigenschaften öffnet.  So hätte man eine schnelle Methode APIDS ect. zu ändern.

so, hab jetzt auch mal ein wenig gebastelt... Hier ist der patch.

Code: [Auswählen]

--- kaffeine/src/input/dvb/dvbpanel.cpp 2007/03/28 23:12:15 647633
+++ kaffeine/src/input/dvb/dvbpanel.cpp 2007/04/26 18:00:34 658054
@@ -53,6 +53,7 @@
 #include "dvbevents.h"
 #include "kevents.h"
 #include "broadcasteditor.h"
+#include "channeleditor.h"
 
 #define CHANICONSIZE 28
 
@@ -303,6 +304,7 @@
 
  QPopupMenu *pop = new QPopupMenu();
  pop->insertItem( i18n("Select icon..."), 1 );
+ pop->insertItem( i18n("Edit..."), 2 );
  i = 0;
  i = pop->exec( QCursor::pos() );
  switch ( i ) {
@@ -329,10 +331,59 @@
  pix.save( dvbConfig->dvbConfigIconsDir+name, "PNG" );
  fillChannelList();
  break;
+ case 2:
+ if ( editChannel( name ) ) {
+ fillChannelList();
+ }
+ break;
+
  }
  delete pop;
 }
 
+bool DvbPanel::editChannel( QString &name )
+{
+ int j;
+ ChannelDesc *chan=0;
+ QStringList list;
+ QPixmap pix;
+
+ for ( j=0; j<(int)channels.count(); j++ ) {
+ chan = channels.at(j);
+ if ( chan->name==name ) {
+ j = -1;
+ list.append( chan->tp.source );
+ break;
+ }
+ }
+ if ( j==-1 ) {
+ ChannelEditor dlg( list, false, chan, &channels, mainWidget );
+ int ret = dlg.exec();
+ if ( ret==ChannelEditor::Accepted ) {
+ pix.load( dvbConfig->dvbConfigIconsDir+chan->name );
+ if ( !pix.isNull() )
+ chan->pix = pix;
+ else {
+ if ( chan->type==1 ) {
+ if ( chan->fta )
+ pix = tvcPix;
+ else
+ pix = tvPix;
+ }
+ else {
+ if ( chan->fta )
+ pix = racPix;
+ else
+ pix = raPix;
+ }
+ }
+ return true;
+ }
+ }
+ return false;
+}
+
+
 
 
 void DvbPanel::getTargets( QStringList &uiNames, QStringList &iconNames, QStringList &targetNames )
@@ -969,7 +1020,7 @@
  getChannelList();
  rtp = new Ts2Rtp();
  rtp->setSocket( dvbConfig->broadcastAddress, dvbConfig->broadcastPort, dvbConfig->senderPort );
- cleaner = new Cleaner( dvbConfig->shiftDir );
+ cleaner = new Cleaner( dvbConfig->shiftDir, dvbConfig->recordDir );
  split->setSizes( dvbConfig->splitSizes );
 }
 
@@ -997,7 +1048,7 @@
  if ( ret==DvbConfigDialog::Rejected )
  return;
  rtp->setSocket( dvbConfig->broadcastAddress, dvbConfig->broadcastPort, dvbConfig->senderPort );
- cleaner->setPath( dvbConfig->shiftDir );
+ cleaner->setPaths( dvbConfig->shiftDir, dvbConfig->recordDir );
 }
EDIT: hftom (kaffeine Maintainer) hat mir gerade geschreiben, dass dieser Patch bereits im aktuellen SVN aufgenommen wurde