My new review:
#Prevent for asking to install mono
WINEDLLOVERRIDES="mscoree=d"
export WINEDLLOVERRIDES
Caution, PlayOnLinux already defines $WINEDLLOVERRIDES, so you'd better use
WINEDLLOVERRIDES="mscoree=d;$WINEDLLOVERRIDES"
POL_SetupWindow_Init
Embedded debugger is not initialized here (missing POL_Debug_Init)
if [ "$(echo $APP_ANSWER | grep -o "LibreOffice_4")" != "" ]
can be simplified as
if grep -q LibreOffice_4 <<<"$APP_ANSWER"
I'm not sure I should have talked about supporting several major versions, that was just a suggestion about how versions could be handled but I was thinking of eventually writing seperate scripts, supporting it with a single script added a lot of complexity. Hence harder to spot bugs, say
POL_Shortcut "soffice.exe" "$TITLE"
The shortcut name stays the same, so you cannot actually install both versions side-by-side, second installation will overwrite the shortcut created by the first.
Even if the complexity could be probably better managed with associative arrays (I've yet to try), I think having LibreOffice 4 and LibreOffice 5 scripts would be easier to manage in the end.
Edited by petch