-
Notifications
You must be signed in to change notification settings - Fork 520
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
getLoopCount() ? #16
Comments
You seem to be correct in this observation. The easiest way I can see to check the loop count is by accessing the If it were public, you could get the count using:
If you want to, you might want to submit this as a new feature. If not, I might add this feature to my fork of the repository later. |
I don’t see why this couldn’t be added. Happy to look at a PR
…On Mon, Nov 5, 2018 at 10:56 AM Renato Oliveira ***@***.***> wrote:
You seem to be correct in this observation. The easiest way I can see to
check the loop count is by accessing the loopCountMap static variable
(which is private now).
If it were public, you could get the count using:
TriggerHandler.loopCountMap.get('handlerName').getCount();
If you want to, you might want to submit this as a new feature. If not, I
might add this feature to my fork of the repository later.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#16 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAigQGxx5dGIupMy9w1RKFqkUXlChJUAks5usF-ZgaJpZM4YOm8u>
.
|
@kevinohara80 Please take a look at PR #13, thanks. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is it by design, or just an oversight that there doesn't appear to be a way of seeing how many loops/recursions you are current in?
Would be useful to say "only run this on the outer most loop" etc.
Or have I missed the mark here?
The text was updated successfully, but these errors were encountered: