fix #705

[Sneezry]

fix #705

[codecov]

Codecov Report

Merging #706 (a936d5d) into dev (af907ee) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #706   +/-   ##
=======================================
  Coverage   18.34%   18.34%           
=======================================
  Files          15       15           
  Lines        1488     1488           
  Branches      324      324           
=======================================
  Hits          273      273           
  Misses       1182     1182           
  Partials       33       33           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update af907ee...a936d5d. Read the comment docs.

[mymindstorm]

This is more of a larger issue with how we are checking for encrypted entries rather than just this conditional. With the key encryption, all of these encryption checks should be looking for if there is a key present rather than an encrypted entry. I will look into writing some code for this.

[Sneezry]

This is more of a larger issue with how we are checking for encrypted entries rather than just this conditional. With the key encryption, all of these encryption checks should be looking for if there is a key present rather than an encrypted entry.

I agree. When there is no entries, and a password is set, current logic will just ignore the key.

[mymindstorm]

Closing in favor of #724