Follow up for #2004428: Less ugly operations altering

Updated: Comment #0

Problem/Motivation

We did good in #2004428: Less ugly operations altering, but we still need to remove the alter for blocks.

Proposed resolution

Remove the alter once #2027857: Blocks operations cannot be altered is in.

Remaining tasks

TBD

User interface changes

No.

API changes

No.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

YesCT’s picture

Status: Postponed » Needs review
FileSize
588 bytes
588 bytes
1.14 KB

for fun...
takes out blocks.

and then, since it was (right now) the last one, takes out the alter.

Should fail. Then we can move back to postponed.

Status: Needs review » Needs work

The last submitted patch, config_translation.code_.2035407-1b.patch, failed testing.

YesCT’s picture

Status: Needs work » Needs review
FileSize
1.06 KB

oops. :) wrong diff to remove the whole thing. Here it is. (should still fail though.)

Status: Needs review » Needs work

The last submitted patch, config_translation.code_.2035407-1b-fixed.patch, failed testing.

YesCT’s picture

Status: Needs work » Postponed

good. :)
back to postponed on #2027857: Blocks operations cannot be altered

Gábor Hojtsy’s picture

Status: Postponed » Fixed

I actually committed this change in an independent commit. Sorry I did not credit you on it.

Status: Fixed » Closed (fixed)

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