Bug in representative_action()

[Peter Mueller]

The method representative_action() for Sage permutation groups erroneously returns the identity element when it actually should either fail or return something which makes clear if no  representative exists. The corresponding method for libgap groups behaves as expected:

sage: g = PermutationGroup([[(1, 2)], [(3, 4)]])
sage: g.representative_action(1, 3)
()
sage: libgap(g).RepresentativeAction(1, 3)
fail

[Dima Pasechnik]

Thanks - added to https://github.com/sagemath/sage/issues/35130 -
which traces ills of `g.representative_action()`
>
> --
> You received this message because you are subscribed to the Google Groups "sage-devel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+...@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/sage-devel/6641c397-1536-4ead-9efb-4bf4ca2a1b0fn%40googlegroups.com.