IJHack / QtPass / 27703354012

55%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(#1511): snapshot AppSettings in QtPass event handlers (#1559)

* refactor(#1511): snapshot AppSettings in QtPass event handlers (#1559)

Replace per-call QtPassSettings getter boilerplate in doGitPush,
setClippedText, setClipboardTimer, copyTextToClipboard, showTextAsQRCode,
and mountWebDav with a single QtPassSettings::load() snapshot.

Adds inline fallback for empty qrencodeExecutable (SettingsSerializer
applies no defaults; replicates the previous getter default).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* refactor(#1511): revert single-field load() calls to targeted getters

doGitPush and setClipboardTimer each read one field, making a full
QtPassSettings::load() snapshot (which deserialises ~30 keys) wasteful.
Revert to isAutoPush() and getAutoclearSeconds() respectively.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Coverage Stats

0 of 13 new or added lines in 1 file covered. (0.0%)

1 existing line in 1 file now uncovered.

3963 of 6943 relevant lines covered (57.08%)

23.24 hits per line