move logic to model and remove potential harmful code (avoid using del) #32

Open

sergiogimenez wants to merge 2 commits from fix/109-small-refactors-public-website into main

sergiogimenez commented

2024-11-19 07:12:18 +00:00

Owner

Este pull request closes https://gitea.pangea.org/ereuse/projectes/issues/109.

He movido la lógica al modelo dejando en la vista solo funciones que devuelven la respuesta o bien vía JSON o bien vía contexto al template.
He reimplementado la manera en la cual se devuelve la información sensible. En resumen, si el usuario NO está autenticado, se hace una copia de la evidencia para evitar tocar la original, y se quitan los atributos sensibles.
- Quizás podemos repensar este segundo punto, porque hacer una copia de toda la evidencia es poco eficiente. Por ahora no se me ocurre otra alternativa, pero podemos darle una vuelta a ver si se nos ocurriera otra :)

Este pull request closes https://gitea.pangea.org/ereuse/projectes/issues/109. * He movido la lógica al modelo dejando en la vista solo funciones que devuelven la respuesta o bien vía JSON o bien vía contexto al template. * He reimplementado la manera en la cual se devuelve la información sensible. En resumen, si el usuario NO está autenticado, se hace una copia de la evidencia para evitar tocar la original, y se quitan los atributos sensibles. * Quizás podemos repensar este segundo punto, porque hacer una copia de toda la evidencia es poco eficiente. Por ahora no se me ocurre otra alternativa, pero podemos darle una vuelta a ver si se nos ocurriera otra :)

cayop was assigned by sergiogimenez

2024-11-19 07:12:18 +00:00

sergiogimenez added 1 commit 2024-11-19 07:12:19 +00:00

move logic to model and remove potential harmful code (void useing del) 2813a095de

sergiogimenez changed title from ~~move logic to model and remove potential harmful code (void useing del)~~ to move logic to model and remove potential harmful code (avoid using del)

2024-12-05 08:01:56 +00:00

sergiogimenez added 1 commit 2024-12-05 08:14:43 +00:00

Using a shallow copy instead of ineficient json conversion 9fa8cd57ba

cayop commented

2024-12-09 11:49:01 +00:00

Owner

Hay otra forma que te quita el problema del de copiar las evidencias. La forma que se me ocurre es pasar la logica a la plantilla. Osea la plantilla es la que renderiza o no la información dependiendo de que rol tiene ese usuario. Asi no se toca la información base de la evidencia.