search

allnnde / pf2e-esp-translation

Traduccion no oficial del modulo de PF2E en Foundry vtt
Other
4 stars 5 forks source link

Better path resolution #209

Closed vitaly-t closed 3 months ago

vitaly-t commented 3 months ago

https://github.com/allnnde/pf2e-esp-translation/blob/aeb68c586f6d22a4df9c393dc930a054291159eb/src/helper/pack-extractor/pack-extractor.js#L625

it is better to write the code like this:

const p = resolvePath(obj, path);
if(p.exists) {
    const value = p.value; // use the value if exists
}

Otherwise, you are resolving all values twice ;)

allnnde commented 3 months ago

hola, entiendo tu punto, pero todo lo que es extraccion de los archivos es una copia de como se hace en el repositorio del idioma aleman, por ende no voy a modificar el codigo a no ser que sea por algo critico.

Gracias por la sugerencia