FilmAffinity (ES): version 2.82.Compatible con Ant Movie 4.2

If you made a script you can offer it to the others here, or ask help to improve it. You can also report here bugs & problems with existing scripts.
JuanRa_007
Posts: 21
Joined: 2014-06-08 09:08:31

Post by JuanRa_007 »

Arturo wrote:Efectivamente han cambiado la codificación de la pagina de ISO a utf8.
Me ha pillado de vacaciones. Solo he tenido un ratito para modificarlo. Creo que funciona pero no he podido hacer pruebas.
Dejo pues la versión 2.77 en el primer post.
Probadlo y ya direis. y si está bien que antp lo suba.
Cuando vuelva le daré un repaso.
Muchas gracias por solucionarlo tan rápido.

Arturo wrote:He añadido un nueva opción: GroupImages. Si tiene valor 1 agrupa todas las imágenes en una categoria uníca (como hasta ahora). Si está a 0 las divide en las categorias que tiene la página, tal como decia JuanRa_007
Y gracias por añadir esa "mejora"!!!
carlosmg2
Posts: 39
Joined: 2009-08-18 19:21:08
Contact:

Post by carlosmg2 »

genial, ahora funciona perfecto. Muchas gracias!!
jacqlittle
Posts: 26
Joined: 2013-04-11 07:13:02

Post by jacqlittle »

Lo he probado con bastantes películas y no detecto ninguna anomalía.

Yo creo que si lo lee antp que lo suba al servidor, si alguien detectara algún posible error siempre se podrá corregir posteriormente, si es que fallara alguna cosilla, aunque con la cantidad de películas que he importado no creo, y siempre será mejor tener en el servidor un script que por lo menos es operativo, que el actual con los cambios en la web de Filmaffinity no lo es...

Muchísimas gracias como siempre Arturo, que eres un auténtico crack!!! ;)

Saludos.
antp
Site Admin
Posts: 9637
Joined: 2002-05-30 10:13:07
Location: Brussels
Contact:

Post by antp »

I hope I understood everything correctly with Google Translate :D I updated the script on the server with the version 2.77 from the first post.
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

Problema con actores

Post by raulsara »

Hola Arturo, he probado el script y a veces no enseña los actores, en pelis como Grand Ecole, Le sable, .... He mirado el script y no he visto nada raro.
En todos los casos envía los actores en SetField(fieldActors......

Saludos
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

filmaffinity

Post by raulsara »

Parece ser que pasa cuando los actores tienen algún carácter especial como acentos, dieresis o caracteres raros.
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

Arreglado

Post by raulsara »

Si en el comando que carga los actores lo dejais así,

SetField(fieldActors, Item);

Creo que ya funciona.

Si no hay mas cosas ya se le puede pasar a antp.

Ya direis algo

Saludos
masterchipo
Posts: 100
Joined: 2010-02-22 23:15:05

Post by masterchipo »

No sé como cambiarlo y me pasa lo mismo, no carga los actores.
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

No carga actores

Post by raulsara »

Hola, esta tarde le pasare el script que hizo Arturo con el cambio para que salgan los actores arreglado a antp para que os lo bajeis.

Si lo necesitas ya, modifica el script modificando la linea del setfield(fieldactors de la siguiente manera.

SetField(fieldActors, Item);

Aqui no tengo el programa pero de memoria te diría que tienes que ir a Herramientas / Script . Selecciona filmaffinity, le das a editar, modificas la linea y salvas.

Saludos
Arturo
Posts: 52
Joined: 2013-04-11 16:28:52

Re: No carga actores

Post by Arturo »

raulsara wrote:Hola, esta tarde le pasare el script que hizo Arturo con el cambio para que salgan los actores arreglado a antp para que os lo bajeis.

Si lo necesitas ya, modifica el script modificando la linea del setfield(fieldactors de la siguiente manera.

SetField(fieldActors, Item);

Aqui no tengo el programa pero de memoria te diría que tienes que ir a Herramientas / Script . Selecciona filmaffinity, le das a editar, modificas la linea y salvas.

Saludos
Ojo el cambio no debe ser este. Lo que ha pasado es que con poco tiempo para parchear me lié a poner decodificaciones de UTF8 y ocurre que en actores se le puede aplicar dos vees la conversión dependiendo del valor de ActorsInALine. y es lo que provoca el error.

raulsara: creo que lo correcto seria sustituir:

Code: Select all

if GetOption('ActorsInALine') = 0 then Item := StringReplace(ClUTF8(Item), ', ', #13#10);
    SetField(fieldActors, ClUTF8(Item));  
por

Code: Select all

if GetOption('ActorsInALine') = 0 then Item := StringReplace(Item, ', ', #13#10);
    SetField(fieldActors, ClUTF8(Item));
Yo ahora no tengo medios para hacerlo.

Saludos a todos
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

Post by raulsara »

Hola Arturo, el cambio que hice lo había probado con mas de 100 pelis suecas, noruegas, francesas con caracteres raros en el nombre de los actores y funcionaba bien.

He probado tus cambios y también funcionan bien, tu solución es mas correcta, le vuelvo a enviar el script definitivo con tus cambios a antp para que lo publique.

Por cierto le he añadido una clasificación de las pelis por orden alfabético en vez de por años. Cuando selecciona muchas pelis es mas laborioso buscar por años que por orden alfabético. Si no os gusta lo cambiaré pero creo que va mejor.

Gracias por tu colaboración.
Arturo
Posts: 52
Joined: 2013-04-11 16:28:52

Post by Arturo »

oK raulsara, yo lo probare cuando pueda.
La cuestión sobre la decodificación de UTF8, por lo que se ve (no puedo probarlo ahora) es que si se aplica sobre texto ya decodificado el resultado es una cadena vacia. Creo que lo mejor es aplicarlo al final cuando ya se va a asignar al campo. En ese caso en concreto tal como estaba, si ActorsInALine vale 0 se aplicaba dos veces, pero si ActorsInALine vale 1 solo se aplicaba 1 vez. Y por lo menos debe aplicarse 1 vez, de lo contrario quienes tengan la opción de actores separados por comas obtendrán caracteres raros.


Por cierto pude que haya algun otro sitio donde pueda producirte doble decodificación o alguno donde la falte.

Un saludo.
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

Post by raulsara »

Hola me comenta Antp que ya lo ha subido a la web. Ya os lo podéis bajar.

Es la versión 2.78

Cualquier error lo reportáis al foro.

Gracias a todos por vuestra colaboración.
al0203
Posts: 72
Joined: 2013-02-27 05:24:17

Post by al0203 »

raulsara wrote:Hola Arturo, el cambio que hice lo había probado con mas de 100 pelis suecas, noruegas, francesas con caracteres raros en el nombre de los actores y funcionaba bien.

He probado tus cambios y también funcionan bien, tu solución es mas correcta, le vuelvo a enviar el script definitivo con tus cambios a antp para que lo publique.

Por cierto le he añadido una clasificación de las pelis por orden alfabético en vez de por años. Cuando selecciona muchas pelis es mas laborioso buscar por años que por orden alfabético. Si no os gusta lo cambiaré pero creo que va mejor.

Gracias por tu colaboración.
Te importaría explicarme como hacer para que las búsquedas las ordene por fecha y no por orden alfabético. A mi me pasa lo contrario que a ti, por orden alfabético me complica mucho encontrar las películas.

Gracias.
jacqlittle
Posts: 26
Joined: 2013-04-11 07:13:02

Post by jacqlittle »

+1

Yo también coincido contigo, para buscar películas a mi también me convence más por fecha, es decir, como estaba antes...

Saludos.
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

Post by raulsara »

Hola para que lo ordene por fechas únicamente hay que quitar del script el comando PickTreeSort, borrando esta linea ya lo ordena por fecha.

De todas maneras si os va mejor por fecha, en el próximo cambio que haya que hacer en el script se quita el comando para que clasifique por fecha en vez de alfabéticamente.

Saludos
carlosmg2
Posts: 39
Joined: 2009-08-18 19:21:08
Contact:

Post by carlosmg2 »

raulsara wrote:Hola para que lo ordene por fechas únicamente hay que quitar del script el comando PickTreeSort, borrando esta linea ya lo ordena por fecha.

De todas maneras si os va mejor por fecha, en el próximo cambio que haya que hacer en el script se quita el comando para que clasifique por fecha en vez de alfabéticamente.

Saludos
yo también prefiero que se ordene por fecha. podrías ponerlo como opción si tienes tiempo, seria añadir un if mas en algún sitio...

y una sugerencia: las funciones FindLine, TextBetween y TextAfter estan incluidas en StringUtils1.pas. poniendo un "uses StringUtils1;" donde la declaración de variables podríais ahorraros las 3 funciones. (si lo sabéis y lo habeis hecho a proposito por alguna razon no hacerme caso :)
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

Post by raulsara »

Hola Carlos, vistos los comentarios, esta tarde le paso a antp el script con la clasificación por fechas que parece que os va mejor. Por si alguien lo quiere alfabético lo dejaré documentado dentro del script.

Lo de las funciones ya lo ví pero me pareció ver algunas diferencias entre las que hay en el Script de filmaffinity y las que hay en Stringutils1 y preferí dejarlas por si acaso. De todas maneras le echaré un vistazo esta tarde.

Gracias por tus comentarios

Saludos
raulsara
Posts: 54
Joined: 2006-02-13 14:05:04

Arreglado

Post by raulsara »

Ya se ha publicado la versión 2.79 con el sistema de clasificación por año. Ya no ordena por orden alfabético sino por año tal y como lo hace la web.

Si veis cualquier mejora no dudéis en publicarlo en el foro.

Saludos
Keichi
Posts: 5
Joined: 2013-04-09 14:49:30

Post by Keichi »

He instalado el nuevo script pero me da un error:
SetOption('DontWantExtras', 1);
¿Alguna solución? Gracias.
Post Reply