Page MenuHomePhorge

Update phpqrcode from version 1.1.4 to 1.9.9
ClosedPublic

Authored by aklapper on Jul 8 2024, 01:55.

Details

Summary

Update the internal copy of phpqrcode from version 1.1.4 (2010100721) to version 1.9.9 (20130526).
Single reason is that it adds a lot of documentation which may come handy one day.
This more recent code is taken from https://sourceforge.net/p/phpqrcode/code/19/ which equals https://github.com/Darkflib/php-qrcode/blob/master/lib/merged/phpqrcode.php, with our two custom one-line patches still on top.

Closes T15878

Test Plan
  • Manually diff between old downstream and slightly newer upstream version in a visual diff tool to review the changes.
  • Go to http://phorge.localhost/settings/panel/multifactor/ and select "Add Auth Factor", select TOTP, click "Continue", still see a QR code properly rendered which works.

Diff Detail

Repository
rP Phorge
Branch
phpqrcodeT15878
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 1427
Build 1427: arc lint + arc unit

Event Timeline

aklapper requested review of this revision.Jul 8 2024, 01:55

sgtm

I was still able to generate QR codes, and I verified with alternative application "Barcode Scanner" that the desired URL is read.


Unrelated:

I had difficulties in following the test plan, also before this change, but only because I had 4 minutes of delay between my phone and my laptop. I just installed chrony on my Debian and its again in sync with the rest of the planet, and it works.

This revision is now accepted and ready to land.Jul 12 2024, 09:46