fix promise patching, again

[x1B]

The angular adapter needs to hook into promise in order to kick off a digest cycle when an eventBus.publish(...) or an eventBus.publishAndGatherReplies(...) completes.

The current workaround mechanism essentially breaks the native promise by replacing it with $q, so we need to fix it.

Alternatives:

• patch window.Promise.prototype.then to have it trigger a digest cycle when the completion/rejection callback is invoked. This is a complete solution, but may have unintended side effects, e.g. performance problems.

• decorate axEventBus for AngularJS widgets so that publish/publishAndGatherReplies returns a decorated promise, that again triggers a digest cycle. We need to check if this covers all our bases.

[x1B]

Note: it is fine if the solution does not make Promise.resolve() magically trigger digest-cycles when used from AngularJS widgets! We just have to make sure that our libraries (laxar, laxar-patterns) function seamlessly.

[x1B]

Using the second variant (selectively fixing eventBus.publish and eventBus.publishAndGatherReplies).

[x1B]

Fixed on master (v2.0.0).

[x1B]

@alex3683 care to have a look?

[alex3683]

@x1B looks really good to me! Less hacky and more focused. 🤓