Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
K
kohinos-tav
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • agplv3
  • kohinos-tav
  • Merge Requests
  • !58

Merged
Opened Feb 29, 2024 by Damien Moulard@DamienM 
  • Report abuse
Report abuse

allow superadmin to delete users in tav env + fix impersonating

allow superadmin to delete users in tav env + fix impersonating + return to preventing impersonating in prod envs

Edited Feb 29, 2024 by Damien Moulard
  • Discussion 8
  • Commits 5
  • Changes 5
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard @DamienM

    added 1 commit

    • aca3a2c6 - re-allow superadmin impersonating

    Compare with previous version

    Feb 29, 2024

    added 1 commit

    • aca3a2c6 - re-allow superadmin impersonating

    Compare with previous version

    added 1 commit <ul><li>aca3a2c6 - re-allow superadmin impersonating</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/58/diffs?diff_id=2824&start_sha=099c7f78a28dc01a048fb0867d1df04494e77040)
    Toggle commit list
  • Damien Moulard @DamienM

    changed title from allow superadmin to delete users in tav env + fix impersonating + return to… to allow superadmin to delete users in tav env + fix impersonating

    Feb 29, 2024

    changed title from allow superadmin to delete users in tav env + fix impersonating + return to… to allow superadmin to delete users in tav env + fix impersonating

    changed title from **allow superadmin to delete users in tav env + fix impersonating{- + return to…-}** to **allow superadmin to delete users in tav env + fix impersonating**
    Toggle commit list
  • Yvon Kerdoncuff
    @Yvon started a discussion Feb 29, 2024
    Resolved by Yvon Kerdoncuff Apr 08, 2024
    • Yvon Kerdoncuff @Yvon commented Feb 29, 2024
      Master

      Pour moi c'est OK, même si philosophiquement ça me paraît bizarre que le rôle adhérent ait un statut spécial par rapport aux autres rôles. Par exemple, le fait que la suppression d'un utilisateur supprime l'adhérent et pas le prestataire.

      D'autre part, comme le montrent les deux screenshots dans le ticket, la suppression échoue dès lors qu'il y a au moins un flux qui concerne le user (ou l'adhérent ?). Du coup cette fonctionnalité va être assez limitée.

      Si c'est OK pour toi tu peux merger.

      Pour moi c'est OK, même si philosophiquement ça me paraît bizarre que le rôle adhérent ait un statut spécial par rapport aux autres rôles. Par exemple, le fait que la suppression d'un utilisateur supprime l'adhérent et pas le prestataire. D'autre part, comme le montrent les deux screenshots dans le ticket, la suppression échoue dès lors qu'il y a au moins un flux qui concerne le user (ou l'adhérent ?). Du coup cette fonctionnalité va être assez limitée. Si c'est OK pour toi tu peux merger.
    • Damien Moulard @DamienM commented Mar 04, 2024
      Master

      Pour moi c'est OK, même si philosophiquement ça me paraît bizarre que le rôle adhérent ait un statut spécial par rapport aux autres rôles. Par exemple, le fait que la suppression d'un utilisateur supprime l'adhérent et pas le prestataire.

      Pour moi c'est normal, car un compte adhérent est en OneToOne avec User (un compte Adhérent n'existe pas sans un compte User), alors qu'un compte Prestataire par exemple peut être géré par plusieurs utilisateurs (OneToMany).

      Edited Apr 04, 2024 by Damien Moulard
      > Pour moi c'est OK, même si philosophiquement ça me paraît bizarre que le rôle adhérent ait un statut spécial par rapport aux autres rôles. Par exemple, le fait que la suppression d'un utilisateur supprime l'adhérent et pas le prestataire. Pour moi c'est normal, car un compte adhérent est en OneToOne avec User (un compte Adhérent n'existe pas sans un compte User), alors qu'un compte Prestataire par exemple peut être géré par plusieurs utilisateurs (OneToMany).
    • Damien Moulard @DamienM commented Mar 04, 2024
      Master

      D'autre part, comme le montrent les deux screenshots dans le ticket, la suppression échoue dès lors qu'il y a au moins un flux qui concerne le user (ou l'adhérent ?). Du coup cette fonctionnalité va être assez limitée.

      Après discution avec Charles, ce n'est pas gênant pour l'utilisation prévue. Je vais ajouter un message avertissant qu'un utilisateur dont l'adhérent a des flux associés ne pourra pas être supprimé.

      > D'autre part, comme le montrent les deux screenshots dans le ticket, la suppression échoue dès lors qu'il y a au moins un flux qui concerne le user (ou l'adhérent ?). Du coup cette fonctionnalité va être assez limitée. Après discution avec Charles, ce n'est pas gênant pour l'utilisation prévue. Je vais ajouter un message avertissant qu'un utilisateur dont l'adhérent a des flux associés ne pourra pas être supprimé.
    Please register or sign in to reply
  • Yvon Kerdoncuff
    @Yvon started a discussion Feb 29, 2024
    Resolved by Yvon Kerdoncuff Apr 08, 2024
    • Yvon Kerdoncuff @Yvon commented Feb 29, 2024
      Master

      En relisant les attentes de Charles, je vois aussi qu'il souhaite que :

      "Que l'utilisateur soit du Groupe Adhérent, Prestataire, Comptoir... l'utilisateur ne devra plus se retrouver dans la liste relative à son groupe."

      Du coup, peut-être faut-il d'abord vérifier avec lui que c'est OK de ne pas aller supprimer les autres rôles ?

      En relisant les attentes de Charles, je vois aussi qu'il souhaite que : "Que l'utilisateur soit du Groupe Adhérent, Prestataire, Comptoir... l'utilisateur ne devra plus se retrouver dans la liste relative à son groupe." Du coup, peut-être faut-il d'abord vérifier avec lui que c'est OK de ne pas aller supprimer les autres rôles ?
    • Damien Moulard @DamienM commented Mar 04, 2024
      Master

      C'est ok pour ça, c'est bien les comptes utilisateurs qui doivent être supprimés.

      Edited Mar 04, 2024 by Damien Moulard
      C'est ok pour ça, c'est bien les comptes utilisateurs qui doivent être supprimés.
    Please register or sign in to reply
  • Damien Moulard @DamienM

    added 1 commit

    • 20f4c1e3 - user admin: prevent deleting user if its adherent account has flux

    Compare with previous version

    Apr 04, 2024

    added 1 commit

    • 20f4c1e3 - user admin: prevent deleting user if its adherent account has flux

    Compare with previous version

    added 1 commit <ul><li>20f4c1e3 - user admin: prevent deleting user if its adherent account has flux</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/58/diffs?diff_id=2916&start_sha=aca3a2c63bf1d6b8fe8a20caf777fb442a708352)
    Toggle commit list
  • Yvon Kerdoncuff
    @Yvon started a discussion on an outdated change in commit 20f4c1e3 Apr 08, 2024
    Last updated by Damien Moulard May 02, 2024
    src/Controller/CRUD/CRUDController.php
    220 231 return $this->redirectTo($object);
    221 232 }
    222 233
    234 // Prevent deleting user if flux related to its Adherent account exist
    235 if ($object instanceof User && $object->getAdherent()) {
    236 $query = $this->em->getRepository(Flux::class)->getQueryByAdherent($object->getAdherent());
    • Yvon Kerdoncuff @Yvon commented Apr 08, 2024
      Master

      Même si aucun flux n'est lié à l'adhérent, il peut y avoir un flux lié au user, par exemple dans le cas d'une cotisation, ou bien, plus réaliste pour nous, dans le cas d'un achat de monnaie adhérent (vérifié sur prod tav : c'est le format classique d'un flux de type achat de monnaie).

      image image image image image

      Edited Apr 08, 2024 by Yvon Kerdoncuff
      Même si aucun flux n'est lié à l'adhérent, il peut y avoir un flux lié au user, par exemple dans le cas d'une cotisation, ou bien, plus réaliste pour nous, dans le cas d'un achat de monnaie adhérent (vérifié sur prod tav : c'est le format classique d'un flux de type achat de monnaie). ![image](/uploads/a5236ec8521238f2d07e9ed0b8e08ebe/image.png) ![image](/uploads/48e89de2f876398f3b034cf2f19bcd0f/image.png) ![image](/uploads/95dd85270f74bcdd4f8f93d17aca8646/image.png) ![image](/uploads/06478cbd868c6004bed559ef0267d7fa/image.png) ![image](/uploads/71d5af2d922b3a0974b24caa47ba0b61/image.png)
    • Damien Moulard @DamienM commented Apr 08, 2024
      Master

      ok, il faut donc bloquer si un quelconque flux est lié à l'adhérent ou à l'utilisateur

      ok, il faut donc bloquer si un quelconque flux est lié à l'adhérent ou à l'utilisateur
    • Damien Moulard @DamienM

      changed this line in version 5 of the diff

      May 02, 2024

      changed this line in version 5 of the diff

      changed this line in [version 5 of the diff](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/58/diffs?diff_id=2988&start_sha=70f6b7d9d8a1761dbf9fdbd9c4568b092d1eadcf#5fab09d305780b2ed2b64de0de21aaf00630c00c_236_235)
      Toggle commit list
    • Damien Moulard @DamienM commented May 02, 2024
      Master

      corrigé

      corrigé
    Please register or sign in to reply
  • Damien Moulard @DamienM

    added 216 commits

    • 20f4c1e3...b3184aff - 215 commits from branch develop
    • 70f6b7d9 - merge conflicts

    Compare with previous version

    May 02, 2024

    added 216 commits

    • 20f4c1e3...b3184aff - 215 commits from branch develop
    • 70f6b7d9 - merge conflicts

    Compare with previous version

    added 216 commits <ul><li>20f4c1e3...b3184aff - 215 commits from branch <code>develop</code></li><li>70f6b7d9 - merge conflicts</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/58/diffs?diff_id=2987&start_sha=20f4c1e385dee04dfbececb6a0d8e464c1131987)
    Toggle commit list
  • Damien Moulard @DamienM

    added 1 commit

    • 89114128 - prevent deleting user if has flux to its account OR its adherent account

    Compare with previous version

    May 02, 2024

    added 1 commit

    • 89114128 - prevent deleting user if has flux to its account OR its adherent account

    Compare with previous version

    added 1 commit <ul><li>89114128 - prevent deleting user if has flux to its account OR its adherent account</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/58/diffs?diff_id=2988&start_sha=70f6b7d9d8a1761dbf9fdbd9c4568b092d1eadcf)
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    merged

    May 06, 2024

    merged

    merged
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    mentioned in commit e6e07f16

    May 06, 2024

    mentioned in commit e6e07f16

    mentioned in commit e6e07f161507763c012178fc7ae00eafa0d2f1b5
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
Reference: agplv3/kohinos-tav!58
×

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.