Move check on specifier up

As we baseline return if link is None, the check against the specifier should be moved up.
pypa · Sep 28, 2023 · 85c1ba5 · 85c1ba5
1 parent 2c498f7
commit 85c1ba5
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/pipenv/environment.py b/pipenv/environment.py
@@ -769,9 +769,13 @@ def is_satisfied(self, req: InstallRequirement):
             None,
         )
         if match is not None:
+            if req.specifier is not None:
+                return SpecifierSet(str(req.specifier)).contains(
+                    match.version, prereleases=True
+                )
             if req.link is None:
                 return True
-            if req.editable and req.link.is_file:
+            elif req.editable and req.link.is_file:
                 requested_path = req.link.file_path
                 if os.path.exists(requested_path):
                     local_path = requested_path
@@ -798,10 +802,6 @@ def is_satisfied(self, req: InstallRequirement):
                 )
             elif req.link.is_vcs:
                 return False
-            elif req.specifier is not None:
-                return SpecifierSet(str(req.specifier)).contains(
-                    match.version, prereleases=True
-                )
             return True
         return False