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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dermario created an issue. See original summary.

dermario’s picture

Issue summary: View changes
dermario’s picture

Assigned: Unassigned » dermario
Status: Active » Needs review
FileSize
631 bytes

It seems like the boolean FALSE is not a valid default value for the checkbox. A simple cast to integer seems to solve that problem.

Berdir’s picture

Status: Needs review » Needs work

The 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?

Dave Reid’s picture

Huh, 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.'

dermario’s picture

I followed the suggestion of Dave in #5 and converted the radios into one checkbox and adjusted the tests.

Berdir’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs tests

That works for me.

Yes, the settings form is not very usable, there are probably more things that can be improved.

Will commit this soon.

Berdir’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, committed.

  • Berdir committed 7da80a4 on 8.x-1.x authored by dermario
    Issue #2685591 by dermario: Radio is not checked when choosing "Leave...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.