FIX DROP INDEX IF EXISTS it's not possible !

[hregis]

https://dev.mysql.com/doc/refman/8.4/en/drop-index.html

[hregis]

des fois je me demande si des modifications sont réellement testées ! 😄
d'où mon coup de gueule d'avoir deux versions par an, une version bancale est à peine déboguée en v20.0.2 qu'on a une version 21 ! 😄

[frederic34]

#20016

[hregis]

@FHenry oui donc ok pour postgresql mais pas avec Mysql/Mariadb ! #20016

[hregis]

@eldy @FHenry @frederic34 ok sorry, look, it's ok for Postgresql and MariaDB, but not for Mysql !

https://www.jooq.org/doc/latest/manual/sql-building/ddl-statements/drop-statement/drop-index-statement/drop-index-if-exists/

[MarcVJ]

@eldy @mdeweerd @hregis Merci pour vos retours et corrections,
Si nous ne pouvons pas utiliser IF EXISTS avec MySQL, est-ce qu'il est nécessaire d'ajouter du code avant la ligne, pour vérifier que l'index existe, et éviter des erreurs si on essaie d'effacer un index inexistant ?

[mdeweerd]

Ok, in theory this would suggest that the creation of a "dropIndex' method in DoliDB and derived classes so that it adapts to the capabilities of the db engine.
On the other hand this occurs only once.

[eldy]

@eldy @mdeweerd @hregis Merci pour vos retours et corrections,
Si nous ne pouvons pas utiliser IF EXISTS avec MySQL, est-ce qu'il est nécessaire d'ajouter du code avant la ligne, pour vérifier que l'index existe, et éviter des erreurs si on essaie d'effacer un index inexistant ?

Like we have a DDLDropField into the database drivers core/db/xxx.class.php we can have a DDLDropIndex so we can have different code forneach database. For mysql, we can justvmake a drop and ignore the error if index does not exists before.

[hregis]

@eldy @MarcVJ can you try with this PR : #32866