ereuse/devicehub-django
8
0
Fork 0
Code Issues 6 Pull requests 6 Packages Projects Releases Wiki Activity

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
pull from: fix/109-small-refactors-public-website
merge into: ereuse:main
Conversation 2 Commits 2 Files changed 2 +36 -34
sergiogimenez commented 2024-11-19 07:12:18 +00:00
Owner
Copy link

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
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
cayop commented 2024-12-09 11:49:01 +00:00
Owner
Copy link

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.

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.
sergiogimenez commented 2024-12-12 06:07:19 +00:00
Author
Owner
Copy link

Ok para cuando se debe renderizar en la vista, pero para cuando se tiene que devolver el JSON? Como podemos mover esta lógica en la plantilla?

Ok para cuando se debe renderizar en la vista, pero para cuando se tiene que devolver el JSON? Como podemos mover esta lógica en la plantilla?
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/109-small-refactors-public-website:fix/109-small-refactors-public-website
git checkout fix/109-small-refactors-public-website

Merge

Merge the changes and update on Forgejo.
git checkout main
git merge --no-ff fix/109-small-refactors-public-website
git checkout main
git merge --ff-only fix/109-small-refactors-public-website
git checkout fix/109-small-refactors-public-website
git rebase main
git checkout main
git merge --no-ff fix/109-small-refactors-public-website
git checkout main
git merge --squash fix/109-small-refactors-public-website
git checkout main
git merge --ff-only fix/109-small-refactors-public-website
git checkout main
git merge fix/109-small-refactors-public-website
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
cayop
2 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
No description provided.
Delete branch "fix/109-small-refactors-public-website"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?