# HG changeset patch
# User Anton Shestakov <av6@dwimlabs.net>
# Date 1691953706 10800
# Node ID 7b2bd0332b56b588cc07d1d5d4f7f9818c20ac08
# Parent  44bc900ba5dbc5c4fc97b5296122c0c89d6ce219
topic: transaction names sometimes need stripping of URLs

Transaction names during push or pull usually contain URL of the remote repo.
Specifying e.g. tns-report-transactions=pull,push wouldn't work as expected
unless we strip URLs from transaction names, so now we do it for this check.

diff -r 44bc900ba5db -r 7b2bd0332b56 hgext3rd/topic/__init__.py
--- a/hgext3rd/topic/__init__.py	Thu Aug 03 16:32:41 2023 -0300
+++ b/hgext3rd/topic/__init__.py	Sun Aug 13 16:08:26 2023 -0300
@@ -494,9 +494,11 @@
 
 def report_affected_tns(repo, tr):
     report = set(repo.ui.configlist(b'devel', b'tns-report-transactions'))
+    # transaction names sometimes also have a URL after a newline byte
+    trnames = (trname.partition(b'\n')[0] for trname in tr._names)
     if b'*' not in report:
         # * matches any transaction
-        if not any(trname in report for trname in tr._names):
+        if not any(trname in report for trname in trnames):
             return
 
     if tr.changes[b'tns']: