CommentFileSizeAuthor
#2 3112366-2.patch1.38 KBhardik_patel_12

Comments

Hardik_Patel_12 created an issue. See original summary.

hardik_patel_12’s picture

hardik_patel_12 commented
StatusFileSize
new 3112366-2.patch1.38 KB

Kindly review a patch.

hardik_patel_12’s picture

hardik_patel_12 commented
Assigned: hardik_patel_12 » Unassigned
Status: Needs work » Needs review
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Nice find, this should have been caught in #3097428: Remove node.module BC layers

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

This was not deprecated at all well... the entity repository was nullable but it was required - ho hum. This looks fine. I've checked contrib and the only module that I can find that is impacted is http://grep.xnddx.ru/node/29809874

  • alexpott committed a98801d on 9.0.x 
    Issue #3112366 by Hardik_Patel_12: Remove all @deprecated code from node...

Status: Fixed » Closed (fixed)

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