gestex issueshttps://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues2021-07-09T13:46:50+02:00https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/71Variable equipment_blacklist ?2021-07-09T13:46:50+02:00Gabriel MoreauVariable equipment_blacklist ?Je ne comprends pas le sens de equipment_blacklist.
Par exemple, dans le fichier `loan-edit.php`
```php
$equipment_blacklist = get_loans_all_by_equipment_borrowed($pdo, $equipment_id);
```
La fonction retourne manisfestement des loans.....Je ne comprends pas le sens de equipment_blacklist.
Par exemple, dans le fichier `loan-edit.php`
```php
$equipment_blacklist = get_loans_all_by_equipment_borrowed($pdo, $equipment_id);
```
La fonction retourne manisfestement des loans... Donc pourquoi la variable s'appelle `$equipment_blacklist` ?
En parallèle, la fonction devraient certainement s'appeler `get_loan_listall_by_equipment_borrowed` ?
Rappel, on écrit toujours au singulier. Si la fonction retourne une liste, alors on ajoute `listall` ou `listshort`.https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/69duréé limite d'emprunt pour un équipement2021-07-12T09:26:31+02:00De Marchiduréé limite d'emprunt pour un équipementOn pourrait ajouter un champ à chaque équipement pour mettre une valeur en nombre de jours. ce nombre de jours correspondrai au nombre de jours maximum pour l'emprunt de l'équipement.
Ensuite faudra vérifier que la personne ne dépasse pa...On pourrait ajouter un champ à chaque équipement pour mettre une valeur en nombre de jours. ce nombre de jours correspondrai au nombre de jours maximum pour l'emprunt de l'équipement.
Ensuite faudra vérifier que la personne ne dépasse pas lors d'une réservation.
Il peut exister une valeur (par défaut ?), genre "-1" qui signifie pas de limitehttps://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/68date retour avant date pret2021-07-02T10:29:01+02:00De Marchidate retour avant date pretOn peut emprunter et probablement réserver avec une date de retour avant la date de prêt.
ca donne un emprunt du style : 2021-03-04 → 2021-03-01.
=> Ajout d'un test pour date retour >= date prêt.On peut emprunter et probablement réserver avec une date de retour avant la date de prêt.
ca donne un emprunt du style : 2021-03-04 → 2021-03-01.
=> Ajout d'un test pour date retour >= date prêt.https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/66Pas de chaîne évaluée dans une requête SQL / injection SQL2021-06-30T08:43:56+02:00Gabriel MoreauPas de chaîne évaluée dans une requête SQL / injection SQL```php
function set_loan_to_returned($pdo, $id) {
$sql = "UPDATE LOW_PRIORITY pret SET status = '".STATUS_LOAN_RETURNED."', retour = CURRENT_DATE WHERE id = $id;";
```
On ne met donc pas de " et les variables sont passées via ? et elle...```php
function set_loan_to_returned($pdo, $id) {
$sql = "UPDATE LOW_PRIORITY pret SET status = '".STATUS_LOAN_RETURNED."', retour = CURRENT_DATE WHERE id = $id;";
```
On ne met donc pas de " et les variables sont passées via ? et elles sont remplacées automatiquement lors du execute.
On évite ainsi l'injection de SQL.https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/64Fixer le problème de l'emprunt précédent dans equipment-view.php2021-06-29T11:12:31+02:00Estéban RistichFixer le problème de l'emprunt précédent dans equipment-view.phpLe dernier emprunt précédent ne s'affiche pas dans **equipment-view.php** quand il n'y qu'un seul emprunt précédent.Le dernier emprunt précédent ne s'affiche pas dans **equipment-view.php** quand il n'y qu'un seul emprunt précédent.https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/61Status et non state2021-06-25T16:49:34+02:00Gabriel MoreauStatus et non stateDans d'autres table, on a déjà utilisé l'atteribut status plutôt que statte.
```sql
ALTER TABLE `pret` ADD COLUMN `state` ENUM('booking', 'loan', 'undefined', 'old') NOT NULL;
UPDATE `pret` SET `state` = 3;
```
Pourquoi une valeur unde...Dans d'autres table, on a déjà utilisé l'atteribut status plutôt que statte.
```sql
ALTER TABLE `pret` ADD COLUMN `state` ENUM('booking', 'loan', 'undefined', 'old') NOT NULL;
UPDATE `pret` SET `state` = 3;
```
Pourquoi une valeur undefined ?
Pourquoi cette affectation à la valeur 3 ? On peut mettre 'loan' je pense, sinon les ENUM ne servent à rien. D'ailleurs, on n'a pas tendance à écrire les valeurs en MAJUSCULES lorsque ce sont des constantes ?https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/60Emprunt chevauche une réservation2021-06-29T10:41:08+02:00Gabriel MoreauEmprunt chevauche une réservationOn garde toujours le dernier emprunteur. Lorsque le retour se fait en retard et qu'il chevauche une réservation, cela ne doit pas être un soucis. Si la réservation n'est finalement pas utilisée, ce n'est pas grave, on garde toujours le d...On garde toujours le dernier emprunteur. Lorsque le retour se fait en retard et qu'il chevauche une réservation, cela ne doit pas être un soucis. Si la réservation n'est finalement pas utilisée, ce n'est pas grave, on garde toujours le dernier emprunt visible.https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/59De la réservation à l'emprunt2021-07-01T14:59:16+02:00Gabriel MoreauDe la réservation à l'empruntLorsqu'une personne a réservé un équipement et qu'elle viens le chercher, il faut une action de la personne qui gère le magasin pour transformer la réservation en emprunt.Lorsqu'une personne a réservé un équipement et qu'elle viens le chercher, il faut une action de la personne qui gère le magasin pour transformer la réservation en emprunt.https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/58Réservation passée2021-07-02T10:28:25+02:00Gabriel MoreauRéservation passéeIl n'est pas possible de faire une réservation pour le jour même ou pour une date passée. Cependant, si la personne n'est pas venue chercher le matériel, la date de début de la réservation peut être passée.Il n'est pas possible de faire une réservation pour le jour même ou pour une date passée. Cependant, si la personne n'est pas venue chercher le matériel, la date de début de la réservation peut être passée.https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/57Emprunt le jour même d'un retour2021-07-05T16:04:05+02:00Gabriel MoreauEmprunt le jour même d'un retourIl est possible d'emprunter le même jour plusieurs fois le même équipement. Comme les emprunts signifient que la personne prends l'équipement, on est sur qu'il n'y a pas chevauchement !Il est possible d'emprunter le même jour plusieurs fois le même équipement. Comme les emprunts signifient que la personne prends l'équipement, on est sur qu'il n'y a pas chevauchement !https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/56Retour d'un emprunt2021-07-01T15:41:27+02:00Gabriel MoreauRetour d'un empruntComment savoir qu'un équipement emprunté a été rendu. Avant on le détruisait... Désormais, on va garder a minima le dernier emprunteur afin de savoir qui a utilisé l'équipement en dernier.
Possibilité, ajouter un booléen `return` à fals...Comment savoir qu'un équipement emprunté a été rendu. Avant on le détruisait... Désormais, on va garder a minima le dernier emprunteur afin de savoir qui a utilisé l'équipement en dernier.
Possibilité, ajouter un booléen `return` à false par défaut. Ainsi, tous les emprunts actuels seraient automatiquement à false ce qui est une bonne chose.
Autre idée possible ?https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/52Cannot create user2021-06-23T14:44:01+02:00Estéban RistichCannot create userPHP failed to create user, error at [db-functions.php](https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/blob/master/module/db-functions.php) line 63 in function check_val_in_db(). it fails during the verification of the ex...PHP failed to create user, error at [db-functions.php](https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/blob/master/module/db-functions.php) line 63 in function check_val_in_db(). it fails during the verification of the existence of a user.
![Capture1](/uploads/a537b4db2d439df3a7bfbc90711a4874/Capture1.PNG)https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/51Bug in SQL schema2021-06-23T14:44:01+02:00Gabriel MoreauBug in SQL schemaThere is a bun in the SQL schema. The actual schema is not validThere is a bun in the SQL schema. The actual schema is not validhttps://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/50Change login2021-07-02T10:27:21+02:00Gabriel MoreauChange loginThe superadmin user could change (and update) a user loginThe superadmin user could change (and update) a user loginhttps://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/48Éditer une réservation2020-10-05T16:50:10+02:00Gabriel MoreauÉditer une réservationPouvoir mettre à jour une réservation en coursPouvoir mettre à jour une réservation en courshttps://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/45mettre des placeholder dans les formulaires2020-09-30T22:29:17+02:00EXT Sylène moreaumettre des placeholder dans les formulaires exemple : <input type="text" class="form-control" id="tweet" placeholder="Entrez votre texte"> exemple : <input type="text" class="form-control" id="tweet" placeholder="Entrez votre texte">https://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/43Thème sombre - bouton modifier mal positionné2020-09-30T22:31:37+02:00Gabriel MoreauThème sombre - bouton modifier mal positionnéPar exemple, quand on modifie une équipe dans le thème sombre, le bouton modifer est à gauche et non à droitePar exemple, quand on modifie une équipe dans le thème sombre, le bouton modifer est à gauche et non à droitehttps://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/41Lien en blanc - thème sombre2020-07-16T14:40:38+02:00EXT Sylène moreauLien en blanc - thème sombreil faudrait changer la couleur des liens dans le mode sombre car ils ne sont pas très visible en bleu/violetil faudrait changer la couleur des liens dans le mode sombre car ils ne sont pas très visible en bleu/violetEXT Sylène moreauEXT Sylène moreauhttps://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/39change level user2020-07-16T14:26:00+02:00De Marchichange level usermodifier le profil et changer la qualité ne fonctionne pas => pas de changement du levelmodifier le profil et changer la qualité ne fonctionne pas => pas de changement du levelhttps://gricad-gitlab.univ-grenoble-alpes.fr/legi/soft/gestex/-/issues/38change password envoi vers list_manip2020-07-16T14:16:33+02:00De Marchichange password envoi vers list_manipaprès le changement de password sur user_changepwd.php, on arrive sur list_manip.phpaprès le changement de password sur user_changepwd.php, on arrive sur list_manip.php