When placing a block, the Place blocks button is displayed on the Configure block page.

Steps to reproduce:

  1. Navigate to admin/structure/block (Structure -> Blocks)
  2. Click on the Place blocks button
  3. Choose a block to place and click its Place block button

There is both a Place blocks button at the top of the page and a Save block button at the bottom of the page.

Clicking the Save block button operates as expected. Clicking the Place blocks button takes you back to the list of available blocks to choose a block to place. Usually only the second block is placed, but I had it place both blocks once.

Either this is a bug in that the button is there and shouldn't be, or it's a bug in that it's behavior does not allow one to place multiple blocks in a single run through. I'm guessing the former, but I might be wrong.

As an FYI, the Place blocks button does not show up if an already placed block is configured.

CommentFileSizeAuthor
#3 block-2031467-3.patch719 bytestim.plunkett
place-blocks.png55.22 KBoadaeh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

benjy’s picture

Assigned: Unassigned » benjy
tim.plunkett’s picture

Assigned: benjy » tim.plunkett
Status: Active » Postponed
tim.plunkett’s picture

Assigned: tim.plunkett » Unassigned
Status: Postponed » Needs review
Issue tags: +Needs tests, +Block UI overhaul
FileSize
719 bytes

I was mistaken! We're just missing a hook_menu entry for that route, so menu_get_item() is picking up the wrong one.

benjy’s picture

Status: Needs review » Postponed

#2058321: Move the 'place block' UI into the block listing removes the place block page so postponing until that is resolved.

tim.plunkett’s picture

Status: Postponed » Needs work

Sorry that text is wrong. It's the Configure block page. This is still a bug regardless of that issue.

benjy’s picture

Status: Needs work » Reviewed & tested by the community
Issue tags: -Needs tests

This looks good. Don't think we need a test to check for the absence of a button on a page it should never have been on.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Huh. No idea why this fixes it, but I can confirm it fixes it. Woohoo.

Committed and pushed to 8.x. Thanks!

tim.plunkett’s picture

menu_get_item() is still tied to menu_router, so when the local action code is trying to determine what page it was on, even while on admin/structure/block/add/bartik/some_block, it didn't have an entry, so it assumed it was on admin/structure/block. Now it knows what page its on.

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