[WIP] Resolve references before copying entries to another database

[Copilot]

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original issue description:

Overview

REF for username/password in other (but also already opened) DB

Examples

1. clone entry, create in clone REF: for username and password for original entry in same DB
2. clone's username and password refer to correct values
3. move the clone to another also opened db
4. clone's username and password don't refer to correct values anymore, they're just empty

Context

I open a master DB. In it a slave DB is additionally opened via theAutoOpen group. So if I open the master DB, the other one is "daisy chained". I'd like to use REF for entries not only in one DB but across these DBs.

Fixes #7450.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[codecov]

Codecov Report

Attention: Patch coverage is 94.73684% with 1 line in your changes missing coverage. Please review.

Project coverage is 63.91%. Comparing base (c4b4be4) to head (af5261d).
Report is 12 commits behind head on develop.

Files with missing lines	Patch %	Lines
src/core/Entry.cpp	94.74%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #12201      +/-   ##
===========================================
+ Coverage    63.89%   63.91%   +0.02%     
===========================================
  Files          369      369              
  Lines        38943    38962      +19     
===========================================
+ Hits         24882    24901      +19     
  Misses       14061    14061              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[droidmonkey]

Try this plan again

[Copilot]

Try this plan again

I've implemented a solution that resolves references before moving entries to different databases. The fix adds a new method resolveReferencesBeforeDatabaseMove() that's called in Entry::setGroup() when an entry is moved between databases. This ensures references are resolved using the original database before the move, preserving the actual values instead of leaving broken references. Added test case to verify the fix works correctly. Commit c5a9b46.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• 224.0.0.251
• esm.ubuntu.com
  • Triggering command: /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list

[droidmonkey]

Check your solution again now that you have new instructions and environment.

[Copilot]

Check your solution again now that you have new instructions and environment.

I've reviewed and improved my solution with the new instructions and environment. The fix now properly handles edge cases where references cannot be resolved - instead of replacing them with empty strings, it preserves the original reference text. All tests pass including the comprehensive cross-database reference test. Applied code formatting and updated translations as required. Commit af5261d.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

• 184.155.130.134.in-addr.arpa
  • Triggering command: /home/REDACTED/work/keepassxc/keepassxc/build/tests/testicondownloader (dns block)
• de.login.keepassxc.org
  • Triggering command: /home/REDACTED/work/keepassxc/keepassxc/build/tests/testicondownloader (dns block)
• keepassxc.co.uk
  • Triggering command: /home/REDACTED/work/keepassxc/keepassxc/build/tests/testicondownloader (dns block)
• keepassxc.org
  • Triggering command: /home/REDACTED/work/keepassxc/keepassxc/build/tests/testicondownloader (dns block)
• login.keepassxc.co.uk
  • Triggering command: /home/REDACTED/work/keepassxc/keepassxc/build/tests/testicondownloader (dns block)
• login.keepassxc.org
  • Triggering command: /home/REDACTED/work/keepassxc/keepassxc/build/tests/testicondownloader (dns block)
• test.com
  • Triggering command: /home/REDACTED/work/keepassxc/keepassxc/build/tests/testicondownloader (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to my firewall allow list