spip_release_lock => pb mysqli_query #3746

Closed
opened 7 years ago by tofulm · 5 comments
tofulm commented 7 years ago
Collaborator

Bonjour,
dans la fonction spip_release_lock($nom) source:spip/ecrire/req/mysql.php#L1699

la fonction mysqli_query (l1708) est en erreur. Il manque la connexion en premier argument :

$link = _mysql_link();
mysqli_query($link,$q);

Comme cela ca fonctionne

Bonjour, dans la fonction spip_release_lock($nom) source:spip/ecrire/req/mysql.php#L1699 la fonction mysqli_query (l1708) est en erreur. Il manque la connexion en premier argument : $link = _mysql_link(); mysqli_query($link,$q); Comme cela ca fonctionne
b_b commented 7 years ago
Owner
There is no content yet.
Poster
Collaborator

exemple dans un cvt (traiter)

$toto = spip_get_lock("titi");
sleep (5);
spip_release_lock($toto);

( ! ) Warning: mysqli_query() expects at least 2 parameters, 1 given in /var/www/html/cdp_mutu/ecrire/req/mysql.php on line 1707

exemple dans un cvt (traiter) <pre> $toto = spip_get_lock("titi"); sleep (5); spip_release_lock($toto); </pre> ( ! ) Warning: mysqli_query() expects at least 2 parameters, 1 given in /var/www/html/cdp_mutu/ecrire/req/mysql.php on line 1707
b_b commented 7 years ago
Owner

C'est certainement parce que cette fonction est dépréciée, cf :

https://core.spip.net/projects/spip/repository/entry/spip/ecrire/req/mysql.php#L1691

D'ailleurs, elle n'est utilisée nul part dans le core, mais ça ne mange pas de pain de la corriger. Attendons d'autres avis avant d'intégrer le patch...
Statut changé à En cours

C'est certainement parce que cette fonction est dépréciée, cf : https://core.spip.net/projects/spip/repository/entry/spip/ecrire/req/mysql.php#L1691 D'ailleurs, elle n'est utilisée nul part dans le core, mais ça ne mange pas de pain de la corriger. Attendons d'autres avis avant d'intégrer le patch... **Statut changé à En cours**

Appliqué par commit r23126.
Statut changé à Fermé

Appliqué par commit r23126. **Statut changé à Fermé**
b_b commented 6 years ago
Owner
There is no content yet.
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.