'delete user self max post count' لا يقبل '-1' لتعطيله

Arkshine · 11 نوفمبر 2023، 2:11م

يبدو أن المشكلة تم تقديمها هنا:

github.com/discourse/discourse

DEV: Sanitize integer site settings in front- and back-end

main ← dev/integer-site-setting-fields

opened 02:48PM - 06 Oct 23 UTC

Drenmi

+20 -1

### What is this change? Currently, if you set an integer site setting in the… admin interface and include thousands separators, you will silently configure the wrong value. **Example:** ``` 1,000 == 1 1.000 == 1 1 000 == 1 ``` This PR replaces `TextField` inputs for integer site settings with `NumberField`. It also cleans the numeric input of any non-digits in the backend in case any separators make it through. ### Not in this PR - The actual value that is set in the back-end is not sent back to the front-end. We probably should. - We should probably change the latter to use [native number input](https://caniuse.com/input-number) in a separate PR. - The `SiteSettingsController#update` method is pretty big. I'm considering extracting the logic into a Ruby object that can be tested in isolation, but not in this PR.

ومن المحتمل أن تكون مرتبطة بعدم توفير @min= هنا:

https://github.com/discourse/discourse/blob/main/app/assets/javascripts/admin/addon/components/site-settings/integer.hbs

الموضوع		الردود	مرات العرض
Thousand separator silently ruins inputted value in settings Bug fixed	7	763	8 يوليو 2024
SiteSetting numbers not properly validated Bug	9	1830	23 أبريل 2015
Site Setting Integer Validation Issue UX	2	681	23 يونيو 2020
Internal server error trying to access site_settings Bug	4	1386	23 أبريل 2015
How do I remove the option for users to delete themselves? Product Support	1	555	27 يناير 2023

'delete user self max post count' لا يقبل '-1' لتعطيله

الموضوعات ذات الصلة