# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 1524135351 -32400
# Node ID 43221a57e22fd6d1f3d86576c057a331386fd2b9
# Parent  968ac00c4017a9316f7d7381133de4d8a01c48d3
context: translate FilteredIndex/LookupError at repo[changeid] (API)

This partially backs out ecd3f6909184. It seems layering violation for
repo[changeid] to raise storage-level exceptions transparently. Otherwise,
we would have to rewrite callers to catch all of them.

  try:
      repo[rev_or_node]
  except (error.RepoLookupError, error.FilteredIndexError,
          error.FilteredLookupError):
      pass

This would also fix filectx._changectx(), which catches FilteredRepoLookupError
to fall back to the unfiltered path.

diff -r 968ac00c4017 -r 43221a57e22f mercurial/context.py
--- a/mercurial/context.py	Thu Apr 19 20:22:33 2018 +0900
+++ b/mercurial/context.py	Thu Apr 19 19:55:51 2018 +0900
@@ -497,8 +497,10 @@
                     changeid = hex(changeid)
             except TypeError:
                 pass
-        except (error.FilteredIndexError, error.FilteredLookupError,
-                error.FilteredRepoLookupError):
+        except (error.FilteredIndexError, error.FilteredLookupError):
+            raise error.FilteredRepoLookupError(_("filtered revision '%s'")
+                                                % changeid)
+        except error.FilteredRepoLookupError:
             raise
         except IndexError:
             pass
diff -r 968ac00c4017 -r 43221a57e22f mercurial/localrepo.py
--- a/mercurial/localrepo.py	Thu Apr 19 20:22:33 2018 +0900
+++ b/mercurial/localrepo.py	Thu Apr 19 19:55:51 2018 +0900
@@ -850,8 +850,7 @@
         try:
             self[changeid]
             return True
-        except (error.RepoLookupError, error.FilteredIndexError,
-                error.FilteredLookupError):
+        except error.RepoLookupError:
             return False
 
     def __nonzero__(self):