Fix nested accordion open

demos/interactive/accordion-nested.php

@@ -5,25 +5,17 @@
 namespace Atk4\Ui\Demos;
 
 use Atk4\Ui\Accordion;
-use Atk4\Ui\Button;
 use Atk4\Ui\Form;
 use Atk4\Ui\Header;
 use Atk4\Ui\LoremIpsum;
 use Atk4\Ui\Message;
-use Atk4\Ui\View;
 
 /** @var \Atk4\Ui\App $app */
 require_once __DIR__ . '/../init-app.php';
 
-/*
-Button::addTo($app, ['View Form input split in Accordion section', 'class.small right floated basic blue' => true, 'iconRight' => 'right arrow'])
-    ->link(['accordion-in-form']);
-View::addTo($app, ['ui' => 'clearing divider']);
-*/
-
 Header::addTo($app, ['Nested accordions']);
 
-$addAccordionFunc = function ($view, $maxDepth = 2, $level = 0) use (&$addAccordionFunc) {
+$addAccordionFunc = function ($view, $maxDepth, $level = 0) use (&$addAccordionFunc) {
     $accordion = Accordion::addTo($view, ['type' => ['styled', 'fluid']]);
 
     // static section
@@ -61,4 +53,4 @@
 };
 
 // add accordion structure
-$addAccordionFunc($app);
+$addAccordionFunc($app, 2);

src/Accordion.php

@@ -45,7 +45,6 @@ public function addSection($title, \Closure $callback = null, $icon = 'dropdown'
         // if there is callback action, then use VirtualPage
         if ($callback) {
             $section->virtualPage = VirtualPage::addTo($section, ['ui' => '']);
-            $section->virtualPage->stickyGet('__atk-dyn-section', '1');
             $section->virtualPage->set($callback);
         }
 
@@ -152,22 +151,15 @@ public function getSectionIdx(AccordionSection $section)
         return $idx;
     }
 
-    /**
-     * Check if accordion section is dynamic.
-     */
-    public function isDynamicSection(): bool
-    {
-        return isset($_GET['__atk-dyn-section']);
-    }
-
     protected function renderView(): void
     {
         if ($this->type) {
             $this->addClass($this->type);
         }
 
-        // Only set Accordion in Top container. Otherwise Nested accordion won't work.
-        if (!$this->getClosestOwner($this, AccordionSection::class) && !$this->isDynamicSection()) {
+        // initialize top accordion only, otherwise nested accordion won't work
+        // https://github.com/fomantic/Fomantic-UI/issues/254
+        if ($this->getClosestOwner(AccordionSection::class) === null) {
             $this->js(true)->accordion($this->settings);
         }
 

src/Form/Control.php

@@ -125,8 +125,7 @@ protected function renderTemplateToHtml(string $region = null): string
     {
         $output = parent::renderTemplateToHtml($region);
 
-        /** @var Form|null $form */
-        $form = $this->getClosestOwner($this, Form::class);
+        $form = $this->getClosestOwner(Form::class);
 
         return $form !== null ? $form->fixOwningFormAttrInRenderedHtml($output) : $output;
     }

src/Form/Layout/Section/Accordion.php

@@ -30,7 +30,7 @@ protected function init(): void
             $jsError = [$form->js()->form('add prompt', $fieldName, $str)];
 
             // if a form control is part of an accordion section, it will open that section.
-            $section = $form->getClosestOwner($form->getControl($fieldName), AccordionSection::class);
+            $section = $form->getControl($fieldName)->getClosestOwner(AccordionSection::class);
             if ($section) {
                 $jsError[] = $section->getOwner()->jsOpen($section);
             }

src/View.php

@@ -299,22 +299,25 @@ public function add($object, $region = null): AbstractView
     /**
      * Get objects closest owner which is instance of particular class.
      *
-     * If there are no such owner (or grand-owner etc.) object, then return.
+     * If there are no such owner (or grand-owner etc.) object, then return null.
      *
-     * Note: this is internal method, but should be public because other objects
-     *       should be able to call it.
+     * @template T of View
+     *
+     * @param class-string<T> $class
+     *
+     * @phpstan-return T|null
      */
-    public function getClosestOwner(self $object, string $class): ?self
+    public function getClosestOwner(string $class): ?self
     {
-        if ($object->issetOwner()) {
+        if (!$this->issetOwner()) {
             return null;
         }
 
-        if ($object->getOwner() instanceof $class) {
-            return $object->getOwner();
+        if ($this->getOwner() instanceof $class) {
+            return $this->getOwner();
         }
 
-        return $this->getClosestOwner($object->getOwner(), $class);
+        return $this->getOwner()->getClosestOwner($class);
     }
 
     // }}}