search

damil / SQL-Abstract-More

extension to SQL-Abstract with named parameters and support for several additional SQL clauses
https://metacpan.org/pod/SQL::Abstract::More
6 stars 10 forks source link

Disabled redundant warnings on sprintf #8

Closed PJ-DSI-DevOps closed 8 years ago

PJ-DSI-DevOps commented 8 years ago

Avec les nouvelles versions de Perl (5.22.1) il y a un warning si l'on appel sprintf avec un nombre de paramètre qui ne correspond pas aux tags dans le champ FORMAT. C'est simplement reproductible avec les tests déjà disponible (01-sql_abstract_more.t).

Voici le message : t/01-sql_abstract_more.t .... 1/59 Redundant argument in sprintf at /home/yves/workspace/SQL-Abstract-More/blib/lib/SQL/Abstract/More.pm line 665.

Il semble que cela est assez largement utilisé dans SQL::Abstract::More et pour faire simple je propose simplement de désactiver ce warning, puisque c'est clairement le fonctionnement voulu.

damil commented 8 years ago

Hum ... s'il y a un warning c'est qu'il y a quelque part une incohérence logique dans la façon d'invoquer ce sprintf; probablement il serait plus sage de régler le problème à la source. Je vais m'installer un Perl 5.22 pour reproduire le problème, je te donne des nouvelles après.

sezal commented 8 years ago

We faced the same warnings after switch to 5.22.1. After adding debugging output in the code I've noticed $sql had SQL without % placeholders

damil commented 8 years ago

After all, disabling the warnings is indeed the best solution --- Thanks Yves. But since 'redundant' is not a valid warnings category before Perl 5.22, I had to add an additional test. The new versioni (1.28) also supports quote_char.