]> de.git.xonotic.org Git - xonotic/xonotic-data.pk3dir.git/blobdiff - qcsrc/client/mapvoting.qc
Fix #2701 "Unwanted random auto-join". This patch basically reverts 4941322
[xonotic/xonotic-data.pk3dir.git] / qcsrc / client / mapvoting.qc
index 8d4d8ba9f6b8900d7e156d1580302bb188e29817..29640128c0abf575c8a8b43fa03cb81c5d044e9e 100644 (file)
@@ -1,13 +1,18 @@
 #include "mapvoting.qh"
 
-#include "autocvars.qh"
-#include "miscfunctions.qh"
-#include "defs.qh"
-#include "hud/_mod.qh"
-#include "hud/panel/scoreboard.qh"
-
+#include <client/draw.qh>
+#include <client/hud/_mod.qh>
+#include <client/hud/panel/scoreboard.qh>
 #include <common/mapinfo.qh>
+#include <common/util.qh>
+
+// MapVote (#21)
 
+void MapVote_Draw_Export(int fh)
+{
+       // allow saving cvars that aesthetically change the panel into hud skin files
+       HUD_Write_Cvar("hud_panel_mapvote_highlight_border");
+}
 
 int mv_num_maps;
 
@@ -41,9 +46,22 @@ const int NUM_SSDIRS = 4;
 string ssdirs[NUM_SSDIRS];
 int n_ssdirs;
 
+bool PreviewExists(string name)
+{
+       if(autocvar_cl_readpicture_force)
+               return false;
+
+       if (fexists(strcat(name, ".tga"))) return true;
+       if (fexists(strcat(name, ".png"))) return true;
+       if (fexists(strcat(name, ".jpg"))) return true;
+       if (fexists(strcat(name, ".pcx"))) return true;
+
+       return false;
+}
+
 string MapVote_FormatMapItem(int id, string map, float _count, float maxwidth, vector fontsize)
 {
-    TC(int, id);
+       TC(int, id);
        string pre, post;
        pre = sprintf("%d. ", id+1);
        if(mv_detail)
@@ -64,7 +82,7 @@ string MapVote_FormatMapItem(int id, string map, float _count, float maxwidth, v
 
 vector MapVote_RGB(int id)
 {
-    TC(int, id);
+       TC(int, id);
        if(!(mv_flags[id] & GTV_AVAILABLE))
                return '1 1 1';
        if(id == mv_ownvote)
@@ -77,7 +95,7 @@ vector MapVote_RGB(int id)
 
 void GameTypeVote_DrawGameTypeItem(vector pos, float maxh, float tsize, string gtype, string pic, float _count, int id)
 {
-    TC(int, id);
+       TC(int, id);
        // Find the correct alpha
        float alpha;
        if(!(mv_flags_start[id] & GTV_AVAILABLE))
@@ -189,7 +207,7 @@ void GameTypeVote_DrawGameTypeItem(vector pos, float maxh, float tsize, string g
 
 void MapVote_DrawMapItem(vector pos, float isize, float tsize, string map, string pic, float _count, int id)
 {
-    TC(int, id);
+       TC(int, id);
        vector img_size = '0 0 0';
        string label;
        float text_size;
@@ -264,7 +282,7 @@ void MapVote_DrawMapItem(vector pos, float isize, float tsize, string map, strin
 
 void MapVote_DrawAbstain(vector pos, float isize, float tsize, float _count, int id)
 {
-    TC(int, id);
+       TC(int, id);
        vector rgb;
        float text_size;
        string label;
@@ -281,7 +299,7 @@ void MapVote_DrawAbstain(vector pos, float isize, float tsize, float _count, int
 
 vector MapVote_GridVec(vector gridspec, int i, int m)
 {
-    TC(int, i); TC(int, m);
+       TC(int, i); TC(int, m);
        int r = i % m;
        return
                '1 0 0' * (gridspec.x * r)
@@ -321,7 +339,7 @@ float MapVote_Selection(vector topleft, vector cellsize, float rows, float colum
        return mv_mouse_selection;
 }
 
-vector prev_mousepos;
+// draws map vote or gametype vote
 void MapVote_Draw()
 {
        string map;
@@ -338,15 +356,6 @@ void MapVote_Draw()
 
        HUD_Panel_LoadCvars();
 
-       if (!autocvar_hud_cursormode)
-       {
-               if (mousepos.x != prev_mousepos.x || mousepos.y != prev_mousepos.y)
-               {
-                       mv_selection_keyboard = 0;
-                       prev_mousepos = mousepos;
-               }
-       }
-
        center = (vid_conwidth - 1)/2;
        xmin = vid_conwidth * 0.08;
        xmax = vid_conwidth - xmin;
@@ -359,6 +368,11 @@ void MapVote_Draw()
                ymax -= chat_sizey;
 
        hud_fontsize = HUD_GetFontsize("hud_fontsize");
+       if (gametypevote)
+       {
+               gtv_text_size = hud_fontsize * 1.4;
+               gtv_text_size_small = hud_fontsize * 1.1;
+       }
 
        pos.y = ymin;
        pos.z = 0;
@@ -459,7 +473,7 @@ void MapVote_Draw()
        mv_selection = MapVote_Selection(pos, dist, rows, mv_columns);
 
        if (mv_top2_time)
-               mv_top2_alpha = max(0.2, 1 - (time - mv_top2_time)*(time - mv_top2_time));
+               mv_top2_alpha = max(0.2, 1 - (time - mv_top2_time) ** 2);
 
        void (vector, float, float, string, string, float, float) DrawItem;
 
@@ -491,7 +505,7 @@ void MapVote_Draw()
 
 void Cmd_MapVote_MapDownload(int argc)
 {
-    TC(int, argc);
+       TC(int, argc);
        entity pak;
 
        if(argc != 2 || !mv_pk3list)
@@ -522,7 +536,7 @@ void Cmd_MapVote_MapDownload(int argc)
 
 void MapVote_CheckPK3(string pic, string pk3, int id)
 {
-    TC(int, id);
+       TC(int, id);
        entity pak;
        pak = spawn();
        pak.netname = pk3;
@@ -544,7 +558,7 @@ void MapVote_CheckPK3(string pic, string pk3, int id)
 
 void MapVote_CheckPic(string pic, string pk3, int id)
 {
-    TC(int, id);
+       TC(int, id);
        // never try to retrieve a pic for the "don't care" 'map'
        if(mv_abstain && id == mv_num_maps - 1)
                return;
@@ -587,7 +601,7 @@ void MapVote_ReadMask()
 
 void MapVote_ReadOption(int i)
 {
-    TC(int, i);
+       TC(int, i);
        string map = strzone(ReadString());
        string pk3 = strzone(ReadString());
        int j = bound(0, ReadByte(), n_ssdirs - 1);
@@ -604,7 +618,7 @@ void MapVote_ReadOption(int i)
 
 void GameTypeVote_ReadOption(int i)
 {
-    TC(int, i);
+       TC(int, i);
        string gt = strzone(ReadString());
 
        mv_maps[i] = gt;
@@ -623,7 +637,7 @@ void GameTypeVote_ReadOption(int i)
        }
        else
        {
-               Gametype type = MapInfo_Type_FromString(gt);
+               Gametype type = MapInfo_Type_FromString(gt, false);
                mv_pk3[i] = strzone(MapInfo_Type_ToText(type));
                mv_desc[i] = MapInfo_Type_Description(type);
        }
@@ -680,9 +694,6 @@ void MapVote_Init()
                mapvote_chosenmap = strzone(ReadString());
                if ( gametypevote == 2 )
                        gametypevote = 0;
-
-               gtv_text_size = hud_fontsize*1.4;
-               gtv_text_size_small = hud_fontsize*1.1;
        }
 
        MapVote_ReadMask();
@@ -710,13 +721,13 @@ void MapVote_Init()
 
 void MapVote_SendChoice(int index)
 {
-    TC(int, index);
+       TC(int, index);
        localcmd(strcat("\nimpulse ", ftos(index+1), "\n"));
 }
 
 int MapVote_MoveLeft(int pos)
 {
-    TC(int, pos);
+       TC(int, pos);
        int imp;
        if ( pos < 0 )
                imp = mv_num_maps - 1;
@@ -728,7 +739,7 @@ int MapVote_MoveLeft(int pos)
 }
 int MapVote_MoveRight(int pos)
 {
-    TC(int, pos);
+       TC(int, pos);
        int imp;
        if ( pos < 0 )
                imp = 0;
@@ -740,7 +751,7 @@ int MapVote_MoveRight(int pos)
 }
 int MapVote_MoveUp(int pos)
 {
-    TC(int, pos);
+       TC(int, pos);
        int imp;
        if ( pos < 0 )
                imp = mv_num_maps - 1;
@@ -749,9 +760,11 @@ int MapVote_MoveUp(int pos)
                imp = pos - mv_columns;
                if ( imp < 0 )
                {
-                       imp = floor(mv_num_maps/mv_columns)*mv_columns + pos % mv_columns;
-                       if ( imp >= mv_num_maps )
-                               imp -= mv_columns;
+                       int mv_rows = ceil(mv_num_maps / mv_columns);
+                       if (imp == -mv_columns) // pos == 0
+                               imp = mv_columns * mv_rows - 1;
+                       else
+                               imp = imp + mv_columns * mv_rows - 1;
                }
        }
        if ( !(mv_flags[imp] & GTV_AVAILABLE) && imp != mv_ownvote )
@@ -760,7 +773,7 @@ int MapVote_MoveUp(int pos)
 }
 int MapVote_MoveDown(int pos)
 {
-    TC(int, pos);
+       TC(int, pos);
        int imp;
        if ( pos < 0 )
                imp = 0;
@@ -768,7 +781,12 @@ int MapVote_MoveDown(int pos)
        {
                imp = pos + mv_columns;
                if ( imp >= mv_num_maps )
-                       imp = imp % mv_columns;
+               {
+                       if ((imp % mv_columns) == mv_columns - 1)
+                               imp = 0;
+                       else
+                               imp = imp % mv_columns + 1;
+               }
        }
        if ( !(mv_flags[imp] & GTV_AVAILABLE) && imp != mv_ownvote )
                imp = MapVote_MoveDown(imp);
@@ -777,9 +795,9 @@ int MapVote_MoveDown(int pos)
 
 float MapVote_InputEvent(int bInputType, float nPrimary, float nSecondary)
 {
-    TC(int, bInputType);
-       float imp;
+       TC(int, bInputType);
 
+       static int first_digit = 0;
        if (!mv_active)
                return false;
 
@@ -791,29 +809,40 @@ float MapVote_InputEvent(int bInputType, float nPrimary, float nSecondary)
                return true;
        }
 
-       if (bInputType != 0)
+       if (bInputType == 2)
+       {
+               mv_selection_keyboard = 0;
                return false;
+       }
+
+       // at this point bInputType can only be 0 or 1 (key pressed or released)
+       bool key_pressed = (bInputType == 0);
 
-       if ('0' <= nPrimary && nPrimary <= '9')
+       if (key_pressed)
        {
-               imp = nPrimary - '0';
-               if (imp == 0) imp = 10;
-               localcmd(strcat("\nimpulse ", ftos(imp), "\n"));
-               return true;
+               if (nPrimary == K_ALT) hudShiftState |= S_ALT;
+               if (nPrimary == K_CTRL) hudShiftState |= S_CTRL;
+               if (nPrimary == K_SHIFT) hudShiftState |= S_SHIFT;
        }
-       switch(nPrimary)
+       else
        {
-               case K_KP_1: localcmd("\nimpulse 1\n"); return true;
-               case K_KP_2: localcmd("\nimpulse 2\n"); return true;
-               case K_KP_3: localcmd("\nimpulse 3\n"); return true;
-               case K_KP_4: localcmd("\nimpulse 4\n"); return true;
-               case K_KP_5: localcmd("\nimpulse 5\n"); return true;
-               case K_KP_6: localcmd("\nimpulse 6\n"); return true;
-               case K_KP_7: localcmd("\nimpulse 7\n"); return true;
-               case K_KP_8: localcmd("\nimpulse 8\n"); return true;
-               case K_KP_9: localcmd("\nimpulse 9\n"); return true;
-               case K_KP_0: localcmd("\nimpulse 10\n"); return true;
+               if (nPrimary == K_ALT) hudShiftState -= (hudShiftState & S_ALT);
+               if (nPrimary == K_CTRL) hudShiftState -= (hudShiftState & S_CTRL);
+               if (nPrimary == K_SHIFT) hudShiftState -= (hudShiftState & S_SHIFT);
+
+               if (nPrimary == K_CTRL)
+                       first_digit = 0;
+       }
 
+       // Key release events must be handled by the engine otherwise the on-press command such as +jump
+       // executed by pressing SPACE before entering the map voting screen won't be followed by the
+       // on-release command (-jump) on key release once entered the map voting screen, causing +jump
+       // to stay active even on the next map and automatically forcing the player to join
+       if (!key_pressed) return false;
+
+       int imp = 0;
+       switch(nPrimary)
+       {
                case K_RIGHTARROW:
                        mv_selection_keyboard = 1;
                        mv_selection = MapVote_MoveRight(mv_selection);
@@ -836,6 +865,27 @@ float MapVote_InputEvent(int bInputType, float nPrimary, float nSecondary)
                        if ( mv_selection_keyboard )
                                MapVote_SendChoice(mv_selection);
                        return true;
+               case '1': case K_KP_1: imp = 1; break;
+               case '2': case K_KP_2: imp = 2; break;
+               case '3': case K_KP_3: imp = 3; break;
+               case '4': case K_KP_4: imp = 4; break;
+               case '5': case K_KP_5: imp = 5; break;
+               case '6': case K_KP_6: imp = 6; break;
+               case '7': case K_KP_7: imp = 7; break;
+               case '8': case K_KP_8: imp = 8; break;
+               case '9': case K_KP_9: imp = 9; break;
+               case '0': case K_KP_0: imp = 10; break;
+       }
+
+       if (imp && hudShiftState & S_CTRL)
+       {
+               if (!first_digit)
+               {
+                       first_digit = imp % 10;
+                       return true;
+               }
+               else
+                       imp = first_digit * 10 + (imp % 10);
        }
 
        if (nPrimary == K_MOUSE1)
@@ -843,11 +893,17 @@ float MapVote_InputEvent(int bInputType, float nPrimary, float nSecondary)
                mv_selection_keyboard = 0;
                mv_selection = mv_mouse_selection;
                if (mv_selection >= 0)
-               {
                        imp = min(mv_selection + 1, mv_num_maps);
+       }
+
+       if (nPrimary == K_MOUSE2)
+               return true; // do nothing
+
+       if (imp)
+       {
+               if (imp <= mv_num_maps)
                        localcmd(strcat("\nimpulse ", ftos(imp), "\n"));
-                       return true;
-               }
+               return true;
        }
 
        return false;