[UPDATE] 7TV (HD-Version)

Link ist leider nicht erreichabr

@PUR3 @L0RE: Wäre das hier nicht ein Kandidat für Repo Lost&Found ?

@PUR3

Bromix ist weiterhin aktiv https://github.com/bromix?tab=activity

Er ist nur nicht mehr hier im Forum unterwegs sondern im offiziellen Kodiforum.
Die Addons die er Abgeben wollte sind im Thread Addon Flohmarkt von ihm aufgelistet worden.
Addon-Flohmarkt
wie Du siehst habe ich nowtv und Clipfish übernommen die anschließend aus der Liste gestrichen wurden.

Break Pictures - Schaue lustige Bilder von break.com
DMAX.de - Mediathek für DMAX.de
DZANGO - Filme von DZANGO
Netzkino - Filme von Netzkino
TLC.de - Mediathek für TLC.de

sind die Addons die er aktuell noch abgeben wollte.

Alle anderen Addons pflegt er noch selbst.

Wir können hier nicht wahllos Addons übernehmen und verbessern und erst recht nicht mit der selben AddonID und wenn wir es doch in die Lost and Found Repo stellen können wir es nicht ins offizielle Repo pushen.

Sonst passiert das selbe wie es Bromix mal mit einem ASDE addon passiert ist und wie es Membrane mit ASDE gemacht hat. [Release] ARD Mediathek
Dann brauchen wir uns nicht wundern wenn keiner hier mehr den Weg zurück findet.

Der korrekt weg ist das Addon von Github zu forken es dann zu verbessern oder fixen und einen Pull Request zu machen.
Reagiert der Addon Author nach einer Weile nicht oder ist lange Zeit unaktiv kann man immer noch über andere Sachen nachdenken.

Das das Den Usern völlig egal ist ist mir klar aber das interessiert in so einen Fall nicht.

Gruß Jan

PS Pure3

Das ist nicht böse gemeint aber bei den Bromix addons die er nicht im Addon Flohmarkt erwähnt hat müssen wir erst unsere Veränderungen dem Addon Author übermitteln er wird dann schon reagieren.

@PUR3 Soll ich den Pull Request machen für dich?

Zitat von PUR3

@skatulskijean
Das ist mir klar, ich habe ihm das auch schon mitgeteilt und er hat mir mitgeteilt, dass ich einen 'PR' erstellen soll.
Da ich mich mit Github nicht auskenne habe ich es hier halt veröffentlicht. Da es in keinem Repo ist stellt es kein Problem dar, dass die ID dieselbe ist,
insbesondere deshalb nicht, dar ich die Versionsnummer auf der letzten Ebene erhöht habe, die Bromix sowieso nicht benutzt.

Das ist doch alles ok und klar und mein Post war eher an die User gerichtet als an Dich selbst .
Denn für sie ist das alles nicht so klar aber nun sollten sie es verstanden haben.
Bevor wir irgendein Addon ,eines anderen ,das wir verändert haben in eine Repo geben müssen wir :

-forken des nicht mehr funktionierenden addons
-Veränderungen vornehmen
-PullRequest
-abwarten und kein in irgendeine repo packen außer der Developer reagiert nicht und ist dauerhaft abwesend
- Wenn der Developer reagiert und unsere changes übernimmt oder funktionell übernimmt.
können wir das von uns geforkte addon löschen.
-Wenn der Developer nicht reagiert aber anwesend ist (können wir ein eigenes Addon mit einer anderen
Addon Id erstellen das dann unsere changes beinhaltet.
-Wenn der Developer nicht reagiert und auch dauerhaft abwesend ist können wir (damit tue ich mich schwer) das Addon übernehmen oder aber ein eigenes Addon mit einer anderen Addon ID erstellen.

Gruß Jan

Habs den Pull request gemacht:
https://github.com/bromix/plugin.video.7tv/pull/18

@Lore
Pure

Cool

Aber Pure warum nimmst Du denn um aus einem Json request die Werte zu holen nicht json oder requests das einen json interpreter integriert hat anstatt das über Regex zu machen.

ich meine mit urllib2 und json:

url=die url im json format
req = urllib2.Request(url)
response = urllib2.urlopen(req)
data = json.load(response)
response.close()

oder mit requests im Einzeiler:
data=requests.get(url).json()

Gruß Jan

@PUR3

Ja bei einem einzigem Wert kann man es auch mit regex machen.
Aber json ist schnell und alles was ne api hat erreicht man über xml oder json besser als über einen regex
War ja auch eigentlich auch auf den Addon fix gemünzt denn Bromix benutzt ja in seinem addon für alle json api aufrufe auch json und nicht re er macht es über sein simple_requests modul unter dem Kodi Modul.

requests ist ein scriptmodul das habe ich schon in xbmc4xbox addons benutzt es macht vieles simpler und den code kürzer und ist nicht langsamer als die urllib2 oder schlechter als urllib2 sondern besser!

Gruß Jan

Hallo, hab antwort von Bromix bekommen:
fyI

Thanks, I'll look into it...I'm just updating the addon from "scratch"
See: https://github.com/bromix/plugin.video.7tv/tree/nightcrawler

I'll try to integrate this PR into the new version.

Zitat von L0RE

Hallo, hab antwort von Bromix bekommen:
fyI

Thanks, I'll look into it...I'm just updating the addon from "scratch"
See: https://github.com/bromix/plugin.video.7tv/tree/nightcrawler

I'll try to integrate this PR into the new version.

Wunderbar,wenn das mit ASDE auch klappen würde wäre es ja fein.
Hast Du eigentlich irgendwann bei irgend einen gefixten Addon von ASDE einen Pullrequest gemacht?

Gruß Jan

Jep
https://github.com/AddonScriptorD…ntral_de/pull/1
Jep eigentlich 2 Pull Requests. den ersten hatte ich dan zurückgezogen weil ich inzwischen mehr gepatcht hatte, und habe nen neuen gemacht. Das ist der der jetzt noch offen ist. Seit Januar hat da sich leider nix getan

Zitat von L0RE

Jep
https://github.com/AddonScriptorD…ntral_de/pull/1
Jep eigentlich 2 Pull Requests. den ersten hatte ich dan zurückgezogen weil ich inzwischen mehr gepatcht hatte, und habe nen neuen gemacht. Das ist der der jetzt noch offen ist. Seit Januar hat da sich leider nix getan

Himm das ist so ne Sache mit ihm ,eigentlich sehr schade.
Komisch das auch keiner weiß was mit ihm ist.
Ja dann werden wir wohl das bei ihm so handhaben wie bisher seit dem er Abwesend ist fixen und ab jetzt in die repo lost ans found.

Gruß Jan

Update von Bromix
A similar version is implemented in the **3.0.0~alpha1** of the addon
and available for tests via my repo...hope to finish this by the end of
the month.

https://github.com/bromix/plugin.…6159d2562e52492