Erstellt vor 4 Jahren

Geschlossen vor 3 Jahren

Zuletzt geändert vor 3 Jahren

#2595 closed enhancement (fixed)

[7390] Konfigurationseite zum An/Abschalten der Box-LEDs wieder verfügbar machen

Erstellt von: make Verantwortlicher:
Priorität: normal Meilenstein: freetz-next
Komponente: patches Version: devel
Stichworte: Beobachter:
Product Id: 7390 Firmware Version: 06.0x

Beschreibung

Den angehängten Patch verwende ich seit geraumer Zeit um die vor einiger Zeit abgeschaltete, aber nicht entfernte Seite zum Abschalten der FritzBox-LEDs im AVM-Webinterface wieder verfügbar zu machen. Die Seite ist dann wie gehabt über System→Tasten und LEDS→[LED-Anzeige] erreichbar.
Da ich nur eine 7390 zum Testen habe, ist der Patch entsprechen auf dieses Modell und Firmwares ≥ 06.0x beschränkt). Prinzipiell müsste das auch mit einer 7490 funktionieren.

Sofern das für andere von Nutzen ist und nicht irgendwelchen Prinzipien zuwider läuft könnte man den Patch integrieren. Das "Wording" wäre dann ggf. noch etwas zu überarbeiten.

Anhänge (1)

make_leds_switchable.patch (1.8 KB) - hinzugefügt von make vor 4 Jahren.
Korrigierter Patch

Alle Anhänge herunterladen als: .zip

Änderungshistorie (11)

comment:1 Geändert vor 4 Jahren durch Dropsrolle

Iwas stimmt mit dem Patch net …

###ars@ubuntu:~/freetz/trunk-12656$ patch -p0 < make_leds_switchable.patch
patching file config/ui/patches.in
patch: **** malformed patch at line 20: Index: patches/cond/535-enable_led_deactivation.patch

comment:2 Geändert vor 4 Jahren durch make

Hängt bestimmt damit zusammen, dass ich den Patch nach dem svn diff… nachträglich editiert habe. Ok, danke für den Hinweis.

Geändert vor 4 Jahren durch make

Korrigierter Patch

comment:3 Geändert vor 4 Jahren durch make

Müsste jetzt funktionieren. @Dropsrolle: Bitte vorher ggf. config/ui/patches.in auf den trunk-Stand zurück setzen.

comment:4 Geändert vor 4 Jahren durch er13

Muss .exists_page wirklich in .show_page geändert werden? Wenn ja, warum dann nur in einem Fall?

comment:5 Antwort: Geändert vor 4 Jahren durch make

Gegenfrage: Warum hat AVM sinnfrei zweimal hintereinander menu.exists_page einen Wert zugewiesen?

Ich hab mich (meiner Erinnerung nach) mit dem Patch an der Firmware-Version orientiert, in der die LED-Seite noch eingeblendet wurde.
Generell gibt es wohl in menu.show_page[] die Funktion zum Generieren einer Seite und in menu.exists_page[] ein boolean zum Abfragen, ob eine bestimmte Seite aktuell verfügbar ist (Stichworte: Repeater-Mode, Expert Mode etc.). Das wird in meinem Patch wieder hergestellt, die LED-Seite ist (nur) im Experten-Modus verfügbar.

Zuletzt geändert vor 4 Jahren von make (vorher) (Diff)

comment:6 Geändert vor 3 Jahren durch er13

In 12938:

AVM web-interface:

  • add patch enabling "LED display"-page in AVM web-if (available but disabled by AVM for unknown reason) [by make + some tweaks by me]
  • compile-tested only
  • refs #2595

comment:7 als Antwort auf: ↑ 5 Geändert vor 3 Jahren durch er13

Replying to make:

Ich hab mich (meiner Erinnerung nach) mit dem Patch an der Firmware-Version orientiert, in der die LED-Seite noch eingeblendet wurde.

Welche Firmware hast Du denn als Vorlage verwendet? Ich habe bei 7390 zurück bis in die 05.2x geschaut, vom Lesen her sieht so aus, dass die Seite schon in 05.2x gefehlt hat.

Ich habe eine von mir etwas veränderte Version des Patches eingecheckt, könntest Du bitte testen, ob der Patch immer noch funktioniert, ich habe ihn auf der Box nicht getestet. Danke!

comment:8 Geändert vor 3 Jahren durch make

Sieht gut aus. Funktioniert bei mir mit der letzten 7390-Labor und dem aktuellen Trunk wie erwartet.

comment:9 Geändert vor 3 Jahren durch er13

  • Lösung auf fixed gesetzt
  • Status von new nach closed geändert

Danke fürs Testen!

comment:10 Geändert vor 3 Jahren durch er13

In 13357:

Fritz!OS 6.35:

  • adjust led_deactivation-patch
  • refs #2774, refs #2595
Hinweis: Hilfe zur Verwendung von Tickets finden Sie in TracTickets.