move logic to model and remove potential harmful code (void useing del) #32

sergiogimenez · 2024-11-19T07:12:18Z

sergiogimenez commented

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

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

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

This pull request can be merged automatically.

You are not authorized to merge this pull request.

You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.

git checkout -b fix/109-small-refactors-public-website main

git pull origin fix/109-small-refactors-public-website

Step 2:

Merge the changes and update on Gitea.

git checkout main

git merge --no-ff fix/109-small-refactors-public-website

git push origin main

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: ereuse/devicehub-django#32