Failed to generate a password / Custom Dictionnary

[regisdenice]

Thank you for making this plugin, I think your plugin has a great potential 🎉 🥳

Goal

Enjoy your plugin in my language ¹

Action

I am building a French dictionary, and I share my 0.1 test version ReadablePassphrase - French 0_1.xml ²

Issue

The Preview tab Failed to generate a password, withe either message:

1. The sequence contains no matching elements ³
2. maxExlusive must be positive
Parameter Name: maxExlusive
The actual value was 0. ⁴

Note; your plugin creates password with my own dictionary 😄 👍 _

How to reproduce ?

1. KeePass > Tools > Generate Password... > Generating using custom algorithm > Readable Passphrase Generator > Options > Use your own dictionary > ReadablePassphrase - French 0_1.xml > OK > Preview tab ³
2. (Then from the Generating using custom algorithm window) > Options > OK > Preview tab ⁴

Request

In order to build (and share) my dictionary, please could you help me understand what I am missing ?

Footnotes

1. I understand better my language than English than C# :D
   I have more trouble remembering the unknown words that might form a sentence that makes no sense but is grammatically almost correct. ↩

2. As I did not find a template, my dictionary is based on:
   your example: https://github.com/ligos/readablepassphrasegenerator/wiki/Make-Your-Own-Dictionary
   and your dictionaries: https://github.com/ligos/readablepassphrasegenerator/tree/master/trunk/MergePartsOfSpeech ↩

3. The sequence contains no matching elements
   
   ↩ ↩²

4. maxExlusive must be positive Parameter Name: maxExlusive The actual value was 0.
   
   ↩ ↩²

[regisdenice]

I forgot to mention that I do not have this issue when using the built-in dictionary.

[ligos]

Hi @regisdenice, looks like you've made a great start to a French dictionary!
I wish you well with your work 🙂

I was able to reproduce your error. It looks like its related to numbers. It appears the plugin assumes a dictionary has at least one number.

I was able to resolve the issue by adding a <numberRange> to your dictionary:

The only way to add numbers is via <numberRange>. That is OK for numerals (like 1, 2, 3, etc). But I'll make another issue to add <number value="one"> or <number value="un">, at some time in the future.

Let me know if this doesn't work.
Murray

[ligos]

#22