VDR version 2.7.8 freigegeben

Ich habe mal ein bischen rumprobiert und zunächst einen zusätzlichen Hauptmenueintrag für "Gelöschte Aufzeichnungen" angelegt. Dabei ist mir aufgefallen, dass das neue Undelete-Menü ja eigentlich schon alles kann und als Ersatz für das bisherige Aufzeichnugnsmenü benutzt werden kann: Rot schaltet zwischen Aufzeichnungen und gelöschten hin und her. Allerdings mussten dafür die "Befehle" auf der roten Taste weichen. Da die Befehle aber sowieso auf eine einzelne Aufnahme wirken (Pfadname wird angehängt) kann man die auf die gelbe Taste im Info-Menü legen, welche bisher nicht benutzt wird.

Das wäre nur ein Tastendruck mehr um an die Befehle zu kommen und die gelöschten Aufnahmen wären intuitiv zu finden.

Hier der Patch dazu als Proof of Concept (man muss über die gelöschten Aufnahmen einsteigen weil im normalen Aufzeichnungsmenü noch die "Befehle" aufgerufen werden):

--- menu.c.orig 2026-01-22 19:00:38.540369641 +0100
+++ menu.c      2026-01-28 18:32:37.420283612 +0100
@@ -2940,7 +2940,8 @@
      if (recording->Deleted())
         SetHelp(tr("Button$Recordings"), tr("Button$Restore"), tr("Button$Permanently delete"));
      else
-        SetHelp(tr("Button$Play"), tr("Button$Rewind"), NULL, tr("Button$Edit"));
+        //SetHelp(tr("Button$Play"), tr("Button$Rewind"), NULL, tr("Button$Edit"));
+        SetHelp(tr("Button$Play"), tr("Button$Rewind"), RecordingCommands.Count() ? tr("Button$Commands") : NULL, tr("Button$Edit"));
      }
   if (recording->Deleted()) {
      int t = recording->Deleted() + Setup.DeleteRetention * SECSINDAY - time(NULL);
@@ -3022,6 +3023,7 @@
           case kGreen:  cRemote::Put(Key, true);
                         // continue with osBack to close the info menu and process the key
           case kOk:     return osBack;
+          case kYellow: if (RecordingCommands.Count()) return AddSubMenu(new cMenuCommands(tr("Recording commands"), &RecordingCommands, cString::sprintf("\"%s\"", *strescape(recording->FileName(), "\\\"$")))); else break;
           case kBlue:   return AddSubMenu(new cMenuRecordingEdit(recording));
           default: break;
           }
@@ -4718,6 +4720,7 @@
   Add(new cOsdItem(hk(tr("Channels")),   osChannels));
   Add(new cOsdItem(hk(tr("Timers")),     osTimers));
   Add(new cOsdItem(hk(tr("Recordings")), osRecordings));
+  Add(new cOsdItem(hk(tr("Button$Deleted recordings")), osRecsDel));
 
   // Plugins:

Als langjähriger Nutzer des Undelete-Plugins möchte ich zunächst meinen Dank aussprechen, dass diese Funktion endlich in den VDR Einzug hält. Und dazu noch mit konfigurierbarer Aufbewahrungszeit .
Die Undelete-Funktion hat mir in all den Jahren mehrfach den A die Aufzeichnung gerettet uns ist inzwischen unverzichtbar für mich.

Wo die Funktion letztlich untergebracht wird ist mir relativ egal.
Die muss auch nicht super schnell zugänglich sein, nur sollte man sich nicht suchen müssen. Man brauch diese Funktion zwar selten, wenn dann aber dringend.
Den eigenen Hauptmenü-Eintrag Undelete-Plugin fand ich eigentlich gar nicht schlecht. Der war im Notfall gut zu finden und hat nicht gestört.
Ein Shortcut in der reccmds.conf anzulegen wäre bei Bedarf auch möglich.

Da ich, wie wohl die meisten hier, eine reccmds.conf verwende, ist der rote Button im Aufnahmemenü wohl eher ungünstig.

Lediglich eine Bitte habe ich in dem Zusammenhang noch:
Könnte man den Menüeintrag für das endgültige Löschen (Purge) abschaltbar machen?
Es wird zwar nochmal nachgefragt, aber die grüne und gelbe Taste sind dicht beieinander und ich kenne meine Pappenheimer .
Mit einer Aufbewahrungszeit von ein paar Tagen hätte man sonst immer Gelegenheit das noch zu richten.

Quote from kls

- Wenn die Option eingeschaltet ist, soll dann nach wie vor "Setup/Red" zu "Gelöschte Aufzeichnungen" führen, oder soll das "toggeln"?

das würde ich gleich halten.

Wenn das als Hauptmenü Eintrag umgesetzt ist, würde ich das nur anzeigen, wenn auch was gelöschtes existiert.

Quote from kls

Was meinst du mit "das bisherige Aufzeichnungsmenü"?

"Aufzeichnungen" im Hauptmenü (Add(new cOsdItem(hk(tr("Recordings")), osRecordings));)
Das "Gelöschte Aufz." zeigt ja das gleiche nach Umschalten auf "Aufzeichnungen" wie das Menü "Aufzeichnungen", nur die rote Taste ist anders belegt.

Quote from kls

Für mich einer [Edit: Tastendruck] zuviel, denn die reccmds benutze ich deutlich häufiger als gelöschte Aufnahmen wiederherzustellen.

Das ist bei mir genau anders herum. Ich schaue eher mal in die gelöschten Aufzeichnungen und benutze reccmds fast nie. Erst recht jetzt, wo man die Aufzeichnungen sofort purgen kann.

Quote from kls

"Gelöschte Aufzeichnungen" optional im Hauptmenü anzuzeigen könnte ich mir vorstellen. Folgende Fragen stellen sich mir dabei:
- Wenn die Option eingeschaltet ist, soll dann nach wie vor "Setup/Red" zu "Gelöschte Aufzeichnungen" führen, oder soll das "toggeln"?
- Soll der Punkt im Hauptmenü auch dann angezeigt werden, wenn es gar keine gelöschten Aufzeichnungen gibt, oder immer ("Aufzeichnungen" wird ja immer angezeigt).

Der Punkt im Hauptmenü macht eigentlich nur Sinn wenn gelöschte Aufnahmen existieren. Zu "Setup/Red" habe ich keine Preferänzen, aber evtl. ist es konsistenter wenn die Tastenbelegung bleibt, sie stört ja nicht.

Quote from kls

Soll der Punkt im Hauptmenü auch dann angezeigt werden, wenn es gar keine gelöschten Aufzeichnungen gibt, oder immer ("Aufzeichnungen" wird ja immer angezeigt).

Ich würde darum bitten, den Eintrag immer im Menu lassen, wenn man ihn nicht vollständig abschalten kann. Der Grund ist, dass mein Hauptmenü mehr Einträge umfasst, als auf der Startseite angezeigt werden können. Ich wähle Menüpunkte deshalb meist per Menu-Taste + Positionsnummer aus, was mit dem entsprechenden Patch auch bei zweistelligen Nummern funktioniert. Wenn sich das Menü dynamisch verändert, wäre das mehr als ungünstig.

Quote from kls

Du kannst ja jetzt die Zeit, wie lange eine gelöschte Aufzeichnung auf der Platte bleiben soll, einstellen.

Stimmt.
Wird aber wohl eine Weile dauern, bis man sich daran gewöhnt .

Quote from kls

Bisher waren nur etwa 5 Minuten

Die sind in Ruhe betrachtet, eigentlich auch mehr als ausreichend.

Quote from kls

Wenn man mal weiß, wo sie ist, sollte das weiter kein Problem sein ;-).

Man darf die psychologische Komponente nicht vergessen, meinte ich:

Die Wiederherstellungsfunktion benötige ich, wie wohl die meisten, eigentlich nur, wenn ich einen Aufnahme versehentlich gelöscht habe.
In der Zeile verrutscht, falschen Knopf gedrückt ...
Meist weil ich in dem Moment nicht aufgepasst habe, schon halb im Bett bin, irgendwie abgelenkt wurde ...
Das passiert sehr selten und im ersten Moment bekomme ich immer einen kleinen Schreck. Das passiert eigentlich jedes mal, obwohl logisch klein echter Grund besteht und dürfte den meisten wohl so gehen.
Nach meiner Erfahrung neigen viele Computer-User dann eher zur Hektik und fangen an die Knöpfe schneller zu drücken als sie denken können, anstatt erstmal durch zu atmen und zu überlegen was zu tun ist.

Die Restore-Funktion sollte man also finden, ohne groß nachdenken zu müssen, unter welcher Farbtaste in welchem Menü sie nun verborgen ist.
(Andererseits muss sich aber nicht per Hotkey auf der FB erreichbar sein, um das andere Extrem zu nennen).

Quote from SHofmann

Ich würde darum bitten, den Eintrag immer im Menu lassen, wenn man ihn nicht vollständig abschalten kann. Der Grund ist, dass mein Hauptmenü mehr Einträge umfasst, als auf der Startseite angezeigt werden können. Ich wähle Menüpunkte deshalb meist per Menu-Taste + Positionsnummer aus, was mit dem entsprechenden Patch auch bei zweistelligen Nummern funktioniert. Wenn sich das Menü dynamisch verändert, wäre das mehr als ungünstig.

Sich verschiebende Hauptmenü-Einträge sind ungünstig, das kann ich bestätigen.

Wenn man will, könnte man zB. "keine wiederherstellbaren Aufzeichnungen", statt "Aufzeichnungen wiederherstellen" anzeigen.
EPGsearch macht das bei einem der "Hauptmenü-Anzeige-Plugins" so.

Quote from kls

Gut, dass wir hier einen Videorekorder steuern und nicht eine Atomraketen-Abschussbasis ;-).

Da muss ich spontan an die allererste "Schlagseite" der C't denken:
Schlagseite c't 1/1998

Habe neuerdings folgendes Problem (was evt. mir der neuen VDR-Version mitkam):

ExtrecmenuNG zweigt während der Aufnahme die Aufnahmelänge in Sekunden an:

Nach einem Restart des VDR wird die Zeit dann richtig in Minuten angezeigt?!

Das hier wäre dann ein Patch, damit streamdev wieder ohne Warnings (bis auf die in libffdvbmpeg) kompiliert:

• 0001 Deprecated VDR functions.patch.txt

Bitte schaut mal drüber, ob der so passt.

Nein, ich habe den Code aus ExtRecMenuNG geklont. Dort fehlt das dann wohl auch. Danke für den Hinweis!

Zweiter Versuch, nachdem ich in svdrp.c gespickt habe:

• 0001a Deprecated VDR functions.patch.txt

Hoffe, jetzt passt es.

Quote from kls

Es fehlt da nämlich das StateKey.Remove().

Stimmt, im Plugin ist es drin, nur ein paar Zeilen weiter unten. Damit verstehe ich den Code von ExtRecMenuNG auch gleich etwas besser, denn das Plugin macht das Locking über cStateKey zu Fuß:

cRecordings *Recordings = cRecordings::GetRecordingsWrite(recordingsStateKey);
cStateKey StateKey;
cRecordings *DeletedRecordings = cRecordings::GetDeletedRecordingsWrite(StateKey);
cRecording *Recording = Recordings->GetByName(FileName);
if (!Recording || Recording->Delete()) {
   cReplayControl::ClearLastReplayed(FileName);
   Recordings->Del(Recording, false);
   DeletedRecordings->Add(Recording);
   StateKey.Remove();
   recordingsStateKey.Remove();
   SetDeleted(Recording->FileName());
   ...
}

Wenn ich dich richtig verstanden habe, ist das gleichwertig zu:

LOCK_RECORDINGS_WRITE;
if (!Recording || Recording->Delete()) {
   cReplayControl::ClearLastReplayed(FileName);
   LOCK_DELETEDRECORDINGS_WRITE;
   Recordings->Del(Recording, false);
   DeletedRecordings->Add(Recording);
   SetDeleted(Recording->FileName());
   ...
}

Müsste oben das SetDelete() nicht eigentlich vor dem recordingsStateKey.Remove() erfolgen?