# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@octobus.net>
# Date 1677021887 -3600
# Node ID ce657d7b7c394ad43f1e11898446b7a0c9f7d390
# Parent  270dc01481aff5ef5e4749648566e7f83b64a9a5
large-files: open the transaction sooner in `scmutiladdremove`

We want it to encompass the status call.

diff -r 270dc01481af -r ce657d7b7c39 hgext/largefiles/overrides.py
--- a/hgext/largefiles/overrides.py	Wed Feb 22 00:22:16 2023 +0100
+++ b/hgext/largefiles/overrides.py	Wed Feb 22 00:24:47 2023 +0100
@@ -1552,6 +1552,11 @@
         opts = {}
     if not lfutil.islfilesrepo(repo):
         return orig(repo, matcher, prefix, uipathfn, opts, open_tr=open_tr)
+
+    # open the transaction and changing_files context
+    if open_tr is not None:
+        open_tr()
+
     # Get the list of missing largefiles so we can remove them
     lfdirstate = lfutil.openlfdirstate(repo.ui, repo)
     unsure, s, mtime_boundary = lfdirstate.status(
@@ -1562,10 +1567,6 @@
         unknown=False,
     )
 
-    # open the transaction and changing_files context
-    if open_tr is not None:
-        open_tr()
-
     # Call into the normal remove code, but the removing of the standin, we want
     # to have handled by original addremove.  Monkey patching here makes sure
     # we don't remove the standin in the largefiles code, preventing a very