Per https://www.drupal.org/node/2310411 $form_state is now a classed object. devel and devel_generate need to be updated accordingly.

To get all tests to pass successfully, #2313803: MENU_MAX_DEPTH has been obsoleted needs to be fixed first.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

anavarre’s picture

Title: $form_state returns PHP Fatal errors » $form_state is now a classed object

Status: Needs review » Needs work

The last submitted patch, FormStateInterface_changes.patch, failed testing.

anavarre’s picture

Okay, there's more.

[Thu Jul 31 16:04:39.381386 2014] [:error] [pid 16869] [client 127.0.0.1:53608] PHP Fatal error: Call to a member function get() on a non-object in /var/www/html/head/modules/devel/src/Form/SettingsForm.php on line 30

anavarre’s picture

FileSize
8.39 KB

Here's a new attempt. I've simply added Request $request = NULL to buildform() in SettingsForm.php

anavarre’s picture

FileSize
476 bytes

And the interdiff.

anavarre’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: 2313207-4.patch, failed testing.

anavarre’s picture

FileSize
9.67 KB
1.17 KB

And the devel_generate failure.

anavarre’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 8: 2313207-8.patch, failed testing.

anavarre’s picture

It's failing because of:

public function settingsForm(array $form, array &$form_state) {
  $form['num'] = array(
    '#type' => 'textfield',
    '#title' => t('How many users would you like to generate?'),
    '#default_value' => $this->getSetting('num'),
    '#size' => 10,
  );

Which throws:

Argument 2 passed to Drupal\devel_generate\Plugin\DevelGenerate\UserDevelGenerate::settingsForm() must be of the type array, object given, called in /var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/devel/devel_generate/src/Form/DevelGenerateForm.php on line 77 and defined

anavarre’s picture

Status: Needs work » Needs review
FileSize
9.67 KB
514 bytes

Stupid typo.

Status: Needs review » Needs work

The last submitted patch, 12: 2313207-12.patch, failed testing.

anavarre’s picture

Status: Needs work » Needs review
FileSize
10.52 KB
751 bytes

Tests should fail later now.

Status: Needs review » Needs work

The last submitted patch, 14: 2313207-14.patch, failed testing.

anavarre’s picture

Status: Needs work » Needs review
FileSize
12.44 KB
1.6 KB

More fixes.

Status: Needs review » Needs work

The last submitted patch, 16: 2313207-16.patch, failed testing.

anavarre’s picture

Status: Needs work » Needs review
FileSize
17.2 KB
4.06 KB

Finished converting $form_state to a classed object.

Status: Needs review » Needs work

The last submitted patch, 18: 2313207-18.patch, failed testing.

anavarre’s picture

Seems like the scope of this issue is fixed. https://qa.drupal.org/pifr/test/835538 is throwing errors with MENU_MAX_DEPTH that are related to https://www.drupal.org/node/2302069

anavarre’s picture

anavarre’s picture

moshe weitzman’s picture

Status: Needs work » Fixed

Committed. Thanks.

Status: Fixed » Closed (fixed)

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