Skip to content

Fix renaming extension key name in Database Settings#11354

Merged
droidmonkey merged 2 commits intokeepassxreboot:developfrom
varjolintu:fix/rename_extension_key
Dec 26, 2024
Merged

Fix renaming extension key name in Database Settings#11354
droidmonkey merged 2 commits intokeepassxreboot:developfrom
varjolintu:fix/rename_extension_key

Conversation

@varjolintu
Copy link
Copy Markdown
Member

@varjolintu varjolintu commented Oct 12, 2024

When renaming a stored connection key for Browser Integration in Database Settings the key for Created timestamp is not renamed. This causes an empty value to be shown for the renamed key. Also removes the Created key when deleting stored key.

Introduces a static method CustomData::getKeyWithPrefix() instead of different concatenations in various classes.

Fixes #11353

Testing strategy

Manually.

Type of change

  • ✅ Bug fix (non-breaking change that fixes an issue)

@varjolintu
Copy link
Copy Markdown
Member Author

Fixed.

@phoerious phoerious added pr: bugfix Pull request fixes a bug and removed bug labels Nov 22, 2024
@droidmonkey droidmonkey merged commit 9670a5e into keepassxreboot:develop Dec 26, 2024
@varjolintu varjolintu deleted the fix/rename_extension_key branch December 26, 2024 07:22
@droidmonkey droidmonkey added the pr: backported Pull request backported to previous release label Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: Browser pr: backported Pull request backported to previous release pr: bugfix Pull request fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Renaming browser extension keys removes "Created" datetime

3 participants