mirror of
https://github.com/actualbudget/actual.git
synced 2026-07-11 09:32:33 -05:00
[Bug]: Rule to change CLEARED to false is getting run regardless of conditions #2539
Open
opened 2026-02-28 20:17:37 -06:00 by GiteaMirror
·
7 comments
No Branch/Tag Specified
master
enforce-append-only-migrations-ci
delay-initial-sync-until-loaded
jfdoming/mobile-app-build
claude/error-codes-worker-boundary-26i5uu
jfdoming/api-tokens-part-1
jfdoming/api-tokens-part-2
jfdoming/api-tokens-part-3
jfdoming/mobile-app-updates
ai/simplify-categories-get-api
api-browser-custom-data-dir
claude/budget-version-migration-compat-151jlr
claude/worker-error-serialization-9iqtl7
claude/react-table-library-research-afx83z
claude/issue-8401-regression-test-9nw9l3
jfdoming/mobile-app-foundation-part-1
jfdoming/mobile-app-scaffold
release
ai-fix-cleanup-def-8290
claude/relaxed-shannon-jusywa
claude/fervent-keller-s6f2fq
claude/determined-feynman-st1y9z
claude/wizardly-einstein-vi2wce
claude/sweet-albattani-7nseio
feat/bank-sync-per-budget-file
claude/affectionate-cannon-y0kc82
claude/dependabot-findings-m9pgmb
claude/mac-app-ios-publish-zjs1lz
claude/vigilant-tesla-aiiyos
depot-migrate-secrets-1781296330308
claude/quirky-cerf-v5qck7
claude/animation-removal-alternative-2etfh1
claude/netlify-preview-comments-consolidate-9XPYa
osc/7391-error-boundaries-reports
claude/pensive-dijkstra-7DYui
claude/server-app-version-mismatch-HTCGv
claude/pr-8027-review-feedback-mELuO
claude/zealous-wozniak-e5pO0
claude/intelligent-heisenberg-BSHJq
claude/translation-download-refactor-jFRsO
claude/mobile-performance-ideas-G9PAO
claude/github-issue-7988-5A5ly
blacksmith-migration-ff70d2d
claude/great-mayer-KRvOk
claude/react-native-ios-compilation-fcJh5
claude/serene-goldberg-58xv0
claude/kind-euler-LVG3K
release-automation/edge-nightly
7710-bundle-migrations
claude/fix-docker-build-error-WMfed
claude/llm-compatible-release-notes-t20Nr
7710-tests-tdd
cursor/transaction-table-rewrite-f077
claude/plan-ci-secure-context-OtEe1
claude/fix-issue-7667-DPXi3
cursor/resolve-pr-7449-ee11
claude/fix-typescript-build-error-JPtZ5
copilot/add-repository-configs-to-packages
worktree-compressed-drifting-ritchie
claude/api-consumer-verification-kfz1K
pr-7454
claude/fix-issue-7410-LLLQ4
revert-7281-generate-icons
react-query-useSchedules
copilot/sub-pr-6140
package-upgrades
v26.7.0
v26.6.0
v26.5.2
v26.5.1
v26.5.0
v26.4.0
v26.3.0
v26.2.1
v26.2.0
v26.1.0
v25.12.0
v25.11.0
v25.10.0
v25.9.0
v25.8.0
v25.7.1
v25.7.0
v25.6.1
v25.6.0
v25.5.0
v25.4.0
v25.3.1
v25.3.0
v25.2.1
v25.2.0
v25.1.0
v24.12.0
v24.11.0
v24.10.1
v24.10.0
v24.9.0
v24.8.0
v24.7.0
v24.6.0
v24.5.0
v24.4.0
v24.3.0
v24.2.0
v24.1.0
v23.12.0
v23.11.0
v23.10.0
v23.9.0
v23.8.1
v23.8.0
v23.7.2
v23.7.1
v23.7.0
v23.6.0
v23.5.0
v23.4.2
v23.4.1
v23.4.0
v23.3.2
v23.3.0
v23.2.9
v23.2.5
v23.1.12
v22.12.9
Labels
Clear labels
AI generated
API
bank sync
budgeting
bug
can’t replicate
dependencies
docker
documentation
electron
experimental feature
feature
feedback
goal templates
good first issue
help wanted
importers
maintenance
needs info
needs testing
needs triage
needs votes
openid
payees
pull-request
regression
reports
responsive
rules
schedules
server
✨ merged
split transactions
tech debt
tech-support
theme
transaction import
transaction reconciliation
transactions
translations
upstream
user interface
✅ approved
wontfix
Mirrored from GitHub Pull Request
Milestone
No items
No Milestone
Projects
Clear projects
No project
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/actual#2539
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @natemeyerr-ctrl on GitHub (Oct 14, 2025).
Verified issue does not already exist?
What happened?
I created a rule to set the CLEARED flag to false on all imported transactions. The IF is amount < $10,000 (this is just a way to catch all rows), and the THEN is set cleared to false.
It works! The imported rows now import as uncleared.
However, everytime the rule runs, it marks all other previously imported transactions as uncleared. So now all my transactions on my chequing account are uncleared.
I also added another THEN to the rule just to see what rows it's actually meaning to update. I added THEN prepend to notes = TEST.
Imported, and the newly imported rows is uncleared, and it has TEST in the notes line.
However, all other rows in that account are now also uncleared, but there is no TEST in those rows.
So whenever you use cleared equals false in a rule it seems to clear all transactions in the account instead of just the imported transactions
How can we reproduce the issue?
Where are you hosting Actual?
Pikapods
What browsers are you seeing the problem on?
Chrome
Operating System
Windows 11
@youngcw commented on GitHub (Oct 14, 2025):
Are you using bank sync? My guess is that the problem is that the other transactions are getting imported and merged and thus rules are running on those transactions. Bank sync will import up to 90 days of transactions every sync, so your rule is being run on 90 days worth of transactions every time you sync.
I was not able to recreate this when using an ofx file as only a few of the transactions were getting "hit" by the import and only the new/modified transactions get the rule run.
@natemeyerr-ctrl commented on GitHub (Oct 14, 2025):
Yes. I'm using bank sync. What your saying makes sense.
The rule also adds the word test in front of the notes. Do you have an explanation on why the merged records don't receive the test but do receive the unclear flag?
@youngcw commented on GitHub (Oct 14, 2025):
Im only seeing the rule get applied to transactions that were part of the import. So no "test" and no unclearing of transactions that weren't imported or touched by the import.
@youngcw commented on GitHub (Oct 14, 2025):
If you want this rule to only apply to new transactions you will need to make your rule more complicated. For example, exclude transactions that have notes with "#Approved" in the un clear rule, then a separate rule that adds the "#Approved" tag to the notes at some point.
@natemeyerr-ctrl commented on GitHub (Oct 14, 2025):
Gotcha! I made the rule more complicated
I run Bank Sync
That new transaction (Youtube Premium) is imported and has #Imported in the NOTES. The cleared flag is also uncleared. However, the U-Haul transaction is also now UNCLEARED. even though it had #Imported in the notes.
I'm not sure what I'm missing here. But it seems there's a bug with the cleared flag, triggering on all transactions regardless of the criteria.
@youngcw commented on GitHub (Oct 14, 2025):
Ok, I am able to recreate it with these steps. I don't know whats happening, but the rule is getting applied to transactions it shouldn't. The rule edit page doesn't include the transactions marked as #IMPORTED but the rules are getting applied anyway.
I recreated it with both bank sync and OFX import so its probably an issue in the rules code directly.
@natemeyerr-ctrl commented on GitHub (Oct 15, 2025):
Thanks!