Mirror/citra
1
0
Fork 0
mirror of https://github.com/PabloMK7/citra.git synced 2024-11-20 06:24:17 +00:00
Code Issues Projects Releases Packages Wiki Activity

swkbd: Fix digit filter

Browse source 
The DIGIT filter was incorrectly implemented as preventing all digits. It actually limits the maximum digit count to max_digits, according to ctrulib and hardware testing.
This commit is contained in:
zhupengfei 2020-02-04 16:56:30 +08:00
parent eb0364dd5f
commit 05a82b15e6
No known key found for this signature in database
GPG key ID: DD129E108BD09378
2 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/core/frontend/applets/swkbd.cpp
View file

@ -15,9 +15,9 @@ namespace Frontend {
ValidationError SoftwareKeyboard::ValidateFilters(const std::string& input) const {
if (config.filters.prevent_digit) {
if (std::any_of(input.begin(), input.end(),
[](unsigned char c) { return std::isdigit(c); })) {
return ValidationError::DigitNotAllowed;
if (std::count_if(input.begin(), input.end(),
[](unsigned char c) { return std::isdigit(c); }) > config.max_digits) {
return ValidationError::MaxDigitsExceeded;
}
}
if (config.filters.prevent_at) {

5
src/core/frontend/applets/swkbd.h
View file

@ -47,8 +47,7 @@ struct KeyboardConfig {
bool has_custom_button_text; /// If true, use the button_text instead
std::vector<std::string> button_text; /// Contains the button text that the caller provides
struct Filters {
bool prevent_digit; /// Disallow the use of more than a certain number of digits
/// TODO: how many is a certain number
bool prevent_digit; /// Limit maximum digit count to max_digits
bool prevent_at; /// Disallow the use of the @ sign.
bool prevent_percent; /// Disallow the use of the % sign.
bool prevent_backslash; /// Disallow the use of the \ sign.
@ -68,7 +67,7 @@ enum class ValidationError {
// Button Selection
ButtonOutOfRange,
// Configured Filters
DigitNotAllowed,
MaxDigitsExceeded,
AtSignNotAllowed,
PercentNotAllowed,
BackslashNotAllowed,