MerginMaps / mobile

Mobile app for Mergin Maps 📲. Open QGIS projects and manage vector data on your phone, even offline 📶 (Android/iOS/Windows app)
https://merginmaps.com
GNU General Public License v2.0
280 stars 62 forks source link

Recursuive apply on default update #3363

Open jozef-budac opened 2 months ago

jozef-budac commented 2 months ago

There is a problem with a scenario, when there is recursive apply on default update on multiple fields. I have one field with "apply on default update" and another one, which the same and is controlled by the previous field.

In the tc05_forms_derived, there is a tab Data. It is hidden till something is filled into Value on the Information tab. On the Data tab, there are 4 fields:

  1. Population (sets population of bears in Slovakia)
  2. Limit (sets limit, till somebody from Slovakia parliament is going to do about it)
  3. Danger is the first field with "apply on default update" sets with value "Population" > "Limit"
  4. Name is the 2nd field, which is controlled by the Danger field and has "apply on default update" set with if("Danger" IS 'false',' Bear safe',' Bear dangerous')

This means, that when Population is > than Limit, the Danger should be true, and Name should be Bear dangerous. This is not happening and because of it, the Slovak parliament doesn't know what to do with it.

In the video, you can see that the update of the Name field is delayed and with the next update has the correct value from the previous data update.

Please note: if you are not familiar with conflicts between bears and politicians in Slovakia, you can freely ignore all the comments on this topic :)

https://github.com/MerginMaps/mobile/assets/83582181/988be761-900d-42f8-b920-0f11e39a2c4f

tomasMizera commented 2 months ago

@jozef-budac can you please try it on production?

tomasMizera commented 2 months ago

Discussed with @jozef-budac and seems to be gone

jozef-budac commented 1 month ago

I can again observe the issue https://github.com/MerginMaps/mobile/assets/83582181/1237d2ed-544d-49d6-8ff7-43622a3c3b96