In the current BlockInterface, there are pairs of methods like access()/blockAccess() and submit()/blockSubmit().

With #1871696: Convert block instances to configuration entities to resolve architectural issues, the wrapping methods are moved to the Block entity class, so only the blockFoo() methods are left, and should be renamed.

CommentFileSizeAuthor
#5 block-1884860-5.patch25.52 KBtim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Issue tags: +Blocks-Layouts

tagging

xjm’s picture

#1875770: Reconsider BlockBase::block*() method names is no longer relevant with this issue, so closing that as duplicate.

xjm’s picture

Issue tags: +Block plugins
tim.plunkett’s picture

Tagging.

tim.plunkett’s picture

Status: Postponed » Needs review
FileSize
25.52 KB

We can only do settings and build for now, the rest are still being worked out.

Status: Needs review » Needs work
Issue tags: -Blocks-Layouts, -Block plugins

The last submitted patch, block-1884860-5.patch, failed testing.

tim.plunkett’s picture

Status: Needs work » Needs review
Issue tags: +Blocks-Layouts, +Block plugins

#5: block-1884860-5.patch queued for re-testing.

EclipseGc’s picture

Status: Needs review » Reviewed & tested by the community

RTBC

Eclipse

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Yes, please.

Committed and pushed to 8.x. Thanks!

pcambra’s picture

Shall we update this change notice? http://drupal.org/node/1880620

xjm’s picture

Aye, please do.

pcambra’s picture

Change notice updated :)

tim.plunkett’s picture

Assigned: tim.plunkett » Unassigned

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