It's very minor, but the return value descriptions for \Drupal\Core\Entity\Sql\DefaultTableMapping::allowsSharedTableStorage() and \Drupal\Core\Entity\Sql\DefaultTableMapping::requiresDedicatedTableStorage() are wrong, probably due to copy-pasting (they are exactly the same, but both wrong).

CommentFileSizeAuthor
#2 2953880-2--fix_DefaultTableMapping_doc_comments.patch1.38 KBdrunken monkey

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Active » Needs review
StatusFileSize
new 2953880-2--fix_DefaultTableMapping_doc_comments.patch1.38 KB
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

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

Committed and pushed 2ce829ab81 to 8.6.x and 6d19b3cad4 to 8.5.x. Thanks!

  • alexpott committed 2ce829a on 8.6.x 
    Issue #2953880 by drunken monkey: Fix doc comments for...

  • alexpott committed 6d19b3c on 8.5.x 
    Issue #2953880 by drunken monkey: Fix doc comments for...

Status: Fixed » Closed (fixed)

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