Bug 33880: Remove Version param from GB::Barcode->new
authorJake Deery <jake.deery@ptfs-europe.com>
Fri, 2 Jun 2023 13:13:47 +0000 (13:13 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 9 Jun 2023 11:28:59 +0000 (08:28 -0300)
commit20d22366813ddbd2c68fc72b609cee88e74e3872
treeaca3c321176d9209f898cf3b1c5740184cc7b7b9
parent7eafb103b584dcc41b863c3920ee8ab4e94b2c06
Bug 33880: Remove Version param from GB::Barcode->new

Looking at the code here
     if (Koha/Auth/TwoFactorAuth.pmoSelf->{Version}){        #--- auto version select
        for(->{Version}=1; ->{Version} <= 40; ++->{Version}) {
            last if ({->{Ecc}}->[->{Version}]
                        >=  + ->[->{Version}]);
        }
   }

in https://metacpan.org/release/KWITKNR/GD-Barcode-1.15/source/Barcode/QRcode.pm#L349

It appears the version parameter is not required, and will default to whatever version has the necessary bits for the input words given

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Koha/Auth/TwoFactorAuth.pm