do_paradrop(): trust the action system

• Informador: kvilhaugsvik
• Propietario: kvilhaugsvik
• Tipo: Patches
• Estado: Cerrado
• Componente: Server
• Hito: 3.1.0 (cerrado)
• Prioridad: 5 - Medium
• Gravedad: 5 - Medium
• Resolución: Accepted
• Fichero: 2

Details Responder

Trust that the action sent is correct when determining if a city conquest is legal or not in the action performer of the paradrop actions.

Ticket History (3/9 Histories)

2021-03-22 22:44 Updated by: kvilhaugsvik

• New Ticket "do_paradrop(): trust the action system" created

2021-03-22 23:00 Updated by: kvilhaugsvik

• File 0001-do_paradrop-trust-the-action-system.patch (File ID: 6306) is attached

2021-03-22 23:00 Updated by: kvilhaugsvik

• Resolución Update from Ninguno to Accepted

2021-03-22 23:05 Updated by: cazfi

Comentario Responder

- && pplayers_non_attack(pplayer, plrtile->owner)) { + && !pplayers_allied(pplayer, plrtile->owner) + && !action_has_result(paction, ACTRES_PARADROP_CONQUER)) {

notify_player(pplayer, ptile, E_BAD_COMMAND, ftc_server,

_("Cannot attack unless you declare war first."));

Is that message still accurate?

2021-03-22 23:25 Updated by: kvilhaugsvik

Comentario Responder

Reply To cazfi

_("Cannot attack unless you declare war first.")); Is that message still accurate?

Yes, because war is an obligatory hard action requirement.

2021-03-22 23:26 Updated by: kvilhaugsvik

Comentario Responder

Wait! Wrong.

2021-03-22 23:56 Updated by: kvilhaugsvik

• File 0001-do_paradrop-trust-the-action-system.patch (File ID: 6311) is attached

2021-03-22 23:56 Updated by: kvilhaugsvik

Comentario Responder

v2: catch both conquer and non conquer issues

(Edited, 2021-03-23 00:02 Updated by: kvilhaugsvik)

2021-03-24 19:40 Updated by: kvilhaugsvik

• Estado Update from Open to Cerrado