Anonymous | Login | Signup for a new account | 2024-04-24 15:08 UTC |
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0000282 | Zandronum | [All Projects] Suggestion | public | 2011-02-06 22:09 | 2018-09-30 19:51 | ||||
Reporter | DevilHunter | ||||||||
Assigned To | Dusk | ||||||||
Priority | normal | Severity | feature | Reproducibility | N/A | ||||
Status | closed | Resolution | fixed | ||||||
Platform | Microsoft | OS | Windows | OS Version | XP/Vista/7 | ||||
Product Version | 98d | ||||||||
Target Version | Fixed in Version | 1.0 | |||||||
Summary | 0000282: Have Delmap into Console/Rcon Commands | ||||||||
Description | Should have the option, for some wads that have huge maplists, and you need to remove just one. Would be better then Shutting down the server, and starting it back up, or for people who only have rcon, and no way to know what server config to re-exec on the server. | ||||||||
Steps To Reproduce | N/A | ||||||||
Attached Files | |||||||||
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 |
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 |
Copyright © 2000 - 2024 MantisBT Team |