Commit e374c882 authored by alexpott's avatar alexpott

Issue #2507053 by aneek, joshi.rohit100, tstoeckler, tim.plunkett, Eli-T,...

Issue #2507053 by aneek, joshi.rohit100, tstoeckler, tim.plunkett, Eli-T, yoroy: Configuration Single export form should not have REQUIRED (form-required) attribute
parent ac24ea64
......@@ -105,7 +105,6 @@ public function buildForm(array $form, FormStateInterface $form_state, $config_t
'#type' => 'select',
'#options' => $this->findConfiguration($default_type),
'#default_value' => $config_name,
'#required' => TRUE,
'#prefix' => '<div id="edit-config-type-wrapper">',
'#suffix' => '</div>',
'#ajax' => array(
......@@ -118,7 +117,6 @@ public function buildForm(array $form, FormStateInterface $form_state, $config_t
'#title' => $this->t('Here is your configuration:'),
'#type' => 'textarea',
'#rows' => 24,
'#required' => TRUE,
'#prefix' => '<div id="edit-export-wrapper">',
'#suffix' => '</div>',
);
......
......@@ -82,6 +82,10 @@ function testExport() {
$file_contents = file_get_contents(file_directory_temp() . '/' . 'system.maintenance.yml');
$exported = Yaml::decode($file_contents);
$this->assertNotIdentical($exported['message'], 'Foo');
// Check the single export form doesn't have "form-required" elements.
$this->drupalGet('admin/config/development/configuration/single/export');
$this->assertNoRaw('js-form-required form-required', 'No form required fields are found.');
}
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment