Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deux reservations ont le même ID. #397

Open
jfsenechal opened this issue Nov 4, 2024 · 2 comments
Open

Deux reservations ont le même ID. #397

jfsenechal opened this issue Nov 4, 2024 · 2 comments

Comments

@jfsenechal
Copy link

Décrivez le bogue
Lorsqu'on tente de supprimer un réservation, le message Deux reservations ont le même ID. apparait
Infos GRR
Version de GRR : 4.3.7
Version PHP : 8.2
Problème suite à une installation ou mise à jour :

Reproduire
Si il y a une répétition, regarde si plusieurs enregistrements dans la table entry_repeat
Dans mon cas, le count renvoie 0
hors avec ce code if ($test != 1) ça fait une fatale error

La condiction ne devrait pas être $test > 1 ??

if ($repeat_id != 0)
	{
		$res = grr_sql_query("SELECT rep_type, end_date, rep_opt, rep_num_weeks FROM ".TABLE_PREFIX."_repeat WHERE id='".protect_data_sql($repeat_id)."'");
		if (!$res)
			fatal_error(0, grr_sql_error());
		$test = grr_sql_count($res);
		if ($test != 1)
			fatal_error(0, "Deux reservations ont le même ID.");

https://github.com/JeromeDevome/GRR/blob/v4.3.7/include/functions.inc.php#L3078

@jfsenechal jfsenechal added the bug label Nov 4, 2024
JeromeDevome added a commit that referenced this issue Nov 9, 2024
@JeromeDevome
Copy link
Owner

Bonjour,
Je ne sais pas pourquoi il retourne 0, il me semble qu'au minimum c'est un.
J'ai appliqué votre correction qui ne peut que fonctionner d253515
Bien joué ;)

@JeromeDevome JeromeDevome self-assigned this Nov 9, 2024
@jfsenechal
Copy link
Author

En fait le problème se pose avec des entrées qui ont un repeat ID mais aucune entrée dans la table repeat. Donc le base du soucis c 'est pq j'ai des entrées avec un repeat id sans enfants je n'ai pas identifié la cause.

Ma solution proposée permet de supprimer un enregistrement "corrompu"

Merci pour le suivis.

JeromeDevome added a commit that referenced this issue Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants