[#26] Display method visibility in compare report

Author: tomzx

src/PHPSemVerChecker/Operation/ClassMethodOperation.php

@@ -17,33 +17,21 @@ abstract class ClassMethodOperation extends Operation {
 
 	public function getCode()
 	{
-		$visiblityMapping = $this->getVisibilityMapping();
-		return $this->code[$this->context][$visiblityMapping[$this->visibility]];
+		return $this->code[$this->context][Visibility::get($this->visibility)];
 	}
 
 	public function getLevel()
 	{
-		$visiblityMapping = $this->getVisibilityMapping();
-		return $this->level[$this->context][$visiblityMapping[$this->visibility]];
+		return $this->level[$this->context][Visibility::get($this->visibility)];
 	}
 
-	protected function getVisibilityMapping()
+	public function getReason()
 	{
-		return [
-			Class_::MODIFIER_PUBLIC    => 0,
-			Class_::MODIFIER_PROTECTED => 1,
-			Class_::MODIFIER_PRIVATE   => 2,
-		];
+		return '[' . Visibility::toString($this->visibility) . '] ' . $this->reason;
 	}
 
-	protected function getVisibility(ClassMethod $classMethod)
+	protected function getVisibility($context)
 	{
-		if ($classMethod->isPublic()) {
-			return Class_::MODIFIER_PUBLIC;
-		} elseif ($classMethod->isProtected()) {
-			return Class_::MODIFIER_PROTECTED;
-		} else {
-			return Class_::MODIFIER_PRIVATE;
-		}
+		return Visibility::getForContext($context);
 	}
 }

src/PHPSemVerChecker/Operation/Visibility.php

@@ -0,0 +1,57 @@
+<?php
+
+namespace PHPSemVerChecker\Operation;
+
+use PhpParser\Node\Stmt;
+use PhpParser\Node\Stmt\Class_;
+
+class Visibility
+{
+	/**
+	 * @return array
+	 */
+	public static function getMapping()
+	{
+		return [
+			Class_::MODIFIER_PUBLIC    => 0,
+			Class_::MODIFIER_PROTECTED => 1,
+			Class_::MODIFIER_PRIVATE   => 2,
+		];
+	}
+
+	/**
+	 * @param $visibility
+	 * @return int
+	 */
+	public static function get($visibility)
+	{
+		$mapping = self::getMapping();
+		return $mapping[$visibility];
+	}
+
+	/**
+	 * @param \PhpParser\Node\Stmt $context
+	 * @return int
+	 */
+	public static function getForContext(Stmt $context)
+	{
+		if ($context->isPublic()) {
+			return Class_::MODIFIER_PUBLIC;
+		} elseif ($context->isProtected()) {
+			return Class_::MODIFIER_PROTECTED;
+		} else {
+			return Class_::MODIFIER_PRIVATE;
+		}
+	}
+
+	public static function toString($visibility)
+	{
+		if ($visibility === Class_::MODIFIER_PUBLIC) {
+			return 'public';
+		} elseif ($visibility === Class_::MODIFIER_PROTECTED) {
+			return 'protected';
+		} else {
+			return 'private';
+		}
+	}
+}