Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reemplazar llamada a Registry (deprecado) #17

Open
matias179 opened this issue Mar 11, 2020 · 3 comments
Open

Reemplazar llamada a Registry (deprecado) #17

matias179 opened this issue Mar 11, 2020 · 3 comments

Comments

@matias179
Copy link

Se usa el registry para obtener la order actual en el bloque app/code/Mugar/CustomerIdentificationDocument/Block/Order/CidFields.php

@olivertar
Copy link
Contributor

El metodo figura como deprecado pero Magento continua utilizandolo....
Magento\Sales\Block\Order\View.php

@matias179 tenes idea como deberiamos proceder con esto?

@matias179
Copy link
Author

Si, hay mucho de "haz lo que yo digo y no lo que yo hago" en el propio core de Magento.
En este post explican un poco como cambiarlo:
https://www.atwix.com/magento-2/alternatives-for-deprecated-registry-class-magento-2-3/
Y hay un modulo de Vinai Kopp donde tira un ejemplo:
https://github.com/Vinai/module-current-product-example
Tendriamos que buscarle la vuelta para ver como adaptarlo aca

@matias179
Copy link
Author

Ahi comitee un fix para no usar el registry, use la misma forma de revisar la orden que tienen en el controller de Magento antes de setear el valor.
Asi al menos nos evitamos usar la funcion deprecada.
Tambien meti mano y saque los bloques, usando un viewModel, asi evitamos tener codigo duplicado usando bloques que no son necesarios

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants