Bug in owner search (2 account looked for while only 1 is allowed)

[MaelREBOUX]

@tdipisa we think we have found a bug that have passed our tests.

Here the steps to reproduce on our test portal :

1. open cadastrapp
2. clic sur owner search
3. search "chauvin frederic"

Here we have the choice to select a suggestion in the list OR to echap the suggestion to show a special window which lists the owners.

The behaviour of the search by the special window is OK.
Choosing "M CHAUVIN FREDERIC" -> 3 parcels
Choosing "M CHAUVIN THOMAS FREDERIC MICHEL" -> 1 parcel

But then : the UI let us choose "M CHAUVIN FREDERIC" directly and if we valid the search we are getting 4 parcels.
So the user believes that this ower has 4 parcels while he owns only 3.

In mapfishapp it cannot happened : if more than 1 owner account is foud we systemacally open the special owner list window and we are obliged to choose one account.

We noted that the mapstore plugin is building a query to /cadastrapp/services/getParcelle with 2 owner account while the user choose only one row in the dropdown list. So here is the origin of 4 parcels while 3 is awaited.

Here the current behaviour in mapfishapp

And the problem in mapstore

[offtherailz]

Ok, I never find out this use case. I hoped that that step could be skipped if I select a user, but I see that there is still ambiguity.
So I created a PR that should solve this issue. Now, even if I select an existing record, if the search returns more than one owner, it will ask for owner selection.

Please check https://github.com/georchestra/mapstore2-cadastrapp/releases/tag/1.0.0-rc13

[tdipisa]

@catmorales, it should be fixed, let us know if we can close the issue.

[catmorales]

I deployed the tag 1.0.0-rc12 this morning in production. it seems not fixed on this tag.
I dont't have time to test it now .

[tdipisa]

@offtherailz, any feedback?

[offtherailz]

#123 is fixed by rc13 . Please try the rc13.
See this:
https://github.com/georchestra/mapstore2-cadastrapp/releases

[catmorales]

Ok i'll try later. thank you @offtherailz

[MaelREBOUX]

Hello. Sorry to be late.
Away on monday and now I'm busy by the geOcom since tuesday. I test tomorrow friday. Sure.

[MaelREBOUX]

Hello.

OK : the bogue is fixed. It is sure. Congrats.

[tdipisa]

@MaelREBOUX can you please open a new issue for the problem you described above? We will close this one if the related bug is effectively solved.

[MaelREBOUX]

ok. I thinked about it.

[MaelREBOUX]

=> #130