Fix some incorrect EventBus/notification logic in Query.

Change-Id: I4940bb16836fb39a117de7c3c36fc0aebd57eac0
Reviewed-on: http://gerrit.dmdirc.com/3825
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: Id95ee234e909ddb46eb04e24f143fe18658d4194
Reviewed-on: http://gerrit.dmdirc.com/3823
Automatic-Compile: DMDirc Build Manager
Reviewed-by: Greg Holmes <greg@dmdirc.com>

Change-Id: I9a27b003643cc43c16a8757e6c6273cde727f829
Reviewed-on: http://gerrit.dmdirc.com/3822
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

Inspections are fun. A lot of them are things PMD moans about already,
or things we're actively avoiding (pointless @inheritDocs!)

Change-Id: I785fbdce1bf468159ba55747642dfd2d7acd216e
Reviewed-on: http://gerrit.dmdirc.com/3804
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

If post() is used without the result being dereferenced, flag it
as a warning.

Change-Id: Id536f2cb5c31de9f13008d0519069965637ae754
Reviewed-on: http://gerrit.dmdirc.com/3781
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager