Ticket #44706

Editing: remove workers from tiles edited to unworkable terrain

Date d'ouverture: 2022-05-29 08:27 Dernière mise à jour: 2023-01-09 11:23

Rapporteur:
Propriétaire:
Type:
État:
Atteints
Composant:
Priorité:
5 - moyen
Sévérité:
5 - moyen
Résolution:
Fixed
Fichier:
3

Détails

If you switch to edit mode, make radius 10 of placing Inaccessible terrain and click next to a city, its non-central workers remain where they are while the city just can't work Inaccessible. Did not check border changes by editor, probably they are correct.

Ticket History (3/11 Histories)

2022-05-29 08:27 Updated by: ihnatus
  • New Ticket "Editing: remove workers from tiles edited to unworkable terrain" created
2022-07-19 13:08 Updated by: cazfi
Commentaire

Maybe applies to terrain changes in general, and not just by the editor? In fact, "Tile_Workable" effect can have any kind of requirements, so we should check legality of worker placement regularly.

2022-08-05 08:58 Updated by: cazfi
2022-10-07 09:27 Updated by: cazfi
2022-12-07 02:40 Updated by: cazfi
Commentaire

Reply To cazfi

Maybe applies to terrain changes in general, and not just by the editor? In fact, "Tile_Workable" effect can have any kind of requirements, so we should check legality of worker placement regularly.

Both are likely needed. It makes no sense to wait for TC when dealing with direct tile related requirements.

Let's have this ticket about the tile changes. Update at TC can go to another ticket, and might be linked to https://www.hostedredmine.com/issues/873688 / #46182 that is also about TC effect requirement checking.

2023-01-02 17:04 Updated by: cazfi
Commentaire

Reply To cazfi

Let's have this ticket about the tile changes. Update at TC can go to another ticket

There's three tickets now:
- "Tile_Workable" check at TC: #46423
- Editor mode not checking tile change side effects: #46424
- Tile change side effects not including "Tile_Workable" check: this ticket

2023-01-05 01:09 Updated by: cazfi
  • Propriétaire Update from (Aucun) to cazfi
  • Résolution Update from Aucun to Accepted
2023-01-09 11:23 Updated by: cazfi
  • État Update from Ouvert to Atteints
  • Résolution Update from Accepted to Fixed

Modifier

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Connexion