Update EM->trigerUntil to be an alias of trigger

[EvanDotPro]

They contained the exact same code/logic. The only difference was the
conditional around checking if the callback was valid, but the outcome
is the same for both. This simply reduces duplicate code and causes no
change in behavior at all.

[marc-mabe]

The method triggerUntil should be marked as deprecated because it lost it's sense

[prolic]

Agree with @marc-mabe . Please deprecate the triggerUntil method

[weierophinney]

You'll either have to update the tests that call triggerUntil() to ignore the warning, or remove them; the test suite breaks mightily currently!

[weierophinney]

ping @EvanDotPro ... fix the deprecation notices... :)

[weierophinney]

Closing due to inactivity. @EvanDotPro feel free to re-open or open a new PR when you're able to fix the issues.

[j-schumann]

Please reactivate this issue as triggerUntil has additional flaws:

The signature states triggerUntil($event, $target, $argv = null, $callback = null) but the callback should not be optional as it's the reason for this function.
Also calling with $argv = null as seems possible from the signature results in Event parameters must be an array or object; received "NULL", it should be ..., $argv = array(), ... like in the trigger() signature.

[weierophinney]

Marked as a BC break, as it emits a deprecation notice (which can and does break unit tests on code that exercises the method!).