logs.php works not only on apache

See merge request !42

Screen has changed yet again, removing the argument from the -L parameter and adding a new parameter -Logfile.

Corrected crash due to faulty logic.

Uncommented one too many lines last commit

I missed one of these when I last fixed them. PvP servers were not getting the 'server started' message printed due to this

removing these because the latest factorio releases generate zip files with inconsistencies. Game still runs, but php-zip will fail on the inconsistency with how we're currently scripted.

was not needed.

i hate you php

Add non_blocking_saving to server settings configuration options

My fault, I commented a line that shouldn't have been commented

Bot will restart upon encountering an error
A player logging in or sending a chat message will set the server status to 'started' if it is not already correctly set
Pipes are now set to automatically close on exec, so there will no longer be extraneous file handles leaking into servers

The new version of PvP allows an admin to force the round to end. This new code detects that and prints the appropriate message.

Factorio 0.16 added [SHOUT] before shouts. This update is primarily to remove that so that the bot functions properly.

check if this function exists before trying to call it. Otherwise, nginx fails, because nginx isn't apache, i guess.

Update is backwards compatible, just like the version_send is. .setGame was depreciated in 11.3, replaced with .setActivity.

Added extra error detection

Zombie processes are child processes that were not fully closed by their parent process. The servers are launched as child processes. Zombies do not take up any memory, but they do take a spot on the process list. Enough zombies could prevent any more processes from being started. This commit now prevents those zombies from being formed.
Opening this file in Visual Studio made VS automatically format it, so the vast majority of changes in this commit are actually minot formatting changes.

added "@" + per zackmans reccomendations

Unsure if 0.15.x figured this out on it's own, or if it's just never been working right... but 0.16.x didn't like that this wasn't properly worded.

Some servers don't want to have a third role, this update will make the role optional.
There is some incorrect indentation here (4 space instead of tabs), the web editor is refusing to insert proper tabs. I will fix this in a later commit.

Found another place that depended on the word 'stable' being in the URL