[Bug 35355] AppDB codebase needs to be updated to comply with modern php coding standards

https://bugs.winehq.org/show_bug.cgi?id=35355

--- Comment #3 from Rosanne DiMesio  ---
This is mostly fixed, but I still see the following:

1. On the form for processing screenshots,

Strict Standards: Non-static method appData::getDefaultReply() should not be
called statically, assuming $this from incompatible context in
/home/winehq/opt/appdb/include/screenshot.php on line 818

appears between submitted screenshot and the reply text box.

2. In the Control Center, running Update Application Maintainer states produces
a page with thousands of lines of

Strict Standards: Non-static method maintainer::versionHasMaintainer() should
not be called statically, assuming $this from incompatible context in
/home/winehq/opt/appdb/include/version.php on line 1554

Running Update Version Maintainer States has a similar result, but with line
254  referenced at the end.