Freeciv - Bugs: bug #20454, Emergency-loaded enemy transporter... [Gna!]

Freeciv - Bugs: bug #20454, Emergency-loaded enemy transporter...

Group

Main
- Main
- View Members
- Search
Homepage
Download
Docs
- Browse
- Submit
- Edit
- Digest
- Export
- Search
Mailing Lists
Source Code
- Use Subversion
- Browse Sources Repository
Bugs
- Browse
- Reset to all open ones
- Submit
- Digest
- Export
- View Statistics
- Search
Tasks
- Browse
- Reset to all open ones
- Submit
- Digest
- Export
- View Statistics
- Search
Patches
- Browse
- Reset to all open ones
- Submit
- Digest
- Export
- View Statistics
- Search

Show feedback again

bug #20454: Emergency-loaded enemy transporter status not updated

Submitted by:	Marko Lindqvist <cazfi>
Submitted on:	Fri 25 Jan 2013 12:38:14 PM UTC

Category:	general	Severity:	3 - Normal
Priority:	5 - Normal	Status:	Fixed
Assigned to:	Marko Lindqvist <cazfi>	Open/Closed:	Closed
Release:		Operating System:	None
Planned Release:	2.3.4, 2.4.0, 2.5.0

Add a New Comment (Rich Markup):

You are not logged in

Please log in, so followups can be emailed to you.

Sun 27 Jan 2013 10:42:09 PM UTC, SVN revision 22217: Send transport's loaded status to client side when cargo unit enters it. Previously loaded status did not update in case of emergency loading, such as cargo escaping from another transport being destroyed or out-of-fuel unit entering transport. Do not unnecessarily send transport's information if it already had some cargo, and thus 'loaded' status does not change. See gna bug #20454 (Browse SVN revision 22217)	Marko Lindqvist <cazfi>
Sun 27 Jan 2013 10:41:27 PM UTC, SVN revision 22216: Send transport's loaded status to client side when cargo unit enters it. Previously loaded status did not update for enemy transports in case of emergency loading, such as cargo escaping from another transport being destroyed or out-of-fuel unit entering transport. Do not unnecessarily send transport's information if it already had some cargo, and thus 'loaded' status does not change. See gna bug #20454 (Browse SVN revision 22216)	Marko Lindqvist <cazfi>
Sun 27 Jan 2013 10:41:22 PM UTC, SVN revision 22215: Send transport's loaded status to client side when cargo unit enters it. Previously loaded status did not update for enemy transports in case of emergency loading, such as cargo escaping from another transport being destroyed or out-of-fuel unit entering transport. Do not unnecessarily send transport's information if it already had some cargo, and thus 'loaded' status does not change. See gna bug #20454 (Browse SVN revision 22215)	Marko Lindqvist <cazfi>
Fri 25 Jan 2013 01:22:51 PM UTC, comment #1: - Do not send transporter twice in case of normal loading. Before this patch it was sent always when someone entered, now it's sent only for the first cargo unit (change of transported status) - Also S2_3 version (file #17022, file #17023)	Marko Lindqvist <cazfi>
Fri 25 Jan 2013 12:38:14 PM UTC, original submission: This is issue 7 from bug #19821. Reproducing instructions for S2_4 are in bug #19821#comment9 When unit gets emergency-loaded to transporter, transporter loaded status is not updated client side. In S2_3 this affects all transporters, in S2_4 and TRUNK only enemy ones (ones to which player doesn't see inside) Fix for S2_4 & TRUNK attached.	Marko Lindqvist <cazfi>

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):


Comment:

Attached Files

file #17022: SendLoadedStatus-2.patch added by cazfi (5kB - text/x-diff)

file #17023: SendLoadedStatus-S2_3-2.patch added by cazfi (4kB - text/x-diff)

file #17021: SendLoadedStatus.patch added by cazfi (4kB - text/x-diff)

Depends on the following items: None found

Items that depend on this one

[metaticket] Numerous issues with wipe_unit() wiping transporter with cargo (bug #19821, Fixed)

Digest:
bug dependencies.

Carbon-Copy List

-unavailable- added by cazfi (Submitted the item)

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

Please enter the title of George Orwell's famous dystopian book (it's a date):

Follow 7 latest changes.

Date	Changed By	Updated Field	Previous Value	Replaced By
Sun 27 Jan 2013 10:42:44 PM UTC	cazfi	Status	None	Fixed
		Assigned to	None	cazfi
		Open/Closed	Open	Closed
Fri 25 Jan 2013 01:22:51 PM UTC	cazfi	Attached File	-	Added SendLoadedStatus-2.patch, #17022
		Attached File	-	Added SendLoadedStatus-S2_3-2.patch, #17023
Fri 25 Jan 2013 12:39:05 PM UTC	cazfi	Dependencies	-	bugs #19821 is dependent
Fri 25 Jan 2013 12:38:15 PM UTC	cazfi	Attached File	-	Added SendLoadedStatus.patch, #17021

Show feedback again