Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I detected that the radio button Leave case the same as source token values. on admin/config/search/path/settings is unchecked if you select that value. Please see the attached screenshot. As far as i can say the value for case is stored correctly and the functionality is given. It seems to be a UI - Issue.
Comment | File | Size | Author |
---|---|---|---|
#6 | radio_is_not_checked-2685591-6.patch | 2.53 KB | dermario |
#3 | radio_is_not_checked-2685591-3.patch | 631 bytes | dermario |
pathauto-case-unchecked.png | 64.11 KB | dermario |
Comments
Comment #2
dermarioComment #3
dermarioIt seems like the boolean FALSE is not a valid default value for the checkbox. A simple cast to integer seems to solve that problem.
Comment #4
BerdirThe values of those constants is 0 and 1, not TRUE and FALSE.
We should change the type of the schema to integer then. Then it will be saved as that and should work correctly.
Now that the other issue is committed, it should be possible to check this as well without too much additional code?
Comment #5
Dave ReidHuh, I didn't realize this radio UI came from D7. I would actually propose to simplify just with a checkbox for '[x] Convert token values to lowercase.'
Comment #6
dermarioI followed the suggestion of Dave in #5 and converted the radios into one checkbox and adjusted the tests.
Comment #7
BerdirThat works for me.
Yes, the settings form is not very usable, there are probably more things that can be improved.
Will commit this soon.
Comment #8
BerdirThanks, committed.