pallet_asset_tx_payment can't support xcm::v4::Location as an AssetId

[JuaniRios]

Is there an existing issue?

• I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

• This is not a support question.

Description of bug

The pallet has the asset id as an associated type of the OnChargeAssetTransaction trait, but it expects it to implement Copy.
v3::Multilocation used to implement Copy so this was fine, but now v4::Location doesn't implement copy.

This means it's impossible to directly have Location as the AssetId, so we have to accept v3, and do some unnecessary internal conversions to v4 inside the withdraw_fee method

[franciscoaguirre]

A solution is to make pallet_asset_tx_payment require Clone instead of Copy, which is more lenient