Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Aron Novak created an issue. See original summary.

Aron Novak’s picture

oriol_e9g’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Needs work
+++ b/core/modules/workflows/src/Form/WorkflowEditForm.php
@@ -83,7 +83,7 @@ public function form(array $form, FormStateInterface $form_state) {
-      '#collapsible' => 'FALSE',
+      '#collapsible' => FALSE,

This line can be removed - it does nothing. As far as I can see there is no support for #collapsible in core.

alexpott’s picture

maliknaik’s picture

@alexpott, Removed the line '#collapsible' => 'FALSE'

maliknaik’s picture

Status: Needs work » Needs review
idebr’s picture

Title: Boolean - string confusion » States details element uses unsupported '#collapsible' attribute
Version: 8.6.x-dev » 8.8.x-dev
Assigned: Aron Novak » Unassigned
Status: Needs review » Reviewed & tested by the community
  • Updated the issue summary to more accurately reflect the patch.
  • Manually verified the patch applies correctly to 8.8.x
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed b52fe53 and pushed to 8.8.x and 8.7.x. Thanks!

  • alexpott committed b52fe53 on 8.8.x
    Issue #3036461 by Aron Novak, maliknaik, alexpott: States details...

  • alexpott committed 304a556 on 8.7.x
    Issue #3036461 by Aron Novak, maliknaik, alexpott: States details...

Status: Fixed » Closed (fixed)

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