I think this is an expected message. It communicates to the end user that the requested action did not happen.
If the service is started, we try to start it but we did not as it is already running. Same, if the service is stopped and the user requests to stop it again, we tell him there is no action to be done.
I think this is important to distinguish as the user can have a false impression that he started a stopped application even if the application was already running and no action happened.
What I did fix with my commit is a situation when we try to stop an already stopped service, which is unnecessary.
Does it makes sense?

As per comment 3 the current behavior is acceptable. Please verify.
(In reply to Marek Mahut from comment #3)
> I think this is an expected message. It communicates to the end user that
> the requested action did not happen.