Página 1 de 1

Cambio Masivo en pelisalacarta

Publicado: 01 Abr 2010, 18:32
por jurrabi
Aunque este post es continuación de los de xbmc 10.5 como afecta a todo el plugin preferí iniciar un nuevo Hilo.


Pongo este post para explicar que he realizado un cambio masivo en 55 módulos del plugin (deben ser casi todos) para reemplazar el uso de la función xbmcplugin.getSetting por otra llamada a xbmctools.getPluginSetting.

La razón de hacer esto es que me era imposible seguir avanzando en el desarrollo para compatibilidad con 10.5 mientras arreglaba fallos en la versión actual. Por eso he decidido crear una función GetPluginSetting que en función de la versión en que nos encontremos llama a xbmcplugin con los parámetros adecuados a cada versión.

Dado que los cambios son masivos no he podido comprobar completamente que todo funcione. Pero si fallara algo se arreglaría simplemente añadiendo un import del módulo xbmctools.

Espero que el cambio os parezca bien. Ruego que a partir de ahora uséis esta nueva función para obtener parámetros de configuración del plugin.

Re: Cambio Masivo en pelisalacarta

Publicado: 02 Abr 2010, 11:10
por jesus
Gracias por el cambio :)

Me preocupa un poco eso de que "no has podido probarlo completamente"... si vas a hacer un cambio de esas características, debes probarlo bien.

De lo contrario cuando vaya a subir una nueva versión, si veo que no funciona nada te pitarán los oídos :)

Re: Cambio Masivo en pelisalacarta

Publicado: 02 Abr 2010, 15:03
por jurrabi
Aunque te agradezco tu confianza en que yo sea capaz de probar todo, si lo subo al branch es precisamente para que los que dispongan de tiempo puedan asistir en tan hercúlea tarea. Si ves que el cambio no es adecuado siempre se puede dar marcha atrás a la revisión.

Re: Cambio Masivo en pelisalacarta

Publicado: 02 Abr 2010, 16:04
por jurrabi
En encontrado dos imports que faltaban en 2 módulos. Ya está subido.

he revisado uno a uno todos los módulos que llaman a getPluginSetting (102 llamadas en 55 módulos) y todos tienen un import adecuado.

En un caso el import no funcionaba por problema de import circular (xbmctools que importa downloadtools que, para usar getPluginSetting, importa a su vez a xbmctools). Lo solucioné replicando la función en downladtools.

Esto debería cubrir todos los casos.

Prometo no meterme en más camisas de once varas... a partir de ahora seré un usuario modelo. Perdón por los inconvenientes que haya podido causar.

Re: Cambio Masivo en pelisalacarta

Publicado: 03 Abr 2010, 21:15
por jesus
Import circular?

Empiezo a tener la sospecha de que lo de los imports no está bien resuelto... la verdad es que el Python no es mi fuerte.