mirror of
https://github.com/actualbudget/actual.git
synced 2026-03-09 03:32:54 -05:00
[Bug]: Find & Replace not working correctly all the time #2886
Open
opened 2026-02-28 20:31:36 -06:00 by GiteaMirror
·
12 comments
No Branch/Tag Specified
master
claude/fix-simplefin-ssrf-T31gX
claude/release-notes-validation-X7rvR
matiss/7155
claude/fix-simplefin-batch-sync-O8LcD
ai/custom-theme-dual-prefs
matiss/fix-6804
add-claude-github-actions-1772738270730
claude/analyze-internal-errors-4k6O2
react-query-rules
react-query-useSchedules
matiss/separate-lint-format
dependabot/npm_and_yarn/ajv-6.14.0
cursor/sync-performance-notification-9899
react-query-prefs
matiss/chunked-sync-and-progress-ux
v26.2.1
copilot/sub-pr-6880
fix-react-query-clear-on-close-budget
copilot/sub-pr-6140
feat/auto-note
feat/scoped-bank-sync
cursor/desktop-transactions-react-table-1d0c
fix-exhaustive-deps-App
copilot/fix-find-replace-bug
release/v26.2.0-pre
matiss/browser-tests
mobile-fix-drag-and-drop-across-groups
budget-table-v2
PayeeAutocomplete2
pglite
bugfix/plugins/fix-plugins-sw
feat/plugins/plugins-core-package
prerelease
matiss/unicode-minus-fix
cursor/fix-actual-github-issue-6206-gemini-3-pro-preview-9c37
TransactionFormPage
cursor/implement-mortgage-and-loan-account-type-78ca
tests-update-fill-with-pressSequentially
mobile/link-modal
deps/25.11
cursor/fix-update-vrt-apply-ci-job-dispatch-b324
sync-server-plugins
cursor/propose-patch-for-github-issue-5680-2a18
fix/compiler-preserve-inner-dollar-escapes
cursor/analyze-actual-budget-issue-and-propose-fix-5b70
coderabbitai/docstrings/0c070e5
cursor/add-wip-prefix-and-comment-to-prs-d78d
jfdoming/08-21-auto-focus-on-navigate-in-all-browsers
show-totals-on-mobile-budget-banners
allow-child-transactions-make-transfer
mobile-calculator-keyboard
payee-geolocation
enhance/restore_scroll_position
dm-fix-second-click-on-mobile-new-transaction-2
scrollToLocationBudget
alert-autofix-38
tsconfig-composite
mobile-fix-uncategorized-transactions-on-tracking-budgets
server-budget-handlers
fix-sql-injection-in-cleanup-template
non-chrome-draggable-workaround
mobile-budget-page-swipe-navigation
ts-db-all
stable
dark-theme-with-brand-colors
fix-mobile-delete-group
ts-db-select
UnderKoen/reconcile-context-menu
master-before-server-merge
v25.2.1
ts-runQuery
rename-redux-hooks
UnderKoen/3557-persist-state-in-history
remove-redux-CLOSE_BUDGET
fix-exhaustive-deps-errors-FinancesApp
redux-toolkit-createSlice-backup
accounts-function-component
ts-useSplitsExpanded
loot-core-server-package
useTransactios-in-TransactionEdit
react-aria-input
move-redux-to-desktop-client
QueryState-type
fix-themes-applied-late
mobile-vrts
revert-3295-spendingCardFix
react-aria-button-4
split-payee-on-mobile
twk3/pin-apis-crdt
notes-tag-autocomplete
ts-LoadBackup
dnd-kit
package-upgrades
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
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#2886
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 @Juulz on GitHub (Feb 3, 2026).
Originally assigned to: @MatissJanis, @Copilot on GitHub.
Verified issue does not already exist?
What happened?
After a few large find/replace operations, I found a few duplicated transactions and some missing transactions. Previously reconciled accounts with matching running balances no longer matched.
One transaction, in a closed account, was duplicated twice.
I had to revert to a backup.
How can we reproduce the issue?
How can we reproduce the issue? I don't believe I can reproduce it exactly, nor do I want to mess with my data again. Also, the find/replace works strangely and has no "Apply", you have to press "Enter". Not intuitive at all.
Where are you hosting Actual?
None
What browsers are you seeing the problem on?
No response
Operating System
None
@youngcw commented on GitHub (Feb 3, 2026):
I wonder if rules are getting run after and causing the problems.
@Juulz commented on GitHub (Feb 3, 2026):
That might explain the dupes made in a closed account. That's the one that really threw me!
@youngcw commented on GitHub (Feb 3, 2026):
I looked at the code for the find an replace again. Nothing in there seems like it would change amounts or create transactions. I would check your rules to see if anything jumps out.
@Juulz commented on GitHub (Feb 3, 2026):
Schedule with a rule, but no rule on its own. A couple of the operations were done on "All Accounts" and included about 90 transactions. I tested slowly before I did the whole batch, so I was surprised when a few balances were lower than I remembered when I reconciled an hour before. Hopefully it was a one-off as I was find/replacing an emoji (which is now back for Jan-Jun 2025, but I'm just gonna live with it)
@youngcw commented on GitHub (Feb 3, 2026):
If you could do it again and try to figure out edit caused the issue that would be useful. You should be able to CTRL-Z the changes once you find the one that is breaking things
@Juulz commented on GitHub (Feb 3, 2026):
OK... I went through month by month and it happened. I ctrl-z back and went forward and it was fine. I loaded up the backup again and it happened. I'm touching up screenshots, but I'm thinking it was merged transactions.
@Juulz commented on GitHub (Feb 3, 2026):
In order:
Before find/replace in just the Checking Account:
In All Accounts
Find/replace for 🔗 complete. Closed accounts now have amounts!
Here's All Accounts for that date. The 🔗s are gone. Transactions appeared. Looks like the transactions may have been merged?
@Juulz commented on GitHub (Feb 3, 2026):
Ctrl-z back to before the find/replace, redid the find/replace and no error. Closed accounts are fine:
@youngcw commented on GitHub (Feb 3, 2026):
I wonder if it isn't because of merged transactions, but transfers in general. I wonder if new transfers are getting made
@MatissJanis commented on GitHub (Feb 4, 2026):
Which PR introduced find&replace? We (or I) can ask an AI agent to do a preliminary analysis there to see if it finds any problems.
@Juulz commented on GitHub (Feb 4, 2026):
#6282
@MatissJanis commented on GitHub (Feb 4, 2026):
@copilot please investigate what could cause the problem reported in this issue. It was possibly introduced in #6282