[PR #5171] [MERGED] Fix if logic error #3568

New issue

Closed

opened 2026-03-03 10:04:45 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 10:04:45 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/5171
Author: @BlackDex
Created: 11/11/2024
Status: ✅ Merged
Merged: 11/11/2024
Merged by: @dani-garcia

Base: main ← Head: fix-logic-issue

📝 Commits (1)

99656bb Fix if logic error

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 src/api/core/accounts.rs (+2 -2)

📄 Description

Fixing a logical error in an if statement where we used && which should have been ||.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/dani-garcia/vaultwarden/pull/5171 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 11/11/2024 **Status:** ✅ Merged **Merged:** 11/11/2024 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `main` ← **Head:** `fix-logic-issue` --- ### 📝 Commits (1) - [`99656bb`](https://github.com/dani-garcia/vaultwarden/commit/99656bbc7da07dc83a456c5e0040bc1603c4ac4f) Fix if logic error ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/api/core/accounts.rs` (+2 -2) </details> ### 📄 Description Fixing a logical error in an if statement where we used `&&` which should have been `||`. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>