7
93%
master: 93%

Ran 02 Nov 2019 05:05PM UTC

Files 395

Run time 22s

Badge

Embed ▾

Committed 02 Nov 2019 04:41PM UTC coverage: 92.151% (+0.04%) from 92.109%

Job # COLLECT_COVERAGE=1

Build Type

push

travis-ci

Committed by

keradus

Commit Message

feature #3928 Add FinalPublicMethodForAbstractClassFixer (Slamdunk)

This PR was merged into the 2.16-dev branch.

Discussion
----------

Add FinalPublicMethodForAbstractClassFixer

```diff
 abstract class AbstractMachine
 {
-    public function start()
+    final public function start()
     {}
 }
```

#### No exception and no configuration are intentional

If you want to override a method, use the [Template method pattern](https://en.wikipedia.org/wiki/Template_method_pattern).

This fixer enforces API encapsulation in an inheritance environment.

I left out magic methods:

1. Because some of them are not API
1. And the rest have issues with SPL releated array classes etc

Further reading: [When to declare methods final](https://slamdunk.github.io/blog/when-to-declare-methods-final/)

Commits
-------

19f823c6 Add FinalPublicMethodForAbstractClassFixer

Coverage Stats

15415 of 16728 relevant lines covered (92.15%)

61.95 hits per line

FriendsOfPHP / PHP-CS-Fixer / 19178 / 7
93%
master: 93%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 19178.7 (COLLECT_COVERAGE=1)

FriendsOfPHP / PHP-CS-Fixer / 19178 / 7 93% master: 93%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 19178.7 (COLLECT_COVERAGE=1)

FriendsOfPHP / PHP-CS-Fixer / 19178 / 7
93%
master: 93%

README BADGES
x