Add a profile field and in the category include a / (forward slash), e.g. "A/P Information".

Then when you go to edit a user, the tab to get to the profile fields will link to a url that looks something like:

"user/2/edit/A/P+Information"

which is of course wrong, since the last slash is being treated as a path separator.

CommentFileSizeAuthor
#15 jsomers_183457_2-D7-15.patch1.52 KBmgifford
#7 jsomers_183457_2-D7.patch1.62 KBj.somers
#2 jsomers_183457_1-D7.patch967 bytesj.somers

Comments

bradweikel’s picture

bradweikel commented
Version: 5.2 » 7.x-dev

Still an issue in D7 branch

j.somers’s picture

j.somers commented
Status: Active » Needs review
StatusFileSize
new jsomers_183457_1-D7.patch967 bytes

When creating my own custom profile fields I always used pretty clear and simple names. I think it should be safe to assume they can only contain letters, numbers, dashes, underscores and whitespace characters.

Status: Needs review » Needs work

The last submitted patch, jsomers_183457_1-D7.patch, failed testing.

j.somers’s picture

j.somers commented
Status: Needs work » Needs review

#2: jsomers_183457_1-D7.patch queued for re-testing.

bradweikel’s picture

bradweikel commented
Status: Needs review » Needs work

For usability, any character restrictions enforced on submission should also be documented in the form element's description, so users don't have to learn about the restriction by trial and error.

j.somers’s picture

j.somers commented

True, I'll try to provide an updated patch later today.

j.somers’s picture

j.somers commented
Status: Needs work » Needs review
StatusFileSize
new jsomers_183457_2-D7.patch1.62 KB

Attached is an updated patch which adds the restriction information to the field description.

mgifford’s picture

mgifford
he/him
Primary language English
commented

#7: jsomers_183457_2-D7.patch queued for re-testing.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Issue tags: +profile module

Heck, I'm having trouble finding the category field in the profile module. How are we supposed to be grouping user fields in D7?

mgifford’s picture

mgifford
he/him
Primary language English
commented

7: jsomers_183457_2-D7.patch queued for re-testing.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Version: 7.x-dev » 8.x-dev
Issue summary: View changes

Is this still an issue for D8?

Status: Needs review » Needs work

The last submitted patch, 7: jsomers_183457_2-D7.patch, failed testing.

The last submitted patch, 7: jsomers_183457_2-D7.patch, failed testing.

swentel’s picture

swentel commented
Version: 8.0.x-dev » 7.x-dev

Profile is gone in D8, so no.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Status: Needs work » Needs review
StatusFileSize
new jsomers_183457_2-D7-15.patch1.52 KB

Ok, here's a re-roll then of the patch from #7.

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.