Fix silently dismissing user defined errors
All user defined errors were silently dismissed, since error was passed on as a boolean instead of an integer, so no differentiation could be made anymore down the line.
This fix correctly dismisses a cancelled share without an error message but brings back the red indicator together with the error message when an user defined error was raised by a share plugin.
This will bring the back the red indicator when an error is raised which was not reachable in the code anymore:
All tests are passing:
Test project /home/mydayyy/kde/build/spectacle
Start 1: appstreamtest
1/2 Test #1: appstreamtest .................... Passed 1.02 sec
Start 2: filename_test
2/2 Test #2: filename_test .................... Passed 0.06 sec
100% tests passed, 0 tests failed out of 2
Total Test time (real) = 1.08 sec