War leider wieder so ,Addonbrowser war korrekt ,dann dein Plugin upgedatet ,danach den e2 Neustart und dann hab ich wieder den Addonbrowser auf gerufen und dann steht wieder nur deins drin.

Meine user sieht dann so aus ,allso es hat alles rausgehauen nur deins ist drin.

Kann ich nicht sagen weil ich nicht vor dem neustart geschaut hatte sondern erst danach ,die leerzeilen gingen ja auch vorher weg weil die user ersetzt wurde mit deinem einen Eintrag.

Ich werd mal alles runterhauen von dir und neu installieren mit der aktuellen ,mal sehen ob da wo anderst der Wurm drin ist.

Ich hab jetzt das ganze nochmal alles durchgespielt und ich kann deins komplett entfernen ,neu machen, es ändert nix sobald man im GP Addonbrowser externe Kategorien zuerst ausgeschaltet hat und dann zb. deins verschiebt kommt der Fehler wenn man den Addonbrowser schließt ,e2 neustartet und den addonbrowser wieder aufruft ,mache ich das nicht und schalte externe auf ein ist deins immer oben und die anderen sind auch alle da ,da wird dann nix kaputt gemacht in der user.

Ich hab hier ja nie das cam zeug genutzt was auch so was externes installiert ,müsste man eventuell auch damit mal gegentesten ob das dort sich dann genauso verhält ,nicht das es da einen Fehler im GP gibt den noch keiner bemerkt hat weils keiner testet.

Nein ich hatte gerade vorhin alles von dir gelöscht und da waren dann nur im Softmanager deine Einträge und die json in etc/enigma2 (die json hab ich dann auch gelöscht)

Dann hab ich nur deinen Feed installiert und vom Addonbrowser dann dein aktuelles geladen (Addonbrowser hat ja ne neue richtige user dann angelegt).

Danach wie gesagt war es genau das selbe verhalten sobald man im Addonbrowser externe zuerst aus eingestellt hatte ,hat man das auf an geht alles so wie es soll .

Ich hab das jetzt aber auch mal mit dem onestep externen addonlist getestet und da geht es immer ohne Probleme ob ich extern aus oder an hab ,eventuell auch weil nicht immer neu geschrieben wird oder was auch immer ,auf alle Fälle geht es da und mit deinem externen Addon json geht es hier nicht .

Eventuell sollte da auch Sven das mal probieren mit extern aus eventuell kann er es nachstellen .

Ich konnte das gerade nachvollziehen.

Wenn die Option "externe Kategorien immer zuerst" auf "nein" steht, werden nach einem reload von externen Plugins in die FilterAddonUser.json (wenn der Eintrag von gutemine fehlt) nur noch externe Plugins gelistet.

Die Datei sah vor dem Öffnen des AddonManger aber völlig ok aus.

Vollständige Liste ohne die gutemine-Zeile.

Jetzt muss ich mal testen, wo das Problem liegt.

Das ist schonmal gut das du es nachstellen kannst dann könnt ihr ja mal schauen was da quer schießt .Wie gesagt hatte das noch gegen getestet mit den externen was da onestep macht und da passiert das nicht ,daher denke ich muss da bei euch was angepasst werden.

Ich glaub, es liegt am Komma iam Ende des letzten Eintrages

Der letzte Eintrag in der AddonFilterlistuser.json darf am Ende kein Komma haben.

Löscht man jetzt gutemines Zeile, wenn sie der letzte Eintrag in der Liste ist, hat der vorletzte Eintrag dann noch ein Komma, obwohl er dann zum letzten Eintrag wird.

Also muss man nur vor dem .join(lines) aufpassen, dass die Zeile für den letzten Eintrag kein Komma am Ende hat

Oder man löscht die gutemine-Zeile nicht (kein remove), sondern ändert sie einfach direkt.

Oder man baut das auf das json-Modul um, wo man dann nicht aufpassen muss, wenn man einen Eintrag löscht.

Stimmt

An solche Sachen denke ich nicht zuerst

mit json könnte man aber die einzelnen Sachen direkt in der Datei tauschen, ohne die ganze Zeile löschen zu müssen.

Weil dadurch auch das sort oder hide wieder verloren geht.

Auch ist die Vergleichszeile nicht richtig, weil die in der user anders aussieht als in der _gutemine.

Deshalb wird die Zeile aktuell auch noch bei jedem Neustart gelöscht, auch wenn sich eigentlich nicht ändert

Die Vergleichszeile im Plugin hat zu viele \n und das \t:

gp4_json_line = '\t{"name":"gutemine %s", "hide":0, "sort":1, "desc":"gutemine %s %s", "filter":"%s"}\n\t]\n}\n' % (_("Plugins"),_("Plugins"),_("select"),gp4_filter)

In der Datei sieht die Zeile aber so aus (oder auch mit Komma am Ende)

{"name":"gutemine Erweiterungen", "hide":0, "sort":1, "desc":"gutemine Erweiterungen wähle", "filter":"enigma2-plugin-extensions-autodimm,enigma2-plugin-extensions-autopin,enigma2-plugin-extensions-barryallen,enigma2-plugin-extensions-curtain,enigma2-plugin-extensions-dbackup,enigma2-plugin-extensions-instantepgtimer,enigma2-plugin-extensions-userscripts,enigma2-plugin-systemplugins-dreamflash,enigma2-plugin-systemplugins-gutemine,enigma2-plugin-systemplugins-splitscreen,enigma2-plugin-systemplugins-tunererror"}

So wäre die Vergleichszeile wohl besser:

gp4_json_line = '{"name":"gutemine %s", "hide":0, "sort":1, "desc":"gutemine %s %s", "filter":"%s"}' % (_("Plugins"),_("Plugins"),_("select"),gp4_filter)

und beim Vergleich dann noch ein:

if line.strip(",") != gp4_json_line:

Aber ich probiere noch etwas mit json herum

Hallo

gerade mit Dream OS 2.5 /DM900 -GP4 getestet und BA 12.82 +dBackup 2.5 installiert ,funzt beides !!

Danke

So, hier der neue Code, wo folgende Sachen angepasst sind

- die AddonFilterlistuser.json wird nicht mehr kaputt gemacht, wenn der gutemine-Eintrage der letzte war

- der Eintrag wird beim Start/check nur noch entfernt, wenn sich etwas geändert hat

(hab auch mal die Zeilen für gp4_json_line und gp4_json_txt etwas angepasst - ist besser lesbar)

Das mit dem json-modul ist blöd, weil die Datei dann eine völlig andere Struktur bekommt.

Da steht jedes Attribute in einer neuen Zeile.

Bevor es da Probleme mit irgendwas anderem gibt, hab ich das erstmal gelassen

def gp4Write(gp4_filename,gp4_filter):
    cprint("[gutemine] gp4Write: %s, %s " % (gp4_filename, gp4_filter))
    #create AddonFilter_gutemine.json file for the AddonManager if the user file exists
    gp4_userfilename = "/etc/enigma2/AddonFilterlistuser.json"
    if os_path.exists(gp4_userfilename):
        gp4_name = "gutemine %s" % _("Plugins")
        gp4_desc = "gutemine %s %s" % (_("Plugins"),_("select"))
        gp4_json_txt = '{\n\t"entrys": [\n\t{"name":"%s", "hide":0, "sort":1, "desc":"%s", "filter":"%s"}\n\t]\n}\n' % (gp4_name, gp4_desc, gp4_filter)
        gp4_json_line = '\t{"name":"%s", "hide":0, "sort":1, "desc":"%s", "filter":"%s"}' % (gp4_name, gp4_desc,gp4_filter)
        cprint("[gutemine] gp4: %s" % gp4_json_txt)
        if os_path.exists(gp4_filename):
            f = open(gp4_filename, "r")
            old_json=f.read()
            f.close()
            if old_json != gp4_json_txt:
                cprint("[gutemine] re-writing file: %s" % gp4_filename)
                f= open(gp4_filename, "w")
                f.write(gp4_json_txt)
                f.close()
        else:
            cprint("[gutemine] creating file: %s" % gp4_filename)
            f= open(gp4_filename, "w")
            f.write(gp4_json_txt)
            f.close()
        
        lines = open(gp4_userfilename).readlines()
        for line in lines:
            if line.find("gutemine ") is not -1:
                if line.strip("\n").strip(",") != gp4_json_line:
                    # remove gutemine from AddonFilterlistuser.json to reload 
                    # the new AddonFilterlist_gutemine.json in GP4 AddonManager
                    cprint("[gutemine] remove gutemine line from %s to reload" % gp4_userfilename)
                    lines.remove(line)
                    gp4_json_txt = "".join(lines).replace(",\n\t]","\n\t]") #replace ',' on last entry
                    f = open(gp4_userfilename, "w")
                    f.write(gp4_json_txt)
                    f.close()
                    break

Nun wäre aber immer noch das Problem, wenn einer "sort" oder "hide" bei sich geändert hat, wird die Zeile auch immer gelöscht.

Evtl. müsste man die gutemine-Zeile splitten und die Werte für sort/hide übernehmen und nur die geänderten Werte (Name, desc, filter) eintragen.

Dann die alte Zeile durch die geänderte Zeile ersetzen und nicht löschen

Ich probiere da noch etwas herum

Du hast dich aber da festgehalten das der Eintrag der letzt wäre ,ich hab den nur dort hin sortiert bei mir ,jemand anders kann den auch woanders hin sortieren ,das ist ja ne verschiebe Funktion und die sagt nicht das man den gutemine Eintrag bis an letzte stelle verschieben muss.

Das gleiche gilt hide ,sort und hide sind nunmal Funktion des GP addoanager die man verwenden kann ,dafür sind sie gemacht com Plugin Ersteller.

Was willst du damit jetzt sagen ?

Der Fehler trat nur auf, wenn der gutemine-Eintrag als letzter in der Liste stand.

Deshalb ist es vermutlich auch keinem aufgefallen.

Da ergab sich ja das Problem mit dem übriggebliebenen Komma nach dem dann letzten Eintrag.

Sofern der Eintrag weiter vorne stand, hat es ja funktioniert, wie bei den meisten hier

A OK dann ist es was anderes ich hatte es nur immer als letztes geschoben und daher auch nicht probiert mit einer anderen stelle .

Wenn du sagst das der Fehler dann nur passiert und ihr das gefixt habt ist das OK ,ich wollte vorhin nur sagen das du das mit einer anderen stelle auch testen solltest weil ich das halt nicht gemacht habe ,nicht das es dann da auch passiert mehr nicht.

Mit deinem "besonderen" Verschieben ans Ende hast du einen Bug entdeckt

Das hat wohl kein anderer bisher gemacht

Wenn der AddonManager eine geänderte externe Filter-json eigenständig passend nachladen würde, hätte man es hier deutlich einfacher

Weil wer sagt, dass sich die externe Filter-json nicht mal ändern könnte ?!?

Im Moment lädt dieser die Daten ja nur nach, wenn man die ganze Zeile vorher löscht.

Also so müssen wir den Job zum Aktualisieren der externen Filter-json jetzt hier übernehmen, was gerade in Bezug auf hide/sort eben etwas komplizierter wird.