Skip to content

Commit

Permalink
fix: handle recursed effects correctly
Browse files Browse the repository at this point in the history
  • Loading branch information
johnsoncodehk committed Jan 7, 2025
1 parent 18ebcee commit d605271
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,7 @@ export function propagate(link: Link, targetFlag: SubscriberFlags = SubscriberFl
!(subFlags & (SubscriberFlags.Tracking | targetFlag))
|| (
!(subFlags & targetFlag)
&& (subFlags & (SubscriberFlags.InnerEffectsPending | SubscriberFlags.ToCheckDirty | SubscriberFlags.Dirty))
&& isValidLink(link, sub)
)
) {
Expand Down
1 change: 1 addition & 0 deletions src/system.ts
Original file line number Diff line number Diff line change
Expand Up @@ -174,6 +174,7 @@ export function propagate(subs: Link): void {
!(subFlags & (SubscriberFlags.Tracking | targetFlag))
|| (
!(subFlags & targetFlag)
&& (subFlags & (SubscriberFlags.InnerEffectsPending | SubscriberFlags.ToCheckDirty | SubscriberFlags.Dirty))
&& isValidLink(link, sub)
)
) {
Expand Down

0 comments on commit d605271

Please sign in to comment.