Dudas sobre mis primeros aportes y otras cosas

[SeiTaN]

El codigo no lo he mirado mucho, pero se ve que funciona perfecto

[identidad_oculta]

El codigo no lo he mirado mucho, pero se ve que funciona perfecto
...

Me alegro, gracias por probarlo.

Como hoy no he estado muy atareado he creado tambien la ventana para establecer los servidores que a cada uno le gustan ("whitelist" en settings.xml), con un método menos manual que el que había hasta ahora. A ver si me explico bien y se entiende como lo he hecho y mi forma de verlo:
En este caso le he pedido que lea los "*.xml" de la carpeta "servers". A los que no están activos los he puesto en la parte de abajo de la lista en color gris y al resto antes de estos. Si no estaban especificados los valores "whitelist" y/o "blacklist" en "settings.xml", a mi modo de verlo era como si formaran parte de "whitelist", porque eran servidores que quedaban activados, así que, salvo que se deseleccione algún servidor, tras pulsar sobre "OK" todos pasan a formar parte de "whitelist". Si se deselecciona algún servidor pasa al "blacklist", y como es obvio, tambien lee los valores de "whitelist" y "blacklist" del archivo "settings.xml". Para guardar los valores lo primero que se hace es borrar los valores anteriores de "whitelist" y "blacklist", lo cual me ha parecido bastante radical cuando lo he pensado, pero computacionalmente lo veo menos costoso que comprobar si los nuevos ajustes se parecen a los que ya estaban aplicados, aunque pudiera quedar mas "bonito".

Os dejo el enlace al commit y al archivo configuracion.py completo.

Problemas/dudas que he encontrado:

• En la ventana la barra de desplazamiento se vuelve enorme y se "sale" de la pantalla, supongo que es por que la lista es demasiado grande.
• En el log estoy viendo constantemente el error

  Código: Seleccionar todo

  WARNING: CSkinInfo: failed to load skin settings

• Y este error tambien lo da constantemente:

  Código: Seleccionar todo

  ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                               - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                              Error Type: <type 'exceptions.KeyError'>
                                              Error Contents: ('default',)
                                              Traceback (most recent call last):
                                                File "/Users/-----/Library/Application Support/Kodi/addons/plugin.video.pelisalacarta/platformcode/xbmc_config_menu.py", line 831, in onClick
                                                  self.check_default()
                                                File "/Users/-----/Library/Application Support/Kodi/addons/plugin.video.pelisalacarta/platformcode/xbmc_config_menu.py", line 741, in check_default
                                                  def_values = dict([[c["id"], c["default"]] for c in self.controls])
                                              KeyError: ('default',)

• El nombre "Ajustes de servidores: Filtro" no me termina de convencer, y tampoco se si la posición en la que lo he puesto es la mas adecuada.
• Que todos los servidores estén en "whitelist" y/o "blacklist" puede provocar algún tipo de cuello de botella al tener que pasar todos los servidores por el "launcher.py" para ser filtrados?

Como siempre, agradezco cualquier respuesta.

Un saludo!!

[SeiTaN]

Una cosa que me he dado cuenta respecto al desarrollo anterior ¿por que agregas esto a todos los canales?

Código: Seleccionar todo

"needsusrandpass": "no"

Si ya los recorres, y tienes los dos valores, entonces lo agregas al diccionario que corresponda (con uno valdría y dependiendo del tipo le cambias el color)
Luego le doy una vuelta y te pongo mi código a ver que te parece.

Lo de lo servidores en "modo simple" fue Jesús quien lo quiso asi, poniendo un cuadro de texto, yo había pensado agregar una opción por cada servidor en la configuración global, pero como resulta evidente queda inmenso y se quedó asi.

[identidad_oculta]

Una cosa que me he dado cuenta respecto al desarrollo anterior ¿por que agregas esto a todos los canales?

Código: Seleccionar todo

"needsusrandpass": "no"

Si ya los recorres, y tienes los dos valores, entonces lo agregas al diccionario que corresponda (con uno valdría y dependiendo del tipo le cambias el color)
Luego le doy una vuelta y te pongo mi código a ver que te parece.

Eso lo hace nada mas empezar, luego busca que canales requieren usuario y contraseña y se cambia el parámetro "needsusrandpass" de "no" a "yes" o "yes_alt" (en función de como este en el xml los datos del usuarios) de esos canales. En la version anterior tambien se hacia, pero esta vez lo he comentado en el código, por si alguien se extrañaba al verlo. Si se añade un nuevo canal tambien buscara si este requiere usuario y contraseña, pero antes supondrá que no la necesita. Es lo que se me ha ocurrido para que el sistema sea compatible con cualquier canal, existente o que este por venir, pero estoy seguro de que existen mejores métodos.
Gracias por echarme una mano, seguro que tu código esta mejor (pero no me dejes por los suelos, por favor )

Lo de lo servidores en "modo simple" fue Jesús quien lo quiso asi, poniendo un cuadro de texto, yo había pensado agregar una opción por cada servidor en la configuración global, pero como resulta evidente queda inmenso y se quedó asi.

Por esto mismo preguntaba. A ver lo que opina jesus del asunto y se modifica lo que haga falta con tal de "aligerar" el asunto.

[SeiTaN]

Gracias por echarme una mano, seguro que tu código esta mejor (pero no me dejes por los suelos, por favor )

No es para nada mi intención

[identidad_oculta]

Segun he visto en un commit de jesus, se ha agregado a "Configuracion" los canales que tienen ajustes. Cuando hace unas semanas me propuso hacer lo de la ventana esa para el usuario/contraseña entendi que solo debia servir para eso. Inserto en la ventana todos los ajustes que tengan los canales salvo el de la seccion novedades y la busqueda global o lo dejo solo con lo de usuario y contraseña??

SeiTaN, me tienes esperando con ansias de ver tu codigo, para asi poder acabar con el asunto y hacer un PR, es una caracteristica que me parece muy interesante que este disponible lo antes posible, y me da la sensacion que la version 4.1.2 esta a la vuelta de la esquina.

Respecto a lo del whitelist y blacklist; Y si en vez de usar un "bool" uso un "list" con las opciones "whitelist", "blacklist" y "neutral"? Por defecto se quedarian todos los canales en "neutral". Esta opcion no me termina de gustar por que los del tipo "list" no funcionan bien en la TV-Box y en las pantallas tactiles, ya que hace falta un raton para pulsar en las flechitas. Se os ocurre alguna otra opcion que no sea usar lo de "list"??

Un saludo y gracias!

[SeiTaN]

Lo siento [identidad_oculta], llevo unos días muy liado en el curro, y lo he dejado pendiente.

No difiera mucho del tuyo, quitando que no añade variable en settings, voy a intentar sacar un ratillo y lo añado.

Un posible bug que vi gracias a tu desarrollo está al guardar a los datos del canal ya que sobrescribe todo el fichero, en la mayoría de canales no afectaría pero en otro como seriesblanco, seriesdanko o animeflv si, tb le echaré un ojo.

[SeiTaN]

https://github.com/SeiTaN/pelisalacarta ... t_kerenmac <-- rama
https://github.com/SeiTaN/pelisalacarta ... 2f76742c4d <-- commit

Solo serían tus dos metodos, el resto se ha cambiado por los cambios de Jesús que se machacan con los tuyos.

Como ves en la información del commit aún faltan cosillas, pero es poco, seguro que se puede mejorar, está hecho en un rato y siguiendo lo que tú querías me ha salido algo asi

Yo creo que lo suyo sería un mix con lo que publicó Jesús y está en la rama 'develop', que en vez de añadir los canales manualmente se buscaran y con parte de lo que te he puesto se podría hacer fácilmente, y ya cuando abres el canal, llamar a su función.

[identidad_oculta]

https://github.com/SeiTaN/pelisalacarta ... t_kerenmac <-- rama
https://github.com/SeiTaN/pelisalacarta ... 2f76742c4d <-- commit

Solo serían tus dos metodos, el resto se ha cambiado por los cambios de Jesús que se machacan con los tuyos.

Como ves en la información del commit aún faltan cosillas, pero es poco, seguro que se puede mejorar, está hecho en un rato y siguiendo lo que tú querías me ha salido algo asi

Yo creo que lo suyo sería un mix con lo que publicó Jesús y está en la rama 'develop', que en vez de añadir los canales manualmente se buscaran y con parte de lo que te he puesto se podría hacer fácilmente, y ya cuando abres el canal, llamar a su función.

No pretendía meter presión, lo primero es lo primero. Me gusta mucho mas tu método, es mucho mas "limpio" que el mío, pero es que muchas de esas lineas que has puesto no sabia que se podían hacer . Si me lo permites, trabajare sobre tu código para extender lo del usuario/contraseña a cualquier ajuste que puedan tener los canales y evitar así ponerlos manualmente.

Para lo del whitelist/blacklist voy a cambiarlo por el tipo "text", "N" sera para que los servidores se queden en neutral (por defecto), "W" para whitelist y "B" para blacklist.

Un saludo y gracias.

PD: Yo tambien ando liado, así que por el momento se quedara lo que hay en la rama "develop", tampoco creo que desagrade a nadie.

[SeiTaN]

Claro que puedes usarlo, me alegro que te guste

Lo de la lista blanca y negra no entiendo lo que dices de poner una opción como "normal".

Lista blanca: sólo permite los servers añadidos.
Lista negra: permite todo menos los servers añadidos.

Si no lo entendí mal es así.