Adding new field to content type in overlay, will redirect you to"#overlay=admin/structure/types%3Frender%3Doverlay" instead of #overlay=admin/structure/types/manage/%content type/fields
This behavior is not present when overlay is disabled or field is edited.

CommentFileSizeAuthor
#4 field-ui-destinations-querystring-798292-4.patch888 bytesDavid_Rothstein
#2 field_ui-destinationsquerystring.patch1022 bytescasey

Comments

kcybulski’s picture

kcybulski commented
Title: Overlay redirect to list of content types insteed of list of fields after field is added. » Overlay redirect to list of content types instead of list of fields after field is added.
casey’s picture

casey commented
Component: overlay.module » field_ui.module
Status: Active » Needs review
StatusFileSize
new field_ui-destinationsquerystring.patch1022 bytes

It's actually a fields_ui issue incorrectly redirection URLs containing a querystring. Attached path should fix this.

casey’s picture

casey commented
Priority: Normal » Critical
David_Rothstein’s picture

David_Rothstein commented
StatusFileSize
new field-ui-destinations-querystring-798292-4.patch888 bytes

Nice find!

I think the attached patch would be a little cleaner, though. Drupal 7 introduced an API function that seems perfectly suited to this purpose...

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD, but this is terribly stretching the definition of 'critical' (or even 'major') :P

Status: Fixed » Closed (fixed)

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

mustanggb’s picture

mustanggb commented
Priority: Critical » Major