0000282: Have Delmap into Console/Rcon Commands

Notes
(0002598) Dusk (developer) 2012-02-13 18:26 edited on: 2012-02-14 16:13	I got a sample implementation working in Not Ebola: 'https://bitbucket.org/CrimsonDusk/notebola/changeset/19fe066aad1f [^]' EDIT:'https://bitbucket.org/CrimsonDusk/notebola/changeset/1f5895cb0f21 [^]' adds delmap_idx, which allows deletion of a map based on index number. Becomes useful when there's multiple entries of the same map in map rotation. Uses the same indices as what `maplist` prints out - though is this a good idea since `maplist` starts at 1 and not at 0? EDIT 2: Updated links in light of the actual naming of the fork..

(0003139) DevilHunter (reporter) 2012-04-05 16:12	Just wondering, since your a Dev now Dusk, has anything been done wih this in Skulltag?

(0003168) Torr Samaho (administrator) 2012-04-07 14:33	Please don't post notes that essentially just say "When will this be added/fixed?" or "Will this be added/fixed?". This just pollutes the ticket and makes it more difficult to see the essential information. Anyway, I added the patch and made a small fix to delmap_idx. But don't feel encouraged by this to post more notes that are just intended to promote your favorite suggestion.

(0003173) Dusk (developer) 2012-04-07 15:50 edited on: 2012-04-07 16:02	[17:26:47] <+Dusk> actually... wait, what? [17:26:47] <+CIA-1> Skulltag: Benjamin Berkels <> * f81e00233235 r3310 /src/maprotation.cpp: Fixed: delmap_idx removed the first map with the name of the map with the given index, not necessarily the map with the index. This makes a difference if one map is in the rotation more than once. I like your timing. :) Here is insertmap, too:'https://bitbucket.org/CrimsonDusk/notebola/changeset/d7da1d18f8ac [^]' Allows adding maps into other positions rather than just at the end. To avoid code duplication, I extended the functionality of MAPROTATION_AddMap and gave it a third, optional parameter for position. 0, the default, means the end of the rotation. Said function also prints out the position the map was added at, as well. I figured it'd be useful for server admins. Should it be called insmap, though, for consistency?

(0003291) Torr Samaho (administrator) 2012-04-15 18:18	I think insertmap is fine and added your patch.

(0003509) DevilHunter (reporter) 2012-04-30 07:57	Just tested this on Armada.. Due to my laggy connection, it didn't show up in Rcon, but in Screen, it did (linux). Have yet to test this on Windows, but so far looks good, and works. Thanks Dusk, and Torr :p

**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:	No one explicitly supports this issue yet.
Opponents:	No one explicitly opposes this issue yet.

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:

No one explicitly supports this issue yet.

Opponents:

No one explicitly opposes this issue yet.

Issue History
Date Modified	Username	Field	Change
2011-02-06 22:09	DevilHunter	New Issue
2012-02-13 18:26	Dusk	Note Added: 0002598
2012-02-13 20:41	Dusk	Note Edited: 0002598	View Revisions
2012-02-13 22:41	Dusk	Note Edited: 0002598	View Revisions
2012-02-14 16:13	Dusk	Note Edited: 0002598	View Revisions
2012-04-05 16:12	DevilHunter	Note Added: 0003139
2012-04-07 14:33	Torr Samaho	Note Added: 0003168
2012-04-07 14:33	Torr Samaho	Assigned To	=> Dusk
2012-04-07 14:33	Torr Samaho	Status	new => feedback
2012-04-07 15:50	Dusk	Note Added: 0003173
2012-04-07 15:50	Dusk	Note Edited: 0003173	View Revisions
2012-04-07 15:52	Dusk	Note Edited: 0003173	View Revisions
2012-04-07 16:02	Dusk	Note Edited: 0003173	View Revisions
2012-04-15 18:18	Torr Samaho	Note Added: 0003291
2012-04-30 07:57	DevilHunter	Note Added: 0003509
2012-04-30 07:57	DevilHunter	Status	feedback => assigned
2012-05-01 02:00	Torr Samaho	Status	assigned => resolved
2012-05-01 02:00	Torr Samaho	Fixed in Version	=> 1.0
2012-05-01 02:00	Torr Samaho	Resolution	open => fixed
2018-09-30 19:51	Blzut3	Status	resolved => closed

Relationships