Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Order UI contains a dependency on Field UI in the .info file. This seems unnecessary.
We don't use Field UI on production environments, but users should be able to view their own orders.
As far as I can see, this dependency can be safely removed.
Comment | File | Size | Author |
---|---|---|---|
#3 | remove_dependency_on-2521768-3.patch | 960 bytes | BarisW |
#1 | remove_dependency_on-2521768-1.patch | 470 bytes | BarisW |
Comments
Comment #1
BarisW CreditAttribution: BarisW at LimoenGroen commentedComment #2
rszrama CreditAttribution: rszrama commentedTagging for http://contribkanban.com/#/board/commerce/7.x-1.x.
Comment #3
BarisW CreditAttribution: BarisW at LimoenGroen commentedThis also goes for Product UI. Not need to require Field UI.
Comment #4
rszrama CreditAttribution: rszrama commentedCurious. Looks like my tag didn't take. : P
Comment #5
ptmkenny CreditAttribution: ptmkenny commentedThis works for me. It doesn't make sense to have field_ui enabled on production.
Comment #6
rszrama CreditAttribution: rszrama at Centarro commentedEven with this patch, there are other modules that depend on the Field UI module. The problem is the Commerce related UI modules end up being nigh useless without Field UI ... we don't directly invoke Field UI modules, but we do use the UI modules to add admin paths to our entity types for the Field UI module to throw the manage fields / manage display interfaces up on.
I'm not entirely comfortable with this change, though I wouldn't necessarily rule it out for the future.
Comment #7
bojanz CreditAttribution: bojanz at Centarro commentedIt's safe to say this won't happen in 1.x.