Revert "Don't publish twice, while the build isn't finished yet (#446)"

[timja]

This reverts commit e66caf6.

Closes #455

[timja]

@KreAch3R seems like r.isBuilding was returning true when free style jobs were running the complete build bit

[KreAch3R]

What kind of job do you use @dshvedchenko ?

Yeah we should revert this for now since it produces regressions but it will still throw 3 messages, I'll go back to the drawing board. That's why I need a setup where the notifications are failing

[dshvedchenko]

Yep. we use FreeStyle job. and Success is result of Build Step. but job is continue in Publish pahse

[KreAch3R]

What do you mean? Can you try creating a new basic freestyle job like the one I describe here: #446 (comment) and check if you get three notifications, like these:

test - #41 Started by user admin (Open)
test - #41 Success after 4.1 sec and counting (Open)
test - #41 Success after 4.9 sec (Open)