Problem/Motivation

Can we get a followup to remove the last remnants of drupal-ajax (used in tests alot) and it's drupal- friends. Or maybe it should be cleanup here? We still have things like:

$response = $this->drupalPost('editor/' . 'node/1/body/en/full', 'application/vnd.drupal-ajax', array());
and

// We set up a request so it looks like an request in the live preview.
$request = new Request();
$request->setFormat('drupal_ajax', 'application/vnd.drupal-ajax');
$request->headers->set('Accept', 'application/vnd.drupal-ajax');

Proposed resolution

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#6 2502865-6.patch14.72 KBclaudiu.cristea
#1 2502865-1.patch14.72 KBclaudiu.cristea

Comments

claudiu.cristea’s picture

Status: Active » Needs review
StatusFileSize
new14.72 KB

Patch.

claudiu.cristea’s picture

Assigned: Unassigned » dawehner

@dawehner, This cleanup issue was opened by you. If you don't have time for review just unassign.

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

This looks perfect for me

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 1: 2502865-1.patch, failed testing.

claudiu.cristea’s picture

Status: Needs work » Needs review
StatusFileSize
new14.72 KB

rerolled

jibran’s picture

Status: Needs review » Reviewed & tested by the community

Back to RTBC as per #4.

wim leers’s picture

+1 for the changes in editor and quickedit.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 6: 2502865-6.patch, failed testing.

dawehner’s picture

Assigned: dawehner » Unassigned
Status: Needs work » Reviewed & tested by the community

Yet another random failure

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 6: 2502865-6.patch, failed testing.

claudiu.cristea’s picture

Status: Needs work » Reviewed & tested by the community

random failure

  • catch committed 22c915f on 8.2.x
    Issue #2502865 by claudiu.cristea: Remove all remaining usages of the...
catch’s picture

Version: 8.1.x-dev » 8.2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed 22c915f and pushed to 8.2.x. Thanks!

Didn't commit to 8.1.x since this is 100% clean-up.

Status: Fixed » Closed (fixed)

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