Actually there are two and a half issues that were solved with this change.
1.) If you duplicate a keymap and specify a name that alredy exists, the label of the dialog is empty as the property was wrongly named
2. and half) If you cancel the first dialog, or the dialog that is shown if you specified a duplicate name, then the process was not cancelled, but a keymap with name "null" was added as the return value of showInputDialog(...) was not checked in both cases.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
also merge 22350 for coding style
Ah, my fault, sorry.
Thanks for fixing
What is the bug that this fixes? Steps to reproduce?
Actually there are two and a half issues that were solved with this change.
1.) If you duplicate a keymap and specify a name that alredy exists, the label of the dialog is empty as the property was wrongly named
2. and half) If you cancel the first dialog, or the dialog that is shown if you specified a duplicate name, then the process was not cancelled, but a keymap with name "null" was added as the return value of showInputDialog(...) was not checked in both cases.
Committed 22379.