Strange behavior for Permutation()

[sagetrac-lvendramin]

See these examples:

sage: Permutation([1,2,3])
[1, 2, 3]
sage: Permutation([1,2,3,1])
[1, 2, 3, 1]
sage: [1,2,3] in Permutations()
True
sage: [1,2,3,1] in Permutations()
False
sage: Permutation([1,2,3,1]) in Permutations()
True

CC: @jbandlow

Component: combinatorics

Reviewer: Luis Felipe Tabera Alonso

Issue created by migration from https://trac.sagemath.org/ticket/8918

[lftabera]

comment:1

Yes, this is an error

[1,2,3,1] dos not define a permutation, so the method should raise an error. See also #9831

[KPanComputes]

comment:2

I think, this in fact can be closed as a duplicate of the ticket you mention (#9831).

Replying to @lftabera:

Yes, this is an error

[1,2,3,1] dos not define a permutation, so the method should raise an error. See also #9831

[tscrim]

comment:4

One of a few tickets like this. Bonus points if you can find them all.

Also, don't forget to set this to needs_review. Thanks.

[jdemeyer]

Reviewer: Luis Felipe Tabera Alonso

[jdemeyer]

Changed author from lvendramin to none