Merge pull request #7276 from weirdan/drop-deprecated-codeissue-methods

vimeo · Jan 3, 2022 · af7501f · af7501f
2 parents bcff8d9 + 0866866
commit af7501f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 27 deletions.
diff --git a/UPGRADING.md b/UPGRADING.md
@@ -152,3 +152,6 @@
    -  `Psalm\Plugin\Hook\AfterClassLikeAnalysisInterface`
    -  `Psalm\Plugin\Hook\PropertyVisibilityProviderInterface`
    -  `Psalm\Plugin\Hook\AfterStatementAnalysisInterface`
+ - [BC] Method `Psalm\Issue\CodeIssue::getLocation()` was removed
+ - [BC] Method `Psalm\Issue\CodeIssue::getFileName()` was removed
+ - [BC] Method `Psalm\Issue\CodeIssue::getMessage()` was removed
diff --git a/src/Psalm/Issue/CodeIssue.php b/src/Psalm/Issue/CodeIssue.php
@@ -38,15 +38,6 @@ public function __construct(
         $this->message = $message;
     }
 
-    /**
-     * @deprecated going to be removed in Psalm 5
-     * @psalm-suppress PossiblyUnusedMethod
-     */
-    public function getLocation(): CodeLocation
-    {
-        return $this->code_location;
-    }
-
     public function getShortLocationWithPrevious(): string
     {
         $previous_text = '';
@@ -69,24 +60,6 @@ public function getFilePath(): string
         return $this->code_location->file_path;
     }
 
-    /**
-     * @deprecated going to be removed in Psalm 5
-     * @psalm-suppress PossiblyUnusedMethod for convenience
-     */
-    public function getFileName(): string
-    {
-        return $this->code_location->file_name;
-    }
-
-    /**
-     * @deprecated going to be removed in Psalm 5
-     * @psalm-suppress PossiblyUnusedMethod
-     */
-    public function getMessage(): string
-    {
-        return $this->message;
-    }
-
     public static function getIssueType(): string
     {
         $fqcn_parts = explode('\\', static::class);