Completed
Push — develop ( 8eb671...133594 )
by Mike
19:30 queued 09:24
created

tests/features/assets/singlefile/tags/Api.php (2 issues)

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

1
<?php
2
3
/**
4
 * Class A
5
 *
6
 * @api
7
 */
8
class A
0 ignored issues
show
Comprehensibility Best Practice introduced by
The type A has been defined more than once; this definition is ignored, only the first definition in tests/features/assets/si...sicClassInheritance.php (L10-19) is considered.

This check looks for classes that have been defined more than once.

If you can, we would recommend to use standard object-oriented programming techniques. For example, to avoid multiple types, it might make sense to create a common interface, and then multiple, different implementations for that interface.

This also has the side-effect of providing you with better IDE auto-completion, static analysis and also better OPCode caching from PHP.

Loading history...
9
{
10
    /**
11
     * @api
12
     */
13
    public function someMethod()
14
    {
15
    }
16
17
    public function someOtherMethod()
18
    {
19
    }
20
}
21
22
class B extends A
0 ignored issues
show
Comprehensibility Best Practice introduced by
The type B has been defined more than once; this definition is ignored, only the first definition in tests/features/assets/si...sicClassInheritance.php (L21-28) is considered.

This check looks for classes that have been defined more than once.

If you can, we would recommend to use standard object-oriented programming techniques. For example, to avoid multiple types, it might make sense to create a common interface, and then multiple, different implementations for that interface.

This also has the side-effect of providing you with better IDE auto-completion, static analysis and also better OPCode caching from PHP.

Loading history...
23
{
24
}
25