Página 4 de 5
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 12 Dic 2016, 22:30
por SeiTaN
Tiene pinta de que agregaste una serie hace mucho y luego actualizaste la biblioteca, a mi tb me ha pasado en el fichero tvshow.nfo, te pondrá channel="canalxx" y action="findvideos" creo recordar, para que funcione en la biblioteca ahora se guarda channel="biblioteca" y action="get_temporadas"
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 13 Dic 2016, 00:34
por identidad_oculta
SeiTaN escribió:Tiene pinta de que agregaste una serie hace mucho y luego actualizaste la biblioteca, a mi tambien me ha pasado en el fichero tvshow.nfo, te pondrá channel="canalxx" y action="findvideos" creo recordar, para que funcione en la biblioteca ahora se guarda channel="biblioteca" y action="get_temporadas"
Lo de action="get_temporadas" si que lo ponía, lo que era distinto es channel, al editarlo manualmente o borrar y volver a añadir la serie funciona. Hay que editar manualmente todos los .nfo? Al actualizar no los modifica
PD: SeriesBlanco da un error al añadir una serie con el menu contextual:
File "/Users/xxx/Library/Application Support/Kodi/addons/plugin.video.pelisalacarta/platformcode/launcher.py", line 228, in run
library.add_serie_to_library(item, channel)
File "/Users/xxx/Library/Application Support/Kodi/addons/plugin.video.pelisalacarta/core/library.py", line 599, in add_serie_to_library
if "###" in item.extra:
TypeError: argument of type 'int' is not iterable
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 13 Dic 2016, 09:29
por super_berny
kerenmac escribió:Al actualizar no los modifica ?
Habria q saber exactamente q version de la biblioteca tenias cuando agregaste esa serie.
Sobre el error q comentas, no habíamos contemplado la posibilidad de q item. extra fuera otra cosa mas que un str. La solución seria: if isinstance(item.extra, str) and "###" in item.extra:
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 13 Dic 2016, 10:12
por Lortropic
kerenmac escribió:SeiTaN escribió:Tiene pinta de que agregaste una serie hace mucho y luego actualizaste la biblioteca, a mi tambien me ha pasado en el fichero tvshow.nfo, te pondrá channel="canalxx" y action="findvideos" creo recordar, para que funcione en la biblioteca ahora se guarda channel="biblioteca" y action="get_temporadas"
Lo de action="get_temporadas" si que lo ponía, lo que era distinto es channel, al editarlo manualmente o borrar y volver a añadir la serie funciona. Hay que editar manualmente todos los .nfo? Al actualizar no los modifica
PD: SeriesBlanco da un error al añadir una serie con el menu contextual:
File "/Users/AlvaroQF/Library/Application Support/Kodi/addons/plugin.video.pelisalacarta/platformcode/launcher.py", line 228, in run
library.add_serie_to_library(item, channel)
File "/Users/AlvaroQF/Library/Application Support/Kodi/addons/plugin.video.pelisalacarta/core/library.py", line 599, in add_serie_to_library
if "###" in item.extra:
TypeError: argument of type 'int' is not iterable
Con que menú contextual? Porque yo he añadido todas las series en SeriesBlanco con la versión actual del addon. Y son casi 20.
Entiendo que pete en esa linea, lo que a mi no me había pasado. Describe los pasos que has seguido hasta llegar al pete.
Edit: Veo que es posible que no lo viera (aunque me parece que hace bastante) porque parece que el código se añadió posteriormente:
https://github.com/tvalacarta/pelisalac ... library.py
Separación del código xbmc del generico en library.py
13 days ago
Aunque creo que esto ya debería estar antes (tal vez en otro sitio), pero bueno. Le daré un vistazo esta tarde. (el motivo es que uso el "extra" como número de página y eso no es iterable, y esa linea de código hace "####" in item.extra)
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 13 Dic 2016, 12:12
por SeiTaN
Lortropic si no me equivoco eso está ahí por retrocompatibilidad con otros canales, si quieres evitar cualquier problema, le puedes cambiar el nombre a la variable "extra", con las modificaciones que se hicieron puedes usar cualquier nombre de variable.
"extra="..." por "mi_variable="..."
De este modo no habría ningún problema.
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 13 Dic 2016, 19:50
por Lortropic
Oki, en otro canal que tengo ya usé una inventada "letter", pero por si acaso daba algún problema al guardar en biblioteca o algo usé extra, que se que es un campo.
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 17 Dic 2016, 12:51
por identidad_oculta
Hola, vistos los problemas con los archivos tvshow.nfo y algunos problemas misterioso con los channel_data.json (se me han vaciado unos cuantos sin razón alguna) he implementado unas "herramientas" para poder solucionar esos problemas desde Kodi (al menos). Probablemente se pueda hacer de una forma mucho mas sencilla, pero esos requería esperar a los maestros, así que como parche temporal valdrá

. Para no llenar demasiado el 'mainlist' de "Configuración" he creado un sub-menu con las siguientes funciones:
- Activar/Desactivar canales (ya estaba implementada)
- Comprobar que los archivos channel_data.json existan o que no estén vacíos, y de los contrario crearlos o modificarlos, insertando los ajustes por defecto
- Comprobar que los parámetros ‘action’ y ‘get_temporadas’ de cada tvshow.nfo sean los correctos para la version actual, y sino los modifica
A mi me han servido de mucho, y supongo que a los demás tambien les servirá, especialmente el punto 1 y 3. Os dejo el commit por si podéis probarlo/mejorarlo, ya que a mi no me he dado problemas, pero tampoco tengo mas tiempo para hacer pruebas a fondo.
https://github.com/[identidad_oculta]/p ... 269f61a8fc
NOTA 1: Funciona correctamente en Mac OS X y iOS con Kodi 16.1.
NOTA 2: Esta editado sobre el ultimo PR de jesus a la rama 'develop' oficial. Para probarlo solo hace falta copiar y pegar
Un saludo.
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 19 Dic 2016, 00:09
por SeiTaN
Gracias [identidad_oculta].
- Activar/Desactivar canales (ya estaba implementada) <-- estoy de acuerdo de que se implemente, para poder activar o desactivar canales.
- Comprobar que los archivos channel_data.json existan o que no estén vacíos, y de los contrario crearlos o modificarlos, insertando los ajustes por defecto. <-- que ventajas hay con esto? hay canales que dan fallos?
- Comprobar que los parámetros ‘action’ y ‘get_temporadas’ de cada tvshow.nfo sean los correctos para la version actual, y sino los modifica. <-- esto ya hizo el commit superberny para el arreglo cuando se hace la conversión de 4.1.4 a 4.2.0
Sobre el primer punto cuando le he dado para "activar/desactivar" me sale un cuadro de opciones con tres únicas opciones:
- activar todos.
- desactivar todos
- recuperar estado por defecto
Creo que lo ideal sería como se ve en el caso del buscador global.

Que salga en color azul cuando está activo (marcado) y gris cuando está desactivado (desmarcado)
Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 19 Dic 2016, 21:31
por identidad_oculta
SeiTaN escribió:Gracias [identidad_oculta].
- Activar/Desactivar canales (ya estaba implementada) <-- estoy de acuerdo de que se implemente, para poder activar o desactivar canales.
- Comprobar que los archivos channel_data.json existan o que no estén vacíos, y de los contrario crearlos o modificarlos, insertando los ajustes por defecto. <-- que ventajas hay con esto? hay canales que dan fallos?
- Comprobar que los parámetros ‘action’ y ‘get_temporadas’ de cada tvshow.nfo sean los correctos para la version actual, y sino los modifica. <-- esto ya hizo el commit superberny para el arreglo cuando se hace la conversión de 4.1.4 a 4.2.0
Sobre el primer punto cuando le he dado para "activar/desactivar" me sale un cuadro de opciones con tres únicas opciones:
- activar todos.
- desactivar todos
- recuperar estado por defecto
Creo que lo ideal sería como se ve en el caso del buscador global.

Que salga en color azul cuando está activo (marcado) y gris cuando está desactivado (desmarcado)
Esa fue mi idea al principio, pero ya comente que esas ventanas NO FUNCIONAN en el iPad, no me puedo desplazar por ellas, mientras que con el metodo mas rudimentario que he elegido funciona en todos lados. Innovar esta muy bien, pero si un metodo un poco mas "feo" funciona con el 100% de los dispositivos mientras que el nuevo no es compatible con todos, entonces la supuesta innovacion deja de serlo.
Tambien tengo que copiar los archivos xml de resources/skin/default/720p a otra llamada "pal" (o algo por el estilo), sino ni siquiera se abren dichas ventanas.
A raíz de todo esto llevo un tiempo pensando en hacer un fork de pelisalacarta con unos cambios para evitar esas nuevas ventanas y usar las de Kodi, e intentar que sea mas especifico para Kodi, pero no hay tiempo libre para ello y no tengo conocimientos para ser eficiente en el tema.
Lo de tvshows.nfo ya esta quitado. Me hizo gracia ver que el sustituto de la "herramienta" fueron 4 lineas, para la proxima mejor espero.
Respecto a lo de channel_data.json, algunos se quedaron en blanco, y daba constantemente un error de tipo "os" por que los archivos estaban en blanco, o incluso no existían, ya que algunos tenían la terminación ".bk". Curioso. La herramienta solo comprueba que existen y si estan vacíos. Sino existen o están vacíos se cargan los ajustes por defecto del xml del canal y se escriben en su correspondiente *_data.json. Esto se podría incluir en el proceso de actualización de la biblioteca y asi me deshago de esta herramienta tambien. Seguro que son otras 4 lineas

Re: Vamos a por pelisalacarta 4.2 beta 1
Publicado: 19 Dic 2016, 21:59
por SeiTaN
Esa fue mi idea al principio, pero ya comente que esas ventanas NO FUNCIONAN en el iPad, no me puedo desplazar por ellas
Lo tenía pendiente, pero como no tengo iPad no se me ha ocurrido que probar. Si quieres los podemos ver juntos
Tambien tengo que copiar los archivos xml de resources/skin/default/720p a otra llamada "pal" (o algo por el estilo), sino ni siquiera se abren dichas ventanas.
Esto en teoría estaba arreglado, el problema de los skins ¿solo te pasa con el iPad?
Respecto a lo de channel_data.json, algunos se quedaron en blanco, y daba constantemente un error de tipo "os" por que los archivos estaban en blanco, o incluso no existían, ya que algunos tenían la terminación ".bk". Curioso. La herramienta solo comprueba que existen y si estan vacíos. Sino existen o están vacíos se cargan los ajustes por defecto del xml del canal y se escriben en su correspondiente *_data.json. Esto se podría incluir en el proceso de actualización de la biblioteca y asi me deshago de esta herramienta tambien. Seguro que son otras 4 lineas

Creía que esto tb está arreglado, podrías casos específicos y logs? La extensión .bk creo que es cosa mía en filtertools y renumbertools, pero tb se debería haber creado tb el fichero .json.
Gracias como siempre
