Thanks for the nifty module.

I'm not entirely sure this is expected behaviour but currently when we visit a user edit page for a user without permission to see X field we still see the markup around X field even though the contents are not there.

See the attached image as an example. The Twitter text field input is removed but the markup of the typical field markup remains.

Is this expected behaviour or do you think we have something interfering with the rendering of these fields like the theme?

CommentFileSizeAuthor
#8 multiple_registration_markup_remains_2908303-8.patch1.31 KBysamoylenko
#7 multiple_registration_markup_remains_2908303-7.patch1.31 KBysamoylenko
Screen Shot 2017-09-12 at 11.00.59 AM.png30.29 KBpribeh

Comments

pribeh created an issue. See original summary.

pribeh’s picture

pribeh commented
Issue summary: View changes
sillo’s picture

sillo commented

Any plans on fixing this? I see this issue was created about a year ago, and no replies so far.

ysamoylenko’s picture

ysamoylenko commented
Issue tags: +epam-contrib-2019.03
rodman1980’s picture

rodman1980 commented
Issue tags: -epam-contrib-2019.03
ysamoylenko’s picture

ysamoylenko commented
ysamoylenko’s picture

ysamoylenko commented
Status: Active » Needs review
StatusFileSize
new multiple_registration_markup_remains_2908303-7.patch1.31 KB

Hello, I've investigated the problem.

It is related to the core 'field' module logic (field_access hook), it restores the field wrapper to the page.
I've implemented the patch for solving the problem but it needs to be tested.
Please try to apply it.

ysamoylenko’s picture

ysamoylenko commented
StatusFileSize
new multiple_registration_markup_remains_2908303-8.patch1.31 KB

I've updated the patch. (Removed extra whitespace)

IreneV’s picture

IreneV commented
Status: Needs review » Reviewed & tested by the community

Reviewed & tested. Looks good!

  • ysamoylenko committed cdc7f30 on 7.x-1.x 
    Issue #2908303 by ysamoylenko, pribeh, IreneV: Markup remains
ysamoylenko’s picture

ysamoylenko commented
Status: Reviewed & tested by the community » Fixed

The patch was committed. Thank you for the report and review.

Status: Fixed » Closed (fixed)

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