Remove ajax.inc

1. +++ b/core/lib/Drupal/Core/Ajax/AjaxFormProcessor.php
   @@ -0,0 +1,208 @@
   +namespace Drupal\Core\Ajax;
   +use Drupal\Core\Form\FormStateInterface;
   

   missing space.

2. +++ b/core/lib/Drupal/Core/Ajax/AjaxFormProcessor.php
   @@ -0,0 +1,208 @@
   +      // @todo Legacy support. Remove in Drupal 8.
   

   When?

+++ b/core/lib/Drupal/Core/Ajax/AjaxFormProcessor.php
@@ -0,0 +1,208 @@
+class AjaxFormProcessor {

Can we have an interface for this service?

1. +++ b/core/lib/Drupal/Core/Ajax/AjaxFormProcessor.php
   @@ -0,0 +1,208 @@
   +   *   - #ajax['event']
   +   *   - #ajax['prevent']
   +   *   - #ajax['path']
   +   *   - #ajax['options']
   +   *   - #ajax['wrapper']
   +   *   - #ajax['parameters']
   +   *   - #ajax['effect']
   +   *   - #ajax['accepts']
   

   We could point to @group ajax and its excellent documentation at the beginning of ajax.inc

2. +++ b/core/lib/Drupal/Core/Form/FormBuilder.php
   @@ -116,12 +124,13 @@ class FormBuilder implements FormBuilderInterface, FormValidatorInterface, FormS
   +   * @param \Drupal\
   

   One \Drupal to include them all

3. +++ b/core/lib/Drupal/Core/Form/FormBuilder.php
   @@ -753,7 +763,17 @@ public function doBuildForm($form_id, &$element, FormStateInterface &$form_state
   +        if (is_string($process) && strpos($process, '::') === FALSE) {
   +          $callable = $this->controllerResolver->getControllerFromDefinition($process);
   +        }
   +        else {
   +          $callable = $process;
   +        }
   

   Can't the controller resolver handle that automatically internally?

.

Instead of using ControllerResolver, we should probably use #2320491: Add a callable resolver.

Updated the issue summary now that we have #2311393: Remove hook_element_info() and all references to it

Awesome!

OMG OMG OMG OMG AWESOME! At last :) Bye bye, ajax.inc!

This should have a change record or be attached to an existing one.

Added https://www.drupal.org/node/2341041

Conflicted with #2329703: [meta] Spellchecking Drupal, of all things.

#15: ROFL :D

Found a reference to ajax_process_form(), didn't fully re-wrap the docblock, I wanted a minimal diff. Still RTBC, just a minor docs update.

Committed/pushed to 8.0.x, thanks!

Nice one to see green.

publish change record