You are not logged in.

Announcement

 Téléchargez la dernière version stable de GLPI      -     Et vous, que pouvez vous faire pour le projet GLPI ? :  Contribuer
 Download last stable version of GLPI                      -     What can you do for GLPI ? :  Contribute

#1 2008-08-13 17:10:35

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 15,273

Bug sur le purge des utilisateurs

Etat des lieux :

Nous avions des comptes génériques que nous supprimons progressivement de la montée en charge via l'authentification LDAP.

Problème :

Lors de la suppression de ces comptes génériques, les notes attachées sont également supprimées bien qu'elles n'aient pas été mises en privé.

Solution :

dans la classe "User" de clean DBonPurge, la requete est
DELETE FROM glpi_reminder WHERE FK_users = '$ID'";

je pense qu'il faudrait ajouter à la requete "AND private=1"
afin que seules les notes privées soient supprimées.

Remarque :

Le problème est identique pour les bookmarks.

Yllen


CentOS 6.5 - CentOS 7.x
PHP 5.6 - PHP 7.x - MySQL 5.6  - MariaDB 10.2 + APC + oOPcache
GLPI from 0.72 to dev version
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

Offline

#2 2008-08-13 23:08:03

MoYo
GLPI - Lead
From: Poitiers
Registered: 2004-09-13
Posts: 14,513
Website

Re: Bug sur le purge des utilisateurs

et que fait t'on des notes publiques ?
Quel auteur défini t'on ?


MoYo - Julien Dombre - Association INDEPNET
Contribute to GLPI :    Support     Contribute     References     Freshmeat

Offline

#3 2008-08-14 08:45:16

remi
GLPI-DEV
From: Champagne
Registered: 2007-04-28
Posts: 7,127
Website

Re: Bug sur le purge des utilisateurs

Une note publique sans auteur ne perturbe pas le fonctionnement.

Je vois 2 solutions :
- mettre à 0
- attribuer à l'utilisateur courant... (si vraiment on veux garder un auteur).

Après on peut aussi prévoir qu'en cas de modification, l'utilisateur en prenne possession.

++


Dév. Fedora 29 - PHP 5.6/7.0/7.1/7.2/7.3/7.4 - MariaDB 10.3 - GLPI master
Certifié ITILv3 - RPM pour Fedora, RHEL et CentOS sur https://blog.remirepo.net/

Offline

#4 2008-08-14 08:48:33

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 15,273

Re: Bug sur le purge des utilisateurs

MoYo wrote:

et que fait t'on des notes publiques ?

Une note publique étant destinée à toute l'entité, il ne faut surtout pas la supprimer

MoYo wrote:

Quel auteur défini t'on ?

Il est impossible de définir un auteur en automatique, surtout pour les grosses structures où les utilisateurs sont très nombreux.

Mais je pense qu'il vaut mieux afficher une note avec une colonne "auteur" vide (c'est ce qui ce passe si l'auteur n'est pas trouvé)  plutot que de faire disparaitre une note importante pour l'entité.


CentOS 6.5 - CentOS 7.x
PHP 5.6 - PHP 7.x - MySQL 5.6  - MariaDB 10.2 + APC + oOPcache
GLPI from 0.72 to dev version
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

Offline

#5 2008-08-14 08:55:22

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 15,273

Re: Bug sur le purge des utilisateurs

remi wrote:

- attribuer à l'utilisateur courant... (si vraiment on veux garder un auteur).

Pas d'accord du tout ! le premier qui regarde une note en devient le propriétaire ? Illogique !

remi wrote:

Après on peut aussi prévoir qu'en cas de modification, l'utilisateur en prenne possession.

Ca, à la rigueur, c'est envisageable


CentOS 6.5 - CentOS 7.x
PHP 5.6 - PHP 7.x - MySQL 5.6  - MariaDB 10.2 + APC + oOPcache
GLPI from 0.72 to dev version
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

Offline

#6 2008-08-14 10:27:22

MoYo
GLPI - Lead
From: Poitiers
Registered: 2004-09-13
Posts: 14,513
Website

Re: Bug sur le purge des utilisateurs

donc pour résumer :
- mise à 0 de l'auteur
- prise de possession à la modification si auteur = 0

même problème pour les bookmarks d'ailleurs


MoYo - Julien Dombre - Association INDEPNET
Contribute to GLPI :    Support     Contribute     References     Freshmeat

Offline

#7 2008-08-14 10:40:47

remi
GLPI-DEV
From: Champagne
Registered: 2007-04-28
Posts: 7,127
Website

Re: Bug sur le purge des utilisateurs

ok. Je m'en occupe.

Voir : https://dev.indepnet.net/glpi/ticket/1111


Dév. Fedora 29 - PHP 5.6/7.0/7.1/7.2/7.3/7.4 - MariaDB 10.3 - GLPI master
Certifié ITILv3 - RPM pour Fedora, RHEL et CentOS sur https://blog.remirepo.net/

Offline

#8 2008-08-14 10:42:21

MoYo
GLPI - Lead
From: Poitiers
Registered: 2004-09-13
Posts: 14,513
Website

Re: Bug sur le purge des utilisateurs

trop tard désolé. corrigé
https://dev.indepnet.net/glpi/changeset/7215


MoYo - Julien Dombre - Association INDEPNET
Contribute to GLPI :    Support     Contribute     References     Freshmeat

Offline

#9 2008-08-14 10:56:32

yllen
GLPI-DEV
From: Sillery (51)
Registered: 2008-01-14
Posts: 15,273

Re: Bug sur le purge des utilisateurs

Merci de ta logique.

Tu peux fermer ma demande.


CentOS 6.5 - CentOS 7.x
PHP 5.6 - PHP 7.x - MySQL 5.6  - MariaDB 10.2 + APC + oOPcache
GLPI from 0.72 to dev version
Certifiée ITIL (ITV2F, ITILF, ITILOSA)

Offline

Board footer

Powered by FluxBB