This was easy because only README.md and doc/01-About.md were redacted manually, everything else via:
git ls-files -z |xargs -0 perl -pi -e 's/Icinga GmbH \| GPLv2/Icinga GmbH | GPLv2+/'
This is legal because we have only merged PRs with label:cla/signed or made by Icinga staff:
https://github.com/Icinga/icingadb-web/pulls?page=1&q=is%3Apr+is%3Aclosed+-label%3Acla%2Fsigned+-author%3Anilmerg
This has no risk for us in people distributing their own version under GPLv3 only.
After all, we won't take their patches anyway, unless they sign our CLA.
This is the cleanest solution for having e.g. these in one address space:
* Icinga Web, GPLv2+
* K8s Web, AGPLv3
* Thirdparty, some LGPLv3 and Apache-2.0
Apropos, K8s Web is even v3-licensed on purpose, to have a stronger protection against cloud ops.
Nowadays hGet returns null instead of false if there is no data available yet.
But since we are performing strict comparison on false, it won`t enter the if branch anymore.
There is another Predis client method we use which is `hmGet`, but as usual it still returns only an array.
* Command forms: Use strict interfaces to construct ipl\Html objects
* BaseTableRowItem: Use strict interfaces to create ipl\Html objects
* ObjectInspectionDetail: Use strict interfaces..
..to create ipl\Html objects
* Setup/*Step: Use strict interfaces to create ipl\Html objects
* ObjectSuggestions: Use strict interfaces to create ipl\Html objects
* Widget: Use strict interfaces to create ipl\Html objects
* Widget\Detail: Use strict interfaces to create ipl\Html objects
* Widget\ItemList: Use strict interfaces to create ipl\Html objects
* ServiceLink: Don't return an array