Hello all,

I'm using Field Collection: 7.x-1.0-beta2

When I am editing a field collection within a node form, there is a URL Redirect field for each field collection entry. How do I hide this? See attached manage_field_collection.png and node_form.png

I understand I can control what roles see through the permissions page. For now, I'll be hiding via display:none in CSS, however I'd like to just hide this all together even for user 1. I don't see myself ever needing to create a URL alias for a field collection on my site.

Thanks for reading,
-Tim

CommentFileSizeAuthor
#9 field_collection-disable_redirect_field-1340354-9.patch551 bytesrocketeerbkw
#3 1340354.3-field_collection_redirect.patch449 bytesmrfelton
#1 many_url_redirects.png64.9 KB13rac1
node_form.png25.98 KBTimG1
manage_fields.png19.29 KBTimG1

Comments

13rac1’s picture

13rac1 commented
Category: support » bug
StatusFileSize
new many_url_redirects.png64.9 KB

On Node Edit forms, this shows a URL redirect tab for each field collection item.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Closed (duplicate)

It seems this is an issue with the Redirect module and not Field Collection.
Marking as a duplicate of #1263884: Avoid redirect field and form injection on entities that have a path but where this makes no sense (like comments). Please help test the patches in that issue to get this fixed.

mrfelton’s picture

mrfelton commented
Status: Closed (duplicate) » Needs review
StatusFileSize
new 1340354.3-field_collection_redirect.patch449 bytes

This is partially due to the redirect module issue #1263884: Avoid redirect field and form injection on entities that have a path but where this makes no sense (like comments), although, field_collection will need to be updated to support that patch by adding 'redirect' => FALSE to field_collection_entity_info().

Patch attached.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Status: Needs review » Reviewed & tested by the community

@mrfelton, you're correct for reopening this, as it is *not* a duplicate.

Both the patch in #3 plus the patch in comment #11 on #1263884: Avoid redirect field and form injection on entities that have a path but where this makes no sense (like comments) were needed to actually solve this issue for field_collection module.

1) applying #3 without the patch for redirect module has no effect.
2) applying the patch for redirect module without #3 has no effect.

RTBC, and Thanks!

bennos’s picture

bennos commented

works for me. thx

Think it can added in the dev version. We should document that a patch for the redirect module is needed to get it works.

RobW’s picture

RobW commented

Patch looks good, just needs a comment that mentions that the key/value is for integration with another contrib module:

'fieldable' => TRUE,
// For integration with Redirect module.
// @see http://drupal.org/node/1263884
'redirect' => FALSE,
'entity keys' => array(
fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Reviewed & tested by the community » Postponed

If I understand this right, the redirect patch has to go in first.

acrollet’s picture

acrollet commented
Status: Postponed » Active

The patch in #1263884: Avoid redirect field and form injection on entities that have a path but where this makes no sense (like comments) was committed and released in redirect-7.x-1.0-rc1.

rocketeerbkw’s picture

rocketeerbkw commented
Status: Active » Needs review
StatusFileSize
new field_collection-disable_redirect_field-1340354-9.patch551 bytes

Patch #3 with #6. Must clear cache after applying patch.

Status: Needs review » Needs work

The last submitted patch, field_collection-disable_redirect_field-1340354-9.patch, failed testing.

rocketeerbkw’s picture

rocketeerbkw commented
Version: 7.x-1.0-beta2 » 7.x-1.x-dev
Status: Needs work » Needs review
rocketeerbkw’s picture

rocketeerbkw commented

#9: field_collection-disable_redirect_field-1340354-9.patch queued for re-testing.

dynamicdan’s picture

dynamicdan commented

Why isn't this patch re-testing itself?

I could use this patch and be happy : )

markabur’s picture

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

I manually made the change in #9 and it works fine for me, using redirect-7.x-1.0-rc1. RTBC

rocketeerbkw’s picture

rocketeerbkw commented

The patch is postponed because the 7.x-1.x branch is failing tests, it's a pretty minor patch though.

fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Reviewed & tested by the community » Fixed
fago’s picture

fago
Primary language German
Location Vienna
commented

Committed #9, thanks!

Status: Fixed » Closed (fixed)

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