Author: sveinung Date: Wed Jul 27 15:23:42 2016 New Revision: 33337 URL: http://svn.gna.org/viewcvs/freeciv?rev=33337&view=rev Log: Act sel dlg: pop up on long move to target city.
A Caravan that is airlifted to a city where it can establish a trade route should pop up an action selection dialog when the popup_actor_arrival client setting is enabled. Reported by Frank <dunnoob> See bug #24876 Modified: trunk/server/unittools.c Modified: trunk/server/unittools.c URL: http://svn.gna.org/viewcvs/freeciv/trunk/server/unittools.c?rev=33337&r1=33336&r2=33337&view=diff ============================================================================== --- trunk/server/unittools.c (original) +++ trunk/server/unittools.c Wed Jul 27 15:23:42 2016 @@ -3512,6 +3512,20 @@ punit->action_decision_tile = NULL; punit->action_decision_want = ACT_DEC_NOTHING; + if (!adj + && action_tgt_city(punit, pdesttile, FALSE)) { + /* The unit can perform an action to the city at the destination tile. + * A long distance move (like an airlift) doesn't ask what action to + * perform before moving. Ask now. */ + + punit->action_decision_want = ACT_DEC_PASSIVE; + punit->action_decision_tile = pdesttile; + + /* Let the client know that this unit wants the player to decide + * what to do. */ + send_unit_info(player_reply_dest(pplayer), punit); + } + /* Claim ownership of fortress? */ bowner = extra_owner(pdesttile); if ((bowner == NULL || pplayers_at_war(bowner, pplayer)) _______________________________________________ Freeciv-commits mailing list Freeciv-commits@gna.org https://mail.gna.org/listinfo/freeciv-commits