search

E2OpenPlugins / e2openplugin-OpenWebif

GPL Web Interface for e2stabs
GNU General Public License v3.0
204 stars 226 forks source link

Modularization of components #464

Closed Schimmelreiter closed 7 years ago

Schimmelreiter commented 7 years ago

Modularize all possible components of OWIF so that users and distros can decide which features they want or do not want.

This includes but is not limited to the following already existing features in OWIF:

Module/plugin loading should not depend on "if blah exists" but on a "foreach" principle, so anyone can create and add plugins/modules for OWIF in a comparably simple way but just putting the necessary files into a specific subdirectory of OWIF.

Distro teams and others can and are encouraged to add plugins/modules based on their own needs, e.g.

jbleyel commented 7 years ago

Hi @Schimmelreiter .. wie meinst du das genau.
So wie ich das verstanden habe, willst du einen neuen Order "Module" Dort kommt dann für jedes Modul ein Ordner rein.

Nehmen wir mal das Beispiel mobile interface. Es gibt dann also unter Module einen Ordner mobilewebif. In diesem gibt es dann public und z.B. mobile (für die templates) Unter public gibt es dann js und css und images?

In dem Order mobilewebif gibt es dann ein Modul.py in dem alles gemacht wird, dann jetzt in Root drin ist.

Wenn jemand dann das mobilewebif nicht haben will dann muss er nur den Ordner Module/mobilewebf löschen.

RICHTIG????

Schimmelreiter commented 7 years ago

Ganz genau. Wobei ich nie verstehen werde, wieso jeder Scheiß nochmal ein Unterverzeichnis braucht.

Auf jeden Fall sollte OWIF beliebige Plugins einlesen können (Es muß also definierte APIs geben), so daß irgendjemand auch einfach nur ein Plugin unter Module/exotenplugin reinkopieren kann und das wird dann automagisch unterstützt.

Schimmelreiter commented 7 years ago

Es wäre also top, wenn jedes Plugin ein xml bekäme, z.B. Module/blah/plugin.xml in dem dann z.B. steht, ob und wo das Plugin gerne eingehangen werden möchte.

Schimmelreiter commented 7 years ago

So, ich habe heute keine Sprechzeit mehr ...

Schimmelreiter commented 7 years ago

Eines noch: Ich würde für jedes neue Release sehr enge Ziele stecken und dann wirklich nur die verfolgen. Diese Spielerei namens FanControl2 hält jetzt schon wieder alles auf.

Für 1.1 sollte IMHO das Ziel nur sein, daß Themes, WebTV, mobile Ansicht und Paketmanager als Plugins abgespalten werden können Das ist schon eine ganze Menge, dann danach müssen wir ja alle möglichen Konstellationen auch testen (Also OWIF ohne Mobile aber mit WebTV, OWIF ohne WebTV aber mit Paketmanager, usw. usf.). Sonst rutscht uns irgendwas durch, was in mehreren Paketen benötigt, aber in nur eines ausgespalten wurde.

jbleyel commented 7 years ago

Dann schau dir mal die aktuelle Implementierung von Webtv, mobile und themes genau an und du wirst verstehen warum ich das NICHT so eingebaut habe. Die zusätzlichen Web Ordner (css,js,..) sind kein Problem. Probleme bekommen wir mit den Templates, wenn es nicht nur eines gibt. (z.B. mobile) Siehe /views/mobile.py

Mag sein, das andere plugins einfacher zu integrieren sind. Die 3 oben genannten auf jeden Fall nicht.

Der Packagemanager wird richtig hässlich.

jbleyel commented 7 years ago

Fancontrol2 ist übrigens pille palle. Das sind nur 10 Zeilen Code und sogar für andere plugins ganz einfach zu erweitern.

Schimmelreiter commented 7 years ago

Deshalb ja: Konzentrieren wir uns auf genau diese eine Aufgabe, also die vorhandenen Teile in Module abspalten.

Alles weitere dann in 1.2.

Ab morgen kann ich wieder mitmachen/testen.

jbleyel commented 7 years ago

Morgen mach ich nix. Es sollte vorher auf jeden Fall geklärt werden wie das Abspalten ausschaut.

Kommt das Openwebif mit themes,webtv und mobile und du kannst diese 3 Teile raus löschen. Aus Platzgründen. Oder kommt das openwebif ohne diese 3 Teile und du musst ein extra Plugin installieren, was diese 3 Teile dazu kopiert.

Packagemanger abzuspalten geht aktuell nur mit extrem viel Aufwand. Gleiches gilt für autotimer, bouqueteditor und epgrefresh

Ich würde es für die 1.1 so lassen wie es jetzt ist und wenn du nicht genug Platz auf der Box hast, dann musst du nur den einen Order für den Teil löschen, den du nicht willst.

jbleyel commented 7 years ago

YEP Der eine Fehler stört mich.

Collected errors:

Da ist was anderes Faul.

sgse commented 7 years ago

Sehe gerade er bietet mir eine Neue Version am TV an. Update von VTi von 1.0.4 auf 1.1.0 ist aber gestern auch fehlerhaft abgebrochen. Update Datum ist der 02.11.2016?

jetzt alles frisch aber kein Web-TV mehr

jbleyel commented 7 years ago

Webtv ist gefixt

sgse commented 7 years ago

Bei mir leider nicht. Muss ich jetzt immer erst deinstallieren? net5 Ich muss aktualisieren damit es geht net6

jbleyel commented 7 years ago

browser cache?

sgse commented 7 years ago

Wenn wir dich nicht hätten :-) jepp.

sgse commented 7 years ago

Man kann mit @jbleyel Probleme lösen die man ohne @jbleyel nicht hätte :-D ;-) :-)

Besser oben @jbleyel mit Computer ersetzen. ;-)

Schimmelreiter commented 7 years ago

Jbleyel ist doch nicht Python. ...

jbleyel commented 7 years ago

Und jbleyel ist auch nicht HTML und auch nicht javascript.

sgse commented 7 years ago

Wenn sich meine linke Shift-Taste nicht ständig lösen würde (da ist so ein micro Plastikteil abgebrochen) wäre ich mit dem neuen OWIF 1.1.0 vollständig zufrieden ;-)

sgse commented 7 years ago

Das verwirrt mich jetzt, 1.1 ist drauf net8 Zumindest bietet mir VTi kein Update mehr auf 1.1 an. :-)

jbleyel commented 7 years ago

keine Ahnung was da faul war .. die 1.0.4 ist die aktuelle master und nicht die 1.1.0

sgse commented 7 years ago

Das auslesen erfolgt wann? Ich bekomme bei installed und all, enigma2-plugin-extensions-openwebif (1.0.4-e2openpluginsgit-r0), angezeigt aber 1.1 ist aktuell drauf. Habe ja keine Ahnung von mergen und so aber evtl. kommt es ja davon das da dieser Wert herkommt, macht ja auch keine Probleme. :-) https://github.com/E2OpenPlugins/e2openplugin-OpenWebif/commit/fd74b1adec7c7a633717141141ac9192411015b1

sgse commented 7 years ago

opkg list-installed = enigma2-plugin-extensions-openwebif - 1.1.0-e2openpluginsgit-r0

sgse commented 7 years ago

Mir wird immer noch enigma2-plugin-extensions-openwebif (1.0.4-e2openpluginsgit-r0) angezeigt. Wo wird das ausgelesen?

sgse commented 7 years ago

Kein Web-TV mit https://github.com/E2OpenPlugins/e2openplugin-OpenWebif/commit/7bc25dd190d781ce332fca1cc311d986210563f4 FF cache gelöscht.

jbleyel commented 7 years ago

müsste jetzt wieder gehen.

sgse commented 7 years ago

Danke :-)