FIX: Action, permuting lists, tests for perms

Use list hashing for lists of integers, not lists of vectors. This fixes #5786
gap-system · Sep 18, 2024 · b6d69e8 · b6d69e8
1 parent 2b482f7
commit b6d69e8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/oprt.gi b/lib/oprt.gi
@@ -3609,6 +3609,7 @@ InstallMethod(DomainForAction,"permutations on lists of integers",true,
 function(pnt,acts,act)
   local m;
   if not (Length(pnt)>0 and ForAll(pnt,IsPosInt) and
+    ForAll(acts,IsPerm) and
     (act=OnSets or act=OnPoints or act=OnRight or act=\^)) then
     TryNextMethod();
   fi;

diff --git a/lib/vecmat.gi b/lib/vecmat.gi
@@ -2142,7 +2142,8 @@ InstallMethod(DomainForAction,"matrix/matrix",IsElmsCollsX,
 function(pnt,acts,act)
 local l,f;
   if (not ForAll(acts,IsMatrix)) or
-    (act<>OnPoints and act<>OnSubspacesByCanonicalBasis and act<>OnRight) then
+    (act<>OnPoints and act<>OnSubspacesByCanonicalBasis and act<>OnRight and act<>OnSets and
+    act<>OnTuples) then
     TryNextMethod(); # strange operation, might extend the domain
   fi;
   l:=NaturalActedSpace(acts,pnt);