DVB-Cube <<< Das deutsche PC und DVB-Forum >>>
Diverse DVB Software für verschiedene Karten => # Linux DVB (vdr, kaffeine, ect.) => # Kaffeine => Thema gestartet von: amin am 21 März, 2007, 12:23
-
Keine Ahnung wie der patch heissen soll.
Ich zeige am besten mit Bild. ;)
(http://img485.imageshack.us/img485/3391/followlx7.png) (http://imageshack.us)
hxxp://paste.uni.cc/13819
-
Sieht schick aus. Magst Du noch ein kurzes Howto für Newbies im Thread drunterlegen? Das wär ne richtig runde Sache. ;)
-
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.
--- 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;
-
danke, sehr nützlich 8) habe wiedermal ein binary package für ubuntu vom aktuellen SVN - Snapshot von kaffeine 0.8.4 (revision 644978) mit deinem EPG patch gebaut (http://www.dvbcube.org/index.php?topic=13543.msg75593#msg75593).
Wird Zeit, dass du dich selbst auf kaffeine-devel registrierst ;)
-
Wird Zeit, dass du dich selbst auf kaffeine-devel registrierst ;)
Lieber nicht mit meinem English. ;)
-
@ 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. ;) ;)
-
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
-
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
-
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. ;)
(http://img177.imagevenue.com/loc562/th_31872_beweis_122_562lo.jpg) (http://img177.imagevenue.com/img.php?image=31872_beweis_122_562lo.jpg)
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 ;)
-
- 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 ?
-
Hier patch2+patch3 mit Suchfunktion.(ist noch beta)
(http://img11.imagevenue.com/loc366/th_13123_search_122_366lo.jpg) (http://img11.imagevenue.com/img.php?image=13123_search_122_366lo.jpg)
ps. Keine Ahnung ob die Englische Menu-Beschriftung so bleiben kann.
//EDIT
Patch nicht mehr gültig, neue ist unten.
-
und hier noch kleine patch für dvbpanel.
(http://img179.imagevenue.com/loc436/th_14438_source_122_436lo.jpg) (http://img179.imagevenue.com/img.php?image=14438_source_122_436lo.jpg)
http://pastebin.ca/455682
-
Amin, was soll ich sagen - genial, genau das hatte ich mir vorgestellt! ;D 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.
-
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.
-
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.
--- 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 :D
-
EDIT: hftom (kaffeine Maintainer) hat mir gerade geschreiben, dass dieser Patch bereits im aktuellen SVN aufgenommen wurde :D
Supi :)
-
Hi amin!
Danke für deine Arbeit! Ich hoffe dir wird es nicht zu lästig wenn ich immer verbesserungs - Vorschläge unterbreite ;)
Ich hab grade deine korrektur zum epg-search patch (http://www.dvbcube.org/index.php?topic=20143.msg78901#msg78901) getestet, funktioniert mit aktuellem svn super. Die epg Suche wird jetzt auch bei Eingabetaste aktiviert :)
bez. Gui - Design der Suche: im aktuellen SVN ist ebenso eine Filtermöglichkeit für Kanalnamen in der linken Seitenleiste eingebaut.
(http://mtrons.googlepages.com/kaffeine-dvbpanel.jpg)
hftom meint, dass es die Gui Usability erhöhen würde wenn die epg - suche vielleicht ähnlich aussieht.
hier ein mockup von mir:
EDIT: hat sich erübrigt, amin hat's schon besser gemacht...
Wie oben im mockup wäre neben "Titles only" noch eine zweite Filtermöglichkeit ob paytv kanäle mitgesucht werden sollen (wird in der Kanallist ja mit z.b. "TVC" gekennzeichnet) interessant.
Vielleicht hast du ja Lust und Zeit die Anregungen aufzugreifen.
-
Hi mtron.
(http://img408.imageshack.us/img408/3492/epg2sl8.jpg) (http://imageshack.us)
Was sagst dazu ?
Beschreibung für CheckBox "TV" usw. und was da als ToolTip steht ist aber schon deine Arbeit. ;)
So wie du es vorgeschlagen hast, geht nicht.
Hier diff gegen aktuelle svn.
http://pastebin.ca/480685
cu.
ps.
Schau noch hier ( http://www.dvbcube.org/index.php?topic=20132.0 ) wegen scan-patch.
-
noch besser ;)
schaut gut aus. Ich habe nur einen Rechtschreibfehler in "Electronic Program Guide" gefunden und die englischen Tooltips ein wenig angepasst:
+ QToolTip::add( searchBtn, i18n("Electronic Program Guide Search"));
...
+ QToolTip::add( tvradioCb, i18n("Search TV Channels only (omit Radio)"));
...
+ QToolTip::add( titleCb, i18n("Search in Event Titles only (omit Description)"));
...
+ QToolTip::add( ftaCb, i18n("Search Free to Air Channels only (omit PayTV)"));
hier ist der gesamte patch fürs momentane SVN (kaffeine revision 663183) :
http://www.dvbcube.org/index.php?topic=20143.msg79510#msg79510
-
Schön, schau noch der scan-patch an. Ich habe da ein Fehler bei den Icons load gefunden und die checkChannUpdate Funktion etwas geändert.
Wenn du willst kann ich auch diff gegen svn machen.
-
Schau noch der scan-patch an ....Wenn du willst kann ich auch diff gegen svn machen.
Schon getestet, funktioniert wunderbar.
diff: http://www.dvbcube.org/index.php?topic=20143.msg78557#msg78557
-
Ja, das ist OK.
Ich habe noch bei mir den FTA-Status drin. Weil mich nervt, dass die Sender auf HB13 sehr oft das falsch senden.
chan->fta=chandesc->at(i)->fta;
Aber das muss nicht unbedingt in svn.
-
ah, danke hab ich übersehen.
ist jetzt eingearbeitet in http://www.dvbcube.org/index.php?topic=20143.msg78557#msg78557