Darkensses / mexwe-mcr

Sofifa stats scrapper to convert into mcr psx for WE2002
https://mexwemcr.netlify.com/
5 stars 4 forks source link

El resultado de Stats() es incorrecto debido a valores repetidos #2

Closed inrumi closed 5 years ago

inrumi commented 5 years ago

https://github.com/Darkensses/sofifa-psx-react/blob/674c4b8be8dbf11542d035a8658003498b4e5718/src/api/mcr.js#L330

Indagando un poco en el codigo, me encontre con esta suma, intente reducir los valores de los parametros que vienen el objeto player pero me di cuenta que solo se suman ciertos valores.

Me gustaria resolver este issue, pero tambien quisiera saber si estos son todos los stats que se deben sumar de player

Darkensses commented 5 years ago

Gracias por levantar el Issue 🎈🐞 el stat de stamina debería sumarse una vez y no dos como actualmente está.

En cuanto a los stats que se deben de sumar son esos (son 16) y nada más. Esta formula esta estipulada en el documento de excel de polipoli. Añado captura solo para ilustrar.

Agradecería un PR eliminando el stat duplicado de stamina y los ajustes que sean necesarios.

Gracias!

image

image

Diego-Pino commented 5 years ago

Gracias @inrumi

Darkensses commented 5 years ago

Issue Resuelto en #3 🎉