[Announce] director plugin Modifikation

Quote

Original von zulu

oder, das Plugin zeigt einfach die Daten von Direkt an bzw wenn keine vorhanden, die zur Verfügung stehenden Portale und/oder Sender.

Hmm - sone Art Portalübersicht? Find ich gar ned schlecht die Idee

Quote

Bei Systemen mit mehreren Karten und min. einer freien könnte sich das Plugin die Daten ja auch holen.

Gruß
Marc

Hmm - das versteh ich ned ...

arghgra

Quote

Original von dortje
Super arghgra dass dus weiterentwickelst! Folgende Sache stört mich beim täglichen benutzen: Wenn man das director-menü schliesst, also zbsp mit Menü und auf nen anderen Kanal geht (zbsp ZDF), dan poppt da das director Fenster auf mit einem Eintrag nämlich der momentan (in dem Beispiel auf ZDF) laufenden Sendung: ganz schön nervig.

Sonst find ich das plugin sehr gut!

Sollte eigentlich nicht passieren ... muss ich mal ausprobieren, ob ich das hinbekomme.

arghgra

Quote

Original von horchi
Moin,

das liegt an einem 'anderen' verhalten des VDR der neueren Versionen. Hier der angepasste Code (Methode cDirectorStatus::ChannelSwitch in arghdirector.c tauschen).

Code

void cDirectorStatus::ChannelSwitch(const cDevice* Device, int ChannelNumber)
{
   const cChannel* Channel = Channels.GetByNumber(ChannelNumber);


   if (Channel && parent)
   {
#if VDRVERSNUM < 10333
      if(Channel->GetLinkChannels() && Channel->GetLinkChannels()->Count() > 1)
#else
      if(Channel->LinkChannels() && Channel->LinkChannels()->Count() > 1)
#endif
      {
			cRemote::CallPlugin(parent->Name());
			dsyslog("cDirectorStatus::ChannelSwitch "
                 "Channelnumber: %d (%d), Links: %d", 
                 ChannelNumber, Channel->Number(),Channel->LinkChannels()->Count());
		}
   }
}

Display More

Grüße

Hmm - kann ich von der Logik weder mit 1.4.7 noch mit 1.5.13 nachvollziehen - ChannelNumber == 0 indiziert einen PreChannel-Switch, den das Plugin auslässt.
Der bisherige Source hat bisher zumindest halbwegs die LinkChannels geprüft ...

Wenn sich hier 2,3 Leute finden, die mal ne bereinigte Grundversion testen können, wäre das prima - ich bau noch 2 Sachen ein und lade die Tage mal was hoch.

arghgra

Quote

Original von horchi
die Entscheidende Änderung war, über die ChannelNumber "Channels.GetByNumber(ChannelNumber);" zu gehen, statt über "Device->CurrentChannel())" da CurrentChannel nun erst erst später nach dem ersten Aufruf von ChannelSwitch aktualisiert wird. Bei mir ist seit dem das starten des Plugins beim 'wegschalten' von einem Multi-Feed Kanal wieder weg.

horchi

Habs mal eingebaut und die ChannelSwitch-Methode "eigentlich" blickdicht gemacht - nun passiert mir der Fehler allerdings mit 1.4.7 ;).
Werde das mal genauer prüfen - sollte doch hinzubekommen sein ;).

arghgra

So - da ich momentan irgendwie nix an Zeit habe, hier mal der momentane Stand - vielleicht findet sich ja jemand, ders schon mal testen mag

Quote

History
2008-02-XX: Version 0.2.7
- fixed memleak, removed compiler-waring (thx to horchi)
- enabled hotkeys in skinmenu (thx to horchi)
- dropped compat to VDR 1.3.x
- removed old-director display-mode
- corrected Makefile to use APIVERSION correctly
- updated no of languages in arghdirectori18n.*
- autostart can now be configured during runtime (and works )
- added new setup-option to autostart plugin on referenced channel
- made autostart on channel-switch in cDirectorStatus::ChannelSwitch a bit more robust
- added option to avoid start on simple single channels
- added option to ask before autostarting the plugn

TODO
--
* add latest feature-patches from horchi
* make mainmenu-entry editable in setup
* add thread to check link(ed) channel(s) and epg-info for dynamic-updates
* add finally dbox1-view
* add missing README/...-whatever texts
* judge user-ideas

Display More

DL: https://www.vdr-portal.de/www.arghgra.de….7-test1.tar.gz

arghgra

Quote

Original von zulu
Hallo,

die Meldung bei nicht passenden Kanälen ist schon mal super. Leider kommt die manchmal auch bei passenden Kanälen. Erst wenn ich dann einmal auf das "Direkt Portal" umschalte funktioniert's.

Das ist das Problem, dass VDR ne gewisse Zeit braucht, um die Link-Channel-Infos zu aktualisieren. Aus diesem Grund soll ein permanenter Überwachungsthread ins Plugin rein - damit werden dann Änderungen automatisch erkannt.

Quote

Was hat das denn mit der doppelten Setup Option für den Autostart auf sich?

Meinst du:

Add(new cMenuEditBoolItem(tr("Autostart plugin on reference-channels"), &autoStart, tr("no"), tr("yes")));
  Add(new cMenuEditBoolItem(tr("Autostart plugin on referenced channels"), &autoStartRef, tr("no"), tr("yes")));

Ist quasi der Unterschied Master/Slave-Channel - das Setup wird aber zum Schluss noch sinnvoll entschlackt - da gehört das Zusammenfassen dieser Optionen dazu.

arghgra

Quote

Original von zulu
Danke, genau die meinte ich.

Code

Ist quasi der Unterschied Master/Slave-Channel...

Ah, super! Den Autostart fand ich immer nervig. Autostart nur auf dem Master-Channel ist aber sehr angenehm.

Also lieber die beiden Setup-Optionen so getrennt drinne lassen?

arghgra

Quote

Original von zulu
Von der Funktion her so lassen und im Setup auf einen Button legen fände ich gut:
Autostart plugin on : [ none, Master, Slave, Master + Slave ]

Hehe - manchmal seh ich den Wald vor lauter Bäumen nicht - so machen wir das!

arghgra