Avoid redirect field and form injection on entities that have a path but where this makes no sense (like comments)

This is the better way to support this. Please test the following patch and add 'redirect' => FALSE to field_collection_entity_info().

Thanks Dave, this works as expected, though instead of patching field_collection.module I altered it in my own custom module.

<?php
/**
 * Implements hook_entity_info_alter().
 */
function MYMODULE_entity_info_alter(&$info) {
  if (isset($info['field_collection_item'])) {
    $info['field_collection_item']['redirect'] = FALSE;
  }
}
?>

I've applied that patch to redirect, and also implemented an alter as described above.
Patch works correctly.
Thanks.

Sounds like this is RTBC to me

I rerolled patch #3, with the added benefit of the site builder being able to choose what entities that the redirect module gets applied too. The additional settings were added to the redirect settings page '/admin/config/search/redirect/settings'. By doing it this way we don't have to worry about applying patches to the field collection module or creating a custom module to implement a cleaner UI. Thanks!

Please leave a UI separate from the issue.

Judging by Dave's comment in #8, this should be RTBC for the patch in #3.
Also marked #1340354: How do I remove URL Redirect Field for a Field Collection? as a duplicate of this issue.

Patch from #3 didn't apply for me, updated patch for latest version of code.

commerce_order entity should be in the stop list also.

And commerce_customer_profile entity should be in the stop list also.

I have been working on a field attach UI module. It has an option that will allow you to hide attached fields for a given entity. I found this might be a better approach to solve this type of issue. It's currently a sandbox project, but once I get some feedback on this module, I plan on releasing a full version.

Please post any issues that you have in the field attach UI module issue queue.

https://drupal.org/sandbox/droath/1349866

The patch in #11 applies cleanly. However I'm not quite sure what it should do - I still get redirect fields on my custom entities for which I've specified 'redirect' => FALSE,

Actually it did work: the Redirect elements are no longer injected into forms for entities that have 'redirect' => FALSE.

But redirect_field_extra_fields() still doesn't respect redirect_entity_type_supports_redirects().

I am having this same issue with Field Collections where URL Redirects is attaching to every instance of a field collection item within the table row. Which one of the above patches will remove it from the GUI? Patch #11?

#11 + #4 works for me.

Ditto, thanks for this.

Patch in #11 + code in 4 works well.

Comment #20++

#11 + #4 works, thanks!

Patch in #11 works for me too. For some reason #4 didn't work even after clearing caches. Implementing the hook_entity_info_alter() from #4 appears to be a requirement but doesn't work without the patch in #11.

Committed to 7.x-1.x with http://drupalcode.org/project/redirect.git/commit/2b091c6

@maximpodorov - I have created patches for commerce with what you requested.

- #1891990: Make sure that redirect module does not inject redirect into the product edit form
- #1892002: Prevent the Redirect module from altering Commerce entity forms.
- #1891998: Make sure that redirect module does not inject redirect into the order edit form

You could go review them.

I've filed a similar issue w/patch for the relation module #2321509: Prevent relation module from altering relation forms.