I stumbled across a @todo saying to remove some code once #1822458: Move dynamic parts (view modes, bundles) out of entity_info() was fixed. That issue is fixed, so here's that removal.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost’s picture

where is the removal of @todo in a patch? looks nice clean-up

tim.plunkett’s picture

Status: Needs review » Reviewed & tested by the community
+++ b/core/modules/field/tests/modules/field_test/field_test.moduleundefined
@@ -269,20 +269,3 @@ function field_test_field_extra_fields_alter(&$info) {
- * @todo Remove when http://drupal.org/node/1822458 is fixed.

Right here.

I remember hitting that field_test_module_implements_alter() when converting this, I don't know why I didn't just remove it. Thanks @effulgentsia!
If it passes we're good here.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Needs a reroll...

curl https://drupal.org/files/entity_bundle_info-no_alter.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3873  100  3873    0     0   5063      0 --:--:-- --:--:-- --:--:--  6643
error: patch failed: core/modules/field/tests/modules/field_test/field_test.entity.inc:50
error: core/modules/field/tests/modules/field_test/field_test.entity.inc: patch does not apply
effulgentsia’s picture

Status: Needs work » Reviewed & tested by the community
Issue tags: -Needs reroll
FileSize
3.79 KB

Straight reroll. Leaving at RTBC.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed bd6ad51 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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