/convloc ターゲットセレクターの座標もすり替えられる

[kohonayoshi]

不具合の説明

実行時、ターゲットセレクターののx,y,z,dx,dy,dzの内容まですり替えられるので、コマンドが機能しなくなります。

実行前 例
/tp @a[x=10,y=20,z=30,distance=..5] 40 50 60
実行後 例
/tp @a[x=~2004.0,y=~-38.0,z=~-1566.0,distance=..5] ~-18.0 ~-1546.0 60
※コマブロの場所は -1994, 58, 1596

再現手順

1. 権限グループ「Verified」のプレイヤー「X4Z」がコマンド「/aaa bbb」を実行する
2. エラー「Hoge」が表示される

スクリーンショット

No response

追加情報

この例そもそも最後の60変わってないし失敗してるような気がする #858

[book000]

x, y, z を変えるのは正常動作です（少なくとも上記例ではセレクターに関してはうまくいってるように見えます）
tp先の処理がうまくいってないのはよくわかりません、しらべます

[kohonayoshi]

どんなアルゴリズムか見てないからわからんけど
@[ が出てきたら ] が出てくるまで処理をスキップとかそういうアレアレとかにしたほうが良いかもね

[book000]

いえ。ターゲットセレクター内のx,y,zも絶対座標では移動後困るので意図的に変更しています

[book000]

/tp @a[x=10,y=20,z=30,distance=..5] 40 50 60 -> /tp @a[x=~2004.0,y=~-38.0,z=~-1566.0,distance=..5] ~-18.0 ~-1546.0 60 については、tp先の座標については確かに問題がありますがターゲットセレクターについては問題ないはずです

[kohonayoshi]

ターゲットセレクターは絶対座標オンリーです！！！！！！！！！！！

[book000]

じゃあ移動先でconvlocしなおしてもらうべきですねえ

[book000]

というか以前実装時はターゲットセレクターの相対いけたはずなのですけど変わったんでしょうか

[kohonayoshi]

いいえもとからダメです

[book000]

じゃあわかりません

[book000]

とりあえず、一つのissueに二つの用件を混ぜられても困るので分割しますね

[book000]

メモ: ターゲットセレクターの置き換え処理を削除する。移動先でのターゲットセレクターは手動変更？めんどくさ

[kohonayoshi]

昔から手動変更だよ

[book000]

少なくとも、convlocは「PvPコマンドブロックなどを移動する際に使う」というイメージがあるので、移動先で再置換したらうまく動くようになったほうがまだよい気がしますよね
ターゲットセレクターも置き換えるフラグを作ったほうがよさそうです