Closed Bug 180574 Opened 22 years ago Closed 19 years ago

ban, unban aliases not implemented

Categories

(Other Applications :: ChatZilla, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: hcymulng02, Assigned: rginda)

Details

(Whiteboard: [cz-patch][cz-0.9.68])

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3a) Gecko/20021115
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3a) Gecko/20021115

the CIRCChanUser object (in irc.js) has setOp, setVoice, kick, kickBan and
setBan methods defined, but there are only corresponding commands for setOp (/op
/deop), setVoice (/voice, /devoice), and kick (/kick). Where are the commands
(or aliases in mIRCspeak) for kickban and ban/unban? They ought to be in
commands.js and handlers.js, but they're not.

I wrote my own version of /kickban by coping onInputKick and replacing "kick"
with "kickBan" in the appropriate places. It works fine, but I'm puzzled as to
why this rather basic functionality hasn't made it in yet.

Reproducible: Always

Steps to Reproduce:
1. Connect to a server, join a channel, attain op status.
2. Try to kickban using the /kickban command.
3. Try to set or unset a ban using the /ban or /unban commands.

Actual Results:  
An error message was displayed.

Expected Results:  
Called the appropriate method in CIRCChanUser, causing the appropriate action
(kickban/ban/unban) to be taken.
Confirming

As far as I see, there has to be implanted that at line
http://lxr.mozilla.org/mozilla/source/extensions/irc/js/lib/irc.js#1312 (or
elsewhere but there you can see how its done)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Ok, i checked: In the popular irc-client mirc there is only /ban implanted. Ok,
we could also implant your commands in chatzilla, no question :)
The 0.9.46 version I'm using right now has got a /kick-ban command, so that's
been done. That leaves just ban/unban, updating summary.
Summary: kickban, ban, unban aliases not implemented → ban, unban aliases not implemented
Product: Core → Other Applications
This adds ban and unban, both of which attempt to find the nickname on the
current channel or current server, and if that fails bans either "nickname!*@*"
or if the input has ! or @ in it, the exact input.
Attachment #180984 - Flags: review?(samuel)
Whiteboard: cz-patch
Whiteboard: cz-patch → [cz-patch][cz-0.9.68]
Attachment #180984 - Flags: review?(samuel) → review+
OS: Windows XP → All
Hardware: PC → All
Attachment #180984 - Flags: approval1.8b3?
Comment on attachment 180984 [details] [diff] [review]
Add /ban and /unban

a=shaver.
Attachment #180984 - Flags: approval1.8b3? → approval1.8b3+
Checked in -> FIXED.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: