Skip to content

Commit 4c9c3cb

Browse files
ya861220ya861220
committed
fix: Remove the ignore errors of View in phpstan.
1 parent aaf6b34 commit 4c9c3cb

7 files changed

Lines changed: 46 additions & 95 deletions

File tree

‎phpstan-baseline.php‎

Lines changed: 0 additions & 95 deletions
Original file line numberDiff line numberDiff line change
@@ -2191,66 +2191,11 @@
21912191
'count' => 1,
21922192
'path' => __DIR__ . '/system/View/Cell.php',
21932193
];
2194-
$ignoreErrors[] = [
2195-
'message' => '#^Method CodeIgniter\\\\View\\\\Cell\\:\\:getMethodParams\\(\\) has no return type specified\\.$#',
2196-
'count' => 1,
2197-
'path' => __DIR__ . '/system/View/Cell.php',
2198-
];
21992194
$ignoreErrors[] = [
22002195
'message' => '#^Property CodeIgniter\\\\View\\\\Cell\\:\\:\\$cache \\(CodeIgniter\\\\Cache\\\\CacheInterface\\) in empty\\(\\) is not falsy\\.$#',
22012196
'count' => 2,
22022197
'path' => __DIR__ . '/system/View/Cell.php',
22032198
];
2204-
$ignoreErrors[] = [
2205-
'message' => '#^Method CodeIgniter\\\\View\\\\Cells\\\\Cell\\:\\:setView\\(\\) has no return type specified\\.$#',
2206-
'count' => 1,
2207-
'path' => __DIR__ . '/system/View/Cells/Cell.php',
2208-
];
2209-
$ignoreErrors[] = [
2210-
'message' => '#^Method class@anonymous/system/Traits/PropertiesTrait\\.php\\:47\\:\\:getProperties\\(\\) has no return type specified\\.$#',
2211-
'count' => 1,
2212-
'path' => __DIR__ . '/system/View/Cells/Cell.php',
2213-
];
2214-
$ignoreErrors[] = [
2215-
'message' => '#^Method class@anonymous/system/Traits/PropertiesTrait\\.php\\:47\\:\\:getProperties\\(\\) has parameter \\$obj with no type specified\\.$#',
2216-
'count' => 1,
2217-
'path' => __DIR__ . '/system/View/Cells/Cell.php',
2218-
];
2219-
$ignoreErrors[] = [
2220-
'message' => '#^Method CodeIgniter\\\\View\\\\Exceptions\\\\ViewException\\:\\:forInvalidCellClass\\(\\) has no return type specified\\.$#',
2221-
'count' => 1,
2222-
'path' => __DIR__ . '/system/View/Exceptions/ViewException.php',
2223-
];
2224-
$ignoreErrors[] = [
2225-
'message' => '#^Method CodeIgniter\\\\View\\\\Exceptions\\\\ViewException\\:\\:forInvalidCellMethod\\(\\) has no return type specified\\.$#',
2226-
'count' => 1,
2227-
'path' => __DIR__ . '/system/View/Exceptions/ViewException.php',
2228-
];
2229-
$ignoreErrors[] = [
2230-
'message' => '#^Method CodeIgniter\\\\View\\\\Exceptions\\\\ViewException\\:\\:forInvalidCellParameter\\(\\) has no return type specified\\.$#',
2231-
'count' => 1,
2232-
'path' => __DIR__ . '/system/View/Exceptions/ViewException.php',
2233-
];
2234-
$ignoreErrors[] = [
2235-
'message' => '#^Method CodeIgniter\\\\View\\\\Exceptions\\\\ViewException\\:\\:forInvalidDecorator\\(\\) has no return type specified\\.$#',
2236-
'count' => 1,
2237-
'path' => __DIR__ . '/system/View/Exceptions/ViewException.php',
2238-
];
2239-
$ignoreErrors[] = [
2240-
'message' => '#^Method CodeIgniter\\\\View\\\\Exceptions\\\\ViewException\\:\\:forMissingCellParameters\\(\\) has no return type specified\\.$#',
2241-
'count' => 1,
2242-
'path' => __DIR__ . '/system/View/Exceptions/ViewException.php',
2243-
];
2244-
$ignoreErrors[] = [
2245-
'message' => '#^Method CodeIgniter\\\\View\\\\Exceptions\\\\ViewException\\:\\:forNoCellClass\\(\\) has no return type specified\\.$#',
2246-
'count' => 1,
2247-
'path' => __DIR__ . '/system/View/Exceptions/ViewException.php',
2248-
];
2249-
$ignoreErrors[] = [
2250-
'message' => '#^Method CodeIgniter\\\\View\\\\Exceptions\\\\ViewException\\:\\:forTagSyntaxError\\(\\) has no return type specified\\.$#',
2251-
'count' => 1,
2252-
'path' => __DIR__ . '/system/View/Exceptions/ViewException.php',
2253-
];
22542199
$ignoreErrors[] = [
22552200
'message' => '#^Method CodeIgniter\\\\View\\\\Parser\\:\\:addPlugin\\(\\) has parameter \\$callback with no signature specified for callable\\.$#',
22562201
'count' => 1,
@@ -2261,50 +2206,10 @@
22612206
'count' => 1,
22622207
'path' => __DIR__ . '/system/View/Parser.php',
22632208
];
2264-
$ignoreErrors[] = [
2265-
'message' => '#^Method CodeIgniter\\\\View\\\\Table\\:\\:_compileTemplate\\(\\) has no return type specified\\.$#',
2266-
'count' => 1,
2267-
'path' => __DIR__ . '/system/View/Table.php',
2268-
];
2269-
$ignoreErrors[] = [
2270-
'message' => '#^Method CodeIgniter\\\\View\\\\Table\\:\\:_setFromArray\\(\\) has no return type specified\\.$#',
2271-
'count' => 1,
2272-
'path' => __DIR__ . '/system/View/Table.php',
2273-
];
2274-
$ignoreErrors[] = [
2275-
'message' => '#^Method CodeIgniter\\\\View\\\\Table\\:\\:_setFromDBResult\\(\\) has no return type specified\\.$#',
2276-
'count' => 1,
2277-
'path' => __DIR__ . '/system/View/Table.php',
2278-
];
22792209
$ignoreErrors[] = [
22802210
'message' => '#^Property CodeIgniter\\\\View\\\\Table\\:\\:\\$function type has no signature specified for callable\\.$#',
22812211
'count' => 1,
22822212
'path' => __DIR__ . '/system/View/Table.php',
22832213
];
2284-
$ignoreErrors[] = [
2285-
'message' => '#^Method CodeIgniter\\\\View\\\\View\\:\\:endSection\\(\\) has no return type specified\\.$#',
2286-
'count' => 1,
2287-
'path' => __DIR__ . '/system/View/View.php',
2288-
];
2289-
$ignoreErrors[] = [
2290-
'message' => '#^Method CodeIgniter\\\\View\\\\View\\:\\:extend\\(\\) has no return type specified\\.$#',
2291-
'count' => 1,
2292-
'path' => __DIR__ . '/system/View/View.php',
2293-
];
2294-
$ignoreErrors[] = [
2295-
'message' => '#^Method CodeIgniter\\\\View\\\\View\\:\\:logPerformance\\(\\) has no return type specified\\.$#',
2296-
'count' => 1,
2297-
'path' => __DIR__ . '/system/View/View.php',
2298-
];
2299-
$ignoreErrors[] = [
2300-
'message' => '#^Method CodeIgniter\\\\View\\\\View\\:\\:renderSection\\(\\) has no return type specified\\.$#',
2301-
'count' => 1,
2302-
'path' => __DIR__ . '/system/View/View.php',
2303-
];
2304-
$ignoreErrors[] = [
2305-
'message' => '#^Method CodeIgniter\\\\View\\\\View\\:\\:section\\(\\) has no return type specified\\.$#',
2306-
'count' => 1,
2307-
'path' => __DIR__ . '/system/View/View.php',
2308-
];
23092214

23102215
return ['parameters' => ['ignoreErrors' => $ignoreErrors]];

‎system/Traits/PropertiesTrait.php‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,11 @@ final public function fill(array $params): self
4545
final public function getPublicProperties(): array
4646
{
4747
$worker = new class () {
48+
/**
49+
* @param object $obj
50+
*
51+
* @return array
52+
*/
4853
public function getProperties($obj)
4954
{
5055
return get_object_vars($obj);

‎system/View/Cell.php‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -233,6 +233,8 @@ final protected function renderCell(BaseCell $instance, string $method, array $p
233233
* Returns the values from $params that match the parameters
234234
* for a method, in the order they are defined. This allows
235235
* them to be passed directly into the method.
236+
*
237+
* @return array
236238
*/
237239
private function getMethodParams(BaseCell $instance, string $method, array $params)
238240
{

‎system/View/Cells/Cell.php‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,8 @@ public function render(): string
5151

5252
/**
5353
* Sets the view to use when rendered.
54+
*
55+
* @return $this
5456
*/
5557
public function setView(string $view)
5658
{

‎system/View/Exceptions/ViewException.php‎

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,36 +15,57 @@
1515

1616
class ViewException extends FrameworkException
1717
{
18+
/**
19+
* @return ViewException
20+
*/
1821
public static function forInvalidCellMethod(string $class, string $method)
1922
{
2023
return new static(lang('View.invalidCellMethod', ['class' => $class, 'method' => $method]));
2124
}
2225

26+
/**
27+
* @return ViewException
28+
*/
2329
public static function forMissingCellParameters(string $class, string $method)
2430
{
2531
return new static(lang('View.missingCellParameters', ['class' => $class, 'method' => $method]));
2632
}
2733

34+
/**
35+
* @return ViewException
36+
*/
2837
public static function forInvalidCellParameter(string $key)
2938
{
3039
return new static(lang('View.invalidCellParameter', [$key]));
3140
}
3241

42+
/**
43+
* @return ViewException
44+
*/
3345
public static function forNoCellClass()
3446
{
3547
return new static(lang('View.noCellClass'));
3648
}
3749

50+
/**
51+
* @return ViewException
52+
*/
3853
public static function forInvalidCellClass(?string $class = null)
3954
{
4055
return new static(lang('View.invalidCellClass', [$class]));
4156
}
4257

58+
/**
59+
* @return ViewException
60+
*/
4361
public static function forTagSyntaxError(string $output)
4462
{
4563
return new static(lang('View.tagSyntaxError', [$output]));
4664
}
4765

66+
/**
67+
* @return ViewException
68+
*/
4869
public static function forInvalidDecorator(string $className)
4970
{
5071
return new static(lang('View.invalidDecoratorClass', [$className]));

‎system/View/Table.php‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -411,6 +411,8 @@ public function clear()
411411
* Set table data from a database result object
412412
*
413413
* @param BaseResult $object Database result object
414+
*
415+
* @return void
414416
*/
415417
protected function _setFromDBResult($object)
416418
{
@@ -428,6 +430,8 @@ protected function _setFromDBResult($object)
428430
* Set table data from an array
429431
*
430432
* @param array $data
433+
*
434+
* @return void
431435
*/
432436
protected function _setFromArray($data)
433437
{
@@ -442,6 +446,8 @@ protected function _setFromArray($data)
442446

443447
/**
444448
* Compile Template
449+
*
450+
* @return void
445451
*/
446452
protected function _compileTemplate()
447453
{

‎system/View/View.php‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -384,6 +384,8 @@ public function getData(): array
384384

385385
/**
386386
* Specifies that the current view should extend an existing layout.
387+
*
388+
* @return void
387389
*/
388390
public function extend(string $layout)
389391
{
@@ -394,6 +396,8 @@ public function extend(string $layout)
394396
* Starts holds content for a section within the layout.
395397
*
396398
* @param string $name Section name
399+
*
400+
* @return void
397401
*/
398402
public function section(string $name)
399403
{
@@ -407,6 +411,8 @@ public function section(string $name)
407411
/**
408412
* Captures the last section
409413
*
414+
* @return void
415+
*
410416
* @throws RuntimeException
411417
*/
412418
public function endSection()
@@ -429,6 +435,8 @@ public function endSection()
429435

430436
/**
431437
* Renders a section's contents.
438+
*
439+
* @return void
432440
*/
433441
public function renderSection(string $sectionName)
434442
{
@@ -465,6 +473,8 @@ public function getPerformanceData(): array
465473

466474
/**
467475
* Logs performance data for rendering a view.
476+
*
477+
* @return void
468478
*/
469479
protected function logPerformance(float $start, float $end, string $view)
470480
{

0 commit comments

Comments
 (0)