search

operasoftware / dns-ui

Opera's LDAP-authenticated PowerDNS user interface
Apache License 2.0
282 stars 57 forks source link

fix php81 (ErrorException: unserialize(): Passing null to parameter #… #193

Closed sipopo closed 1 year ago

sipopo commented 1 year ago

It happens when change->before or change->after is null and php8.1 is used.

Fix error in PHP8.1

[29-Aug-2022 09:27:38 UTC] 1661765258: ErrorException: unserialize(): Passing null to parameter #1 ($data) of type string is deprecated in /usr/local/share/dns-ui/views/api.p-hp:324
[29-Aug-2022 09:27:38 UTC] 1661765258: Stack trace:

[29-Aug-2022 09:27:38 UTC] 1661765258: #1 /usr/local/share/dns-ui/views/api.php(324): unserialize()
[29-Aug-2022 09:27:38 UTC] 1661765258: #2 /usr/local/share/dns-ui/views/api.php(115): API->show_zone_change()
[29-Aug-2022 09:27:38 UTC] 1661765258: #3 /usr/local/share/dns-ui/views/api.php(305): API->options() [29-Aug-2022 09:27:38 UTC] 1661765258: #4 /usr/local/share/dns-ui/views/api.php(39): API->zone_change() [29-Aug-2022 09:27:38 UTC] 1661765258: #5 /usr/local/share/dns-ui/requesthandler.php(62): require('/usr/local/shar...') [29-Aug-2022 09:27:38 UTC] 1661765258: #6 /usr/local/share/dns-ui/public_html/init.php(18): require('/usr/local/shar...') [29-Aug-2022 09:27:38 UTC] 1661765258: #7 {main}

Signed-off-by: Sergey V. Osipov sipopo@yandex.ru

thomas-pike commented 1 year ago

Fix merged. Thank you for your contribution!