Zandronum Chat on our Discord Server Get the latest version: 3.2
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000981Zandronum[All Projects] Suggestionpublic2012-08-19 01:042014-05-12 00:18
ReporterWatermelon 
Assigned To 
PrioritynormalSeverityfeatureReproducibilityN/A
StatusclosedResolutionduplicate 
PlatformOSOS Version
Product Version1.0-beta 
Target VersionFixed in Version 
Summary0000981: Have BLUERETURN and REDRETURN ACS script types PlayerNumber() results in the 'returnee' of the flag
DescriptionBLUERETURN / REDRETURN / WHITERETURN script types are activated by the world. It would be nice to be able to grab the player number of the player who returned it instead of getting -1 back from the function.

Would it be possible to convert it from World activated to Player activated? Or maybe have it contain a parameter that would have the player number (like DISCONNECT (int playerNum) has) and still run it for the world?



REASON FOR REQUEST:
Allows ACS access to players who return the flag.
One example is that I can record the amount of times someone has returned it and print out who returned the flag the most. The International Doom League uses stuff like this in Odamex for their game stats.
Additional InformationThis is the code inside of the attached wad.


#include "zcommon.acs"
#library "PICKTEST"

script 100 BLUERETURN
{
    PrintBold(s: "Returned by: ", d: PlayerNumber());
}
Attached Files? file icon PickupTest.wad [^] (330 bytes) 2012-08-19 01:04

- Relationships
related to 0001794closedTorr Samaho New script type EVENT 

-  Notes
User avatar (0004400)
Torr Samaho (administrator)
2012-08-19 12:48

 I think I could change the activator of these scripts quite easily, but would this break any existing mods?
User avatar (0004403)
Watermelon (developer)
2012-08-19 14:39
edited on: 2012-08-19 14:42

 I've honestly never come across any wad so far that even uses this. I actually discovered it yesterday by fluke because someone added it on the ZDoomWiki.

I just went through every CTF wad I had and there is none that I've ever played that use any of the PICKUP/COLORFLAGRETURN ACS.
User avatar (0004404)
Dusk (developer)
2012-08-19 14:44

 > I actually discovered it yesterday by fluke because someone added it on the ZDoomWiki.
I added them to ZDoom Wiki after stumbling upon them while going through the source code..

I personally think the existing types should get deprecated and superceded by more generalized types, say script X FLAGRETURN (int returner, int team)
User avatar (0004405)
Torr Samaho (administrator)
2012-08-19 16:04
Quote from Dusk
I personally think the existing types should get deprecated and superceded by more generalized types, say script X FLAGRETURN (int returner, int team)

Good idea. We should look into this after 1.0 is out.
User avatar (0004552)
ZzZombo (reporter)
2012-09-01 15:36
edited on: 2012-09-01 15:38
Quote
I think I could change the activator of these scripts quite easily, but would this break any existing mods?

I don't think that currently there is any mods that relies on activator of return scripts because it's pointless.
User avatar (0004725)
Watermelon (developer)
2012-09-20 19:33

 > I don't think that currently there is any mods that relies on activator of return scripts because it's pointless.

No it's not pointless (or I wouldn't have opened a ticket with an example that you can see). Play some CTF more with us and you'll see some really interesting applications that would benefit CTF ACS-wise.
User avatar (0004918)
Watermelon (developer)
2012-10-02 18:53

 Would it be worth opening a new ticket for the suggestion of stuff like "script X FLAGRETURN (int returner, int team)" ?
User avatar (0004925)
Dusk (developer)
2012-10-03 12:12

 I see no reason why we can't handle it here.. since this FLAGRETURN type would address exactly this ticket.
User avatar (0004964)
Watermelon (developer)
2012-10-05 14:58

 Would it be difficult to create these new ACS flags?
Also would 'int returner' return the player TID or player number?
User avatar (0004965)
Blzut3 (administrator)
2012-10-05 18:12

 One thing to consider before adding this as a third xRETURN script type, perhaps the behavior module could be scanned for a REDRETURN script and if present use the current behavior. Otherwise replace BLUERETURN with FLAGRETURN and pass those arguments?

There's nothing in particular wrong with just adding a new type, but it does seem wasteful since I think most of the maps that used these scripts are probably already broken by the changes that were made to skulltag.wad over the years. (And ever since the automatic "simple" CTF method was introduce the two script types have been mostly forgotten.)

Issue Community Support
This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.
Supporters: Llewellyn
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2012-08-19 01:04 Watermelon New Issue
2012-08-19 01:04 Watermelon File Added: PickupTest.wad
2012-08-19 12:48 Torr Samaho Note Added: 0004400
2012-08-19 14:39 Watermelon Note Added: 0004403
2012-08-19 14:42 Watermelon Note Edited: 0004403 View Revisions
2012-08-19 14:44 Dusk Note Added: 0004404
2012-08-19 16:04 Torr Samaho Note Added: 0004405
2012-09-01 15:36 ZzZombo Note Added: 0004552
2012-09-01 15:37 ZzZombo Note Edited: 0004552 View Revisions
2012-09-01 15:38 ZzZombo Note Edited: 0004552 View Revisions
2012-09-20 19:33 Watermelon Note Added: 0004725
2012-10-02 18:53 Watermelon Note Added: 0004918
2012-10-03 12:12 Dusk Note Added: 0004925
2012-10-05 14:58 Watermelon Note Added: 0004964
2012-10-05 18:12 Blzut3 Note Added: 0004965
2014-05-12 00:18 Watermelon Relationship added related to 0001794
2014-05-12 00:18 Watermelon Status new => closed
2014-05-12 00:18 Watermelon Resolution open => duplicate




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2025 MantisBT Team
Powered by Mantis Bugtracker