hook_menu_alter() had been gone for a long time. We only used it to provide a tab for adding fields to an order. The replacements for this hook in D8 are listed at https://www.drupal.org/node/2118147#alter

case 'field_ui.overview_uc_order' in uc_order_help() will also need to be changed to point to the correct page once the above has been accomplished.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TR created an issue. See original summary.

longwave’s picture

Status: Active » Needs review
FileSize
1.47 KB

I think we can just remove it entirely, for now at least. We don't have a 'field_ui_base_route' defined in OrderProduct, which is the equivalent method of adding the same routes that this hook used to alter.

TR’s picture

That's where you would add a field to the Order entity - where can that be done now?

longwave’s picture

You can add fields to orders at /admin/store/config/orders/fields. You cannot currently add fields to order *products*, which were only partially supported in D7 which is why this alter was needed to tweak the UI for them.

  • longwave committed 7f462dd on 8.x-4.x
    Issue #2681563 by longwave: Eliminate uc_order_menu_alter()
    
longwave’s picture

Status: Needs review » Fixed

Committed, we can make order products properly fieldable later if we need to.

Status: Fixed » Closed (fixed)

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