1 | <?php |
||
7 | class WidgetRenderer |
||
8 | { |
||
9 | public $html; |
||
10 | |||
11 | private $_viewData; |
||
12 | |||
13 | private $_policies; |
||
14 | |||
15 | /** |
||
16 | * BaseWidget constructor. |
||
17 | */ |
||
18 | public function __construct() |
||
22 | |||
23 | /** |
||
24 | * @param $widget object|string |
||
25 | * @param array $args |
||
26 | * @return string |
||
27 | */ |
||
28 | public function renderWidget($widget, ...$args) |
||
29 | { |
||
30 | if (is_string($widget)) { |
||
31 | $widget = $this->makeWidgetObj($widget); |
||
32 | } |
||
33 | |||
34 | app(Normalizer::class)->normalizeWidgetConfig($widget); |
||
35 | |||
36 | try { |
||
37 | $html = $this->generateHtml($widget, ...$args); |
||
38 | } catch (\Exception $e) { |
||
39 | return app()->make(ExceptionHandler::class)->render(app('request'), $e)->send(); |
||
40 | } |
||
41 | |||
42 | return $html; |
||
43 | } |
||
44 | |||
45 | /** |
||
46 | * @param $widget object |
||
47 | * @return \Illuminate\Foundation\Application|mixed |
||
48 | */ |
||
49 | private function makeWidgetObj($widget) |
||
50 | { |
||
51 | if (starts_with($widget, ['\\'])) { |
||
52 | return app($widget); |
||
53 | } |
||
54 | |||
55 | $widget = app()->getNamespace().'Widgets\\'.$widget; |
||
56 | |||
57 | return app($widget); |
||
58 | } |
||
59 | |||
60 | /** |
||
61 | * It tries to get the html from cache if possible, otherwise generates it. |
||
62 | * |
||
63 | * @param $widget object |
||
64 | * @param array ...$args |
||
65 | * |
||
66 | * @return string |
||
67 | */ |
||
68 | private function generateHtml($widget, ...$args) |
||
69 | { |
||
70 | // Everything inside this function is executed only when the cache is not available. |
||
71 | $expensivePhpCode = function () use ($widget, $args) { |
||
72 | $this->makeDataForView($widget, $args); |
||
73 | |||
74 | // render the template with the resulting data. |
||
75 | return $this->renderTemplate($widget); |
||
76 | }; |
||
77 | |||
78 | // We first try to get the output from the cache before trying to run the expensive $expensivePhpCode... |
||
79 | if ($this->_policies->widgetShouldUseCache($widget->cacheLifeTime)) { |
||
80 | return app(Cache::class)->cacheResult($args, $expensivePhpCode, $widget); |
||
81 | } |
||
82 | |||
83 | return $expensivePhpCode(); |
||
84 | } |
||
85 | |||
86 | /** |
||
87 | * @param $widget object |
||
88 | * @param $args array |
||
89 | * |
||
90 | * @return null |
||
91 | */ |
||
92 | private function makeDataForView($widget, array $args) |
||
105 | |||
106 | /** |
||
107 | * @param $widget object |
||
108 | * @return string HTML output |
||
109 | */ |
||
110 | private function renderTemplate($widget) |
||
128 | } |
||
129 |
It seems like the method you are trying to call exists only in some of the possible types.
Let’s take a look at an example:
Available Fixes
Add an additional type-check:
Only allow a single type to be passed if the variable comes from a parameter: