search

doofinder / doofinder-magento2

Open Software License 3.0
10 stars 8 forks source link

PLUGINRANGERS-2486 | Changed getLayerScript method to fix a critical issue #331

Closed davidmolinacano closed 1 month ago

davidmolinacano commented 1 month ago

Required by:

This version only adds the extra doofinderApp data in case it comes from a brand new installation which may not include this part. However, for migrated ones that already includes it directly in the database, it won't be added twice.

image

In case it's a brand new installation, an additional data will be added on the script tag to be able to discern the cases more easily

davidmolinacano commented 1 month ago

@davidmolinacano Esto no esta agregando el doofinderApp script a la db... Me gustaría que fueran consistentes las entradas en la db, o que todas tengan el doofinderApp o que ninguna lo tenga. Que te parece?

Voy a intentarlo, pero no prometo tenerlo a tiempo de hacerlo así, descifrar Magento es complejo :sweat_smile:

davidmolinacano commented 1 month ago

Parece que funciona con la versión antigua también:

image

Inyecta currency y language