make one(::AbstractMatrix) use similar instead of zeros

[nsajko]

Don't force returning Array, basically.

Also generalized to non-one-indexed arrays.

Fixes #54161

[LilithHafner]

Don't force returning Array, basically.

Yep, sounds good to me

Also generalized to non-one-indexed arrays.

one must return a multiplicative identity, and afaict, non-one-based arrays have no multiplicative identities as of yet in Julia (JuliaArrays/OffsetArrays.jl#91).

Theoretically, I would say OffsetArray([1 0; 0 1], 0, 10) is a right multiplicative identity of OffsetArray([a b; c d], 0, 10), and OffsetArray([1 0; 0 1], 10, 0) is a left multiplicative identity of OffsetArray([a b; c d], 10, 0). But one is documented to return a two-sided multiplicative identity, so I think it is correct to throw on all non one-based-square-matrices.

[LilithHafner]

There are some superfluous stylistic/error-check-order changes that I don't love, but are not regressions; and it would be easier (for me) to review if the moving code and the changing implementation changes were in two separate commits.

That said, LGTM. The PR improves the implementation and the new test fails on master but passes on the PR. Thanks!