open-y-subprojects / openy_custom

Custom modules from Open Y core. Decoupled for ability to use outside of Open Y when needed.
GNU General Public License v3.0
0 stars 14 forks source link

fix(openy_redirect): support disabled field widget #23

Closed andriokha closed 1 year ago

andriokha commented 1 year ago

Fixes #22

Thanks!

froboy commented 1 year ago

@andriokha thanks for this fix. Could you give a little more detailed test instructions so we can validate it? Thanks!

andriokha commented 1 year ago

Sorry @froboy, @podarok was too quick for me (:

For clarity/future reference:

  1. Go to the page to configure a content type's form display.
  2. Hide the redirect field widget.
  3. Edit a node of that content type.

Expected result: there's no sign of the redirect field. Actual result: there's an empty Redirects fieldset and a PHP warning's emitted.

Thanks both!

AnastasiiaPys commented 1 year ago

Hi @andriokha,

could you please add more details for testing? There is no button to configure a content type's form display in our builds.

image
andriokha commented 1 year ago

Hi @AnastasiiaPys, that sounds like it might be connected with the permissions of the user you're currently logged in as. I think you'd need the Content: Administer form display permission.