Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.