If you go to admin/config/people/accounts and enable signatures, the 'User signatures' field is not available on the 'Manage form display' page.

Files: 
CommentFileSizeAuthor
#2 2321621-2.patch2.47 KBswentel
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 75,251 pass(es). View
#2 2321621-fail-2.patch1.88 KBswentel
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 75,164 pass(es), 1 fail(s), and 0 exception(s). View

Comments

xjm’s picture

Uhoh. :)

swentel’s picture

Status: Active » Needs review
Issue tags: -Needs tests
FileSize
1.88 KB
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] 75,164 pass(es), 1 fail(s), and 0 exception(s). View
2.47 KB
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 75,251 pass(es). View

Clearing the field definition cache makes the signature field popup in manage form display. Toggling the checkbox was probably fixed somewhere else. Manage display is not there, that will happen when #1548204: Remove user signature and move it to contrib tlands, so we could simply get the patch in and then follow up there. Note, we could also re-scope that other patch to start using formatters and widgets because that's now possible on base fields.

Attached fail test + fix.

The last submitted patch, 2: 2321621-fail-2.patch, failed testing.

sun’s picture

Status: Needs review » Reviewed & tested by the community
jhodgdon’s picture

Status: Reviewed & tested by the community » Needs work

This test does not test what I reported in this issue. The test is verifying that the signature field is available on admin/config/people/accounts/form-display. I reported that the signature checkbox on admin/config/people/accounts doesn't work.

It looks like maybe that box on admin/comfing/people should be removed, but this patch doesn't do that either.

swentel’s picture

Status: Needs work » Needs review

@jhodgdon

See my comments in #2

Toggling the checkbox was probably fixed somewhere else

So checking that box now actually works (no idea where it was fixed though) ... the test also makes sure that field definitions are flushed and the field shows up in manage form display. If anything, we should probably update the issue title, because there's an actual bug here, but not anymore trying to enable or disable the checkbox.

jhodgdon’s picture

Yeah the summary needs an update. This issue seems totally different from what I actually reported.

swentel’s picture

Issue summary: View changes
swentel’s picture

Should be fine now.

swentel’s picture

Status: Needs review » Reviewed & tested by the community

Setting back to RTBC as per #4

  • webchick committed aca1ec3 on 8.0.x
    Issue #2321621 by swentel: Fixed Use signatures checkbox on User config...
webchick’s picture

Status: Reviewed & tested by the community » Fixed

I manually tested and signatures do seem to be working for me, so I think jhodgdon's original bug is fixed.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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