Remove name and crap attributes from <line> element

Author: sebastianbergmann

src/Report/Clover.php

@@ -108,10 +108,8 @@ public function process(CodeCoverage $coverage, ?string $target = null, ?string
                     $lines[$method['startLine']] = [
                         'ccn'        => $method['ccn'],
                         'count'      => $methodCount,
-                        'crap'       => $method['crap'],
                         'type'       => 'method',
                         'visibility' => $method['visibility'],
-                        'name'       => $methodName,
                     ];
                 }
 
@@ -150,10 +148,6 @@ public function process(CodeCoverage $coverage, ?string $target = null, ?string
                 $xmlLine->setAttribute('num', (string) $line);
                 $xmlLine->setAttribute('type', $data['type']);
 
-                if (isset($data['name'])) {
-                    $xmlLine->setAttribute('name', $data['name']);
-                }
-
                 if (isset($data['visibility'])) {
                     $xmlLine->setAttribute('visibility', $data['visibility']);
                 }
@@ -162,10 +156,6 @@ public function process(CodeCoverage $coverage, ?string $target = null, ?string
                     $xmlLine->setAttribute('complexity', (string) $data['ccn']);
                 }
 
-                if (isset($data['crap'])) {
-                    $xmlLine->setAttribute('crap', (string) $data['crap']);
-                }
-
                 $xmlLine->setAttribute('count', (string) $data['count']);
                 $xmlFile->appendChild($xmlLine);
             }

tests/_files/BankAccount-clover-line.xml

@@ -9,16 +9,16 @@
         <class name="BankAccount">
           <metrics complexity="5" elements="12" coveredelements="8" conditionals="0" coveredconditionals="0" statements="8" coveredstatements="5" methods="4" coveredmethods="3"/>
         </class>
-        <line num="6" type="method" name="getBalance" visibility="public" complexity="1" crap="1" count="2"/>
+        <line num="6" type="method" visibility="public" complexity="1" count="2"/>
         <line num="8" type="stmt" count="2"/>
-        <line num="11" type="method" name="setBalance" visibility="protected" complexity="2" crap="6" count="0"/>
+        <line num="11" type="method" visibility="protected" complexity="2" count="0"/>
         <line num="13" type="stmt" count="0"/>
         <line num="14" type="stmt" count="0"/>
         <line num="16" type="stmt" count="0"/>
-        <line num="20" type="method" name="depositMoney" visibility="public" complexity="1" crap="1" count="2"/>
+        <line num="20" type="method" visibility="public" complexity="1" count="2"/>
         <line num="22" type="stmt" count="2"/>
         <line num="24" type="stmt" count="1"/>
-        <line num="27" type="method" name="withdrawMoney" visibility="public" complexity="1" crap="1" count="2"/>
+        <line num="27" type="method" visibility="public" complexity="1" count="2"/>
         <line num="29" type="stmt" count="2"/>
         <line num="31" type="stmt" count="1"/>
       </file>

tests/_files/BankAccount-clover-path.xml

@@ -9,16 +9,16 @@
         <class name="BankAccount">
           <metrics complexity="5" elements="19" coveredelements="11" conditionals="7" coveredconditionals="3" statements="8" coveredstatements="5" methods="4" coveredmethods="3"/>
         </class>
-        <line num="6" type="method" name="getBalance" visibility="public" complexity="1" crap="1" count="2"/>
+        <line num="6" type="method" visibility="public" complexity="1" count="2"/>
         <line num="8" type="stmt" count="2"/>
-        <line num="11" type="method" name="setBalance" visibility="protected" complexity="2" crap="6" count="0"/>
+        <line num="11" type="method" visibility="protected" complexity="2" count="0"/>
         <line num="13" type="stmt" count="0"/>
         <line num="14" type="stmt" count="0"/>
         <line num="16" type="stmt" count="0"/>
-        <line num="20" type="method" name="depositMoney" visibility="public" complexity="1" crap="1" count="2"/>
+        <line num="20" type="method" visibility="public" complexity="1" count="2"/>
         <line num="22" type="stmt" count="2"/>
         <line num="24" type="stmt" count="1"/>
-        <line num="27" type="method" name="withdrawMoney" visibility="public" complexity="1" crap="1" count="2"/>
+        <line num="27" type="method" visibility="public" complexity="1" count="2"/>
         <line num="29" type="stmt" count="2"/>
         <line num="31" type="stmt" count="1"/>
       </file>

tests/_files/class-with-anonymous-function-clover.xml

@@ -9,7 +9,7 @@
         <class name="CoveredClassWithAnonymousFunctionInStaticMethod">
           <metrics complexity="1" elements="9" coveredelements="9" conditionals="0" coveredconditionals="0" statements="8" coveredstatements="8" methods="1" coveredmethods="1"/>
         </class>
-        <line num="5" type="method" name="runAnonymous" visibility="public" complexity="1" crap="1" count="1"/>
+        <line num="5" type="method" visibility="public" complexity="1" count="1"/>
         <line num="7" type="stmt" count="1"/>
         <line num="9" type="stmt" count="1"/>
         <line num="10" type="stmt" count="1"/>