Remove Windows support

[solanum.git] / extensions / helpops.c

diff --git a/extensions/helpops.c b/extensions/helpops.c
index 1fb3b42381610b35e589c4f2f2d27e4d6d4b4ee9..72d812902d0f8e9657b0ebec3d186d7261d567f3 100644 (file)
--- a/extensions/helpops.c
+++ b/extensions/helpops.c
@@ -39,7 +39,7 @@ mapi_hfn_list_av1 helpops_hfnlist[] = {
        { NULL, NULL }
 };
 
-#define UMODECHAR_HELPOPS 'H'
+#define UMODECHAR_HELPOPS 'h'
 
 struct Message dehelper_msgtab = {
        "DEHELPER", 0, 0, 0, 0,
@@ -53,9 +53,9 @@ mo_dehelper(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *sou
 {
        struct Client *target_p;
 
-       if (!IsOperAdmin(source_p))
+       if (!HasPrivilege(source_p, "oper:dehelper"))
        {
-               sendto_one(source_p, form_str(ERR_NOPRIVS), me.name, source_p->name, "admin");
+               sendto_one(source_p, form_str(ERR_NOPRIVS), me.name, source_p->name, "dehelper");
                return;
        }
 
@@ -91,6 +91,7 @@ static void
 do_dehelper(struct Client *source_p, struct Client *target_p)
 {
        const char *fakeparv[4];
+       static const char minus_helpops[3] = {'-', UMODECHAR_HELPOPS, '\0'};
 
        if(!(target_p->umodes & user_modes[UMODECHAR_HELPOPS]))
                return;
@@ -100,7 +101,7 @@ do_dehelper(struct Client *source_p, struct Client *target_p)
        sendto_one_notice(target_p, ":*** %s is using DEHELPER on you", source_p->name);
 
        fakeparv[0] = fakeparv[1] = target_p->name;
-       fakeparv[2] = "-H";
+       fakeparv[2] = minus_helpops;
        fakeparv[3] = NULL;
        user_mode(target_p, target_p, 3, fakeparv);
 }
@@ -219,9 +220,8 @@ h_hdl_umode_changed(hook_data_umode_changed *hdata)
 {
        struct Client *source_p = hdata->client;
 
-       /* didn't change +H umode, we don't need to do anything */
-       if (!((hdata->oldumodes ^ source_p->umodes) & user_modes[UMODECHAR_HELPOPS]))
-               return;
+       /* didn't change +h umode, we don't need to do anything */
+       bool changed = (hdata->oldumodes ^ source_p->umodes) & user_modes[UMODECHAR_HELPOPS];
 
        if (source_p->umodes & user_modes[UMODECHAR_HELPOPS])
        {
@@ -229,13 +229,19 @@ h_hdl_umode_changed(hook_data_umode_changed *hdata)
                {
                        source_p->umodes &= ~user_modes[UMODECHAR_HELPOPS];
                        sendto_one(source_p, form_str(ERR_NOPRIVS), me.name, source_p->name, "usermode:helpops");
+                       /* they didn't ask for +h so we must be removing it */
+                       if (!changed)
+                               helper_delete(source_p);
                        return;
                }
 
-               helper_add(source_p);
+               if (changed)
+                       helper_add(source_p);
        }
-       else if (!(source_p->umodes & user_modes[UMODECHAR_HELPOPS]))
+       else if (changed)
+       {
                helper_delete(source_p);
+       }
 }
 
 static void