search

software-challenge / backend

Server, Client und Spiel-Plugins der Software-Challenge Germany
https://www.software-challenge.de
11 stars 10 forks source link

Not wanted tags __shape and __coordinates in piece XML #351

Closed SKoschnicke closed 3 years ago

SKoschnicke commented 3 years ago

Since version 21.2.1 there are tags __shape and __coordinates in the piece XML (lastMove).

These shouldn't be there.

xeruf commented 3 years ago

Seems fixed

SKoschnicke commented 3 years ago

They seem to be still present in current master (8d2c03bf):

```xml ONE MONO DOMINO TRIO_L TRIO_I TETRO_O TETRO_T TETRO_I TETRO_L TETRO_Z PENTO_L PENTO_T PENTO_V PENTO_S PENTO_Z PENTO_I PENTO_P PENTO_W PENTO_U PENTO_R PENTO_X PENTO_Y MONO DOMINO TRIO_L TRIO_I TETRO_O TETRO_T TETRO_I TETRO_L TETRO_Z PENTO_L PENTO_T PENTO_V PENTO_S PENTO_Z PENTO_I PENTO_P PENTO_W PENTO_U PENTO_R PENTO_X PENTO_Y MONO DOMINO TRIO_L TRIO_I TETRO_O TETRO_T TETRO_I TETRO_L TETRO_Z PENTO_L PENTO_T PENTO_V PENTO_S PENTO_Z PENTO_I PENTO_P PENTO_W PENTO_U PENTO_R PENTO_X PENTO_Y MONO DOMINO TRIO_L TRIO_I TETRO_O TETRO_T TETRO_I TETRO_L TETRO_Z PENTO_L PENTO_T PENTO_V PENTO_S PENTO_Z PENTO_I PENTO_P PENTO_W PENTO_U PENTO_R PENTO_X PENTO_Y BLUE YELLOW RED GREEN ONE TWO <__shape class="linked-hash-set"> <__coordinates class="linked-hash-set"> ONE MONO DOMINO TRIO_L TRIO_I TETRO_O TETRO_T TETRO_I TETRO_L TETRO_Z PENTO_L PENTO_T PENTO_V PENTO_S PENTO_Z PENTO_I PENTO_W PENTO_U PENTO_R PENTO_X PENTO_Y MONO DOMINO TRIO_L TRIO_I TETRO_O TETRO_T TETRO_I TETRO_L TETRO_Z PENTO_L PENTO_T PENTO_V PENTO_S PENTO_Z PENTO_I PENTO_P PENTO_W PENTO_U PENTO_R PENTO_X PENTO_Y MONO DOMINO TRIO_L TRIO_I TETRO_O TETRO_T TETRO_I TETRO_L TETRO_Z PENTO_L PENTO_T PENTO_V PENTO_S PENTO_Z PENTO_I PENTO_P PENTO_W PENTO_U PENTO_R PENTO_X PENTO_Y MONO DOMINO TRIO_L TRIO_I TETRO_O TETRO_T TETRO_I TETRO_L TETRO_Z PENTO_L PENTO_T PENTO_V PENTO_S PENTO_Z PENTO_I PENTO_P PENTO_W PENTO_U PENTO_R PENTO_X PENTO_Y BLUE YELLOW RED GREEN ONE TWO <__shape class="linked-hash-set"> <__coordinates class="linked-hash-set"> ```
xeruf commented 3 years ago

Interesting, I tested a serialisation of lastMove in https://github.com/CAU-Kiel-Tech-Inf/backend/pull/374/files#diff-5e18167ae19f8b1b6b2d539af6baebf5522cacc68c18402830d2f6c90b7b9257R176-R180 and it worked - but that may be related to not using some lazy fields.

xeruf commented 3 years ago

Fixed in 23589a153e8cd3c5b1b3257ff35f66ebbb8d3012