Merge pull request #7663 from kenjis/fix-view-cell-caching

fix: [ViewCells] caching

Author: kenjis

system/View/Cell.php

@@ -64,7 +64,10 @@ public function __construct(CacheInterface $cache)
     /**
      * Render a cell, returning its body as a string.
      *
-     * @param array|string|null $params
+     * @param string            $library   Cell class and method name.
+     * @param array|string|null $params    Parameters to pass to the method.
+     * @param int               $ttl       Number of seconds to cache the cell.
+     * @param string|null       $cacheName Cache item name.
      *
      * @throws ReflectionException
      */
@@ -76,6 +79,8 @@ public function render(string $library, $params = null, int $ttl = 0, ?string $c
             ? get_class($instance)
             : null;
 
+        $params = $this->prepareParams($params);
+
         // Is the output cached?
         $cacheName = ! empty($cacheName)
             ? $cacheName
@@ -93,8 +98,6 @@ public function render(string $library, $params = null, int $ttl = 0, ?string $c
             throw ViewException::forInvalidCellMethod($class, $method);
         }
 
-        $params = $this->prepareParams($params);
-
         $output = $instance instanceof BaseCell
             ? $this->renderCell($instance, $method, $params)
             : $this->renderSimpleClass($instance, $method, $params, $class);