Bug 198206 - [IRC] supporting special chars in userlist, when the nick opped
Summary: [IRC] supporting special chars in userlist, when the nick opped
Status: RESOLVED INVALID
Alias: None
Product: ECF
Classification: RT
Component: ecf.core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 2.0.0   Edit
Assignee: Chris Aniszczyk CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, contributed
Depends on:
Blocks:
 
Reported: 2007-07-29 09:20 EDT by Ferenc Boldog CLA
Modified: 2014-02-14 16:27 EST (History)
2 users (show)

See Also:

Attachments
bug example (13.05 KB, image/png)
2007-07-30 03:42 EDT, Ferenc Boldog CLA 		no flags Details
bugfix (8.88 KB, text/plain)
2007-07-30 03:47 EDT, Ferenc Boldog CLA 		no flags Details
updated patch for the bug (8.67 KB, text/plain)
2007-08-11 18:17 EDT, Ferenc Boldog CLA 		no flags Details
v1.27 version (8.58 KB, text/plain)
2007-08-21 15:15 EDT, Ferenc Boldog CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ferenc Boldog CLA 2007-07-29 09:20:59 EDT 
The fix #198193 show another bug in the channel user list.
When any nick with special chars has operator right, the nickname not displayed correctly. (interesting the voice flag displayed correctly)
I working on this now ;)
Comment 1 Ferenc Boldog CLA 2007-07-30 03:42:22 EDT 
Created attachment 74902 [details]
bug example
Comment 2 Ferenc Boldog CLA 2007-07-30 03:47:45 EDT 
Created attachment 74903 [details]
bugfix

This patch fixing the bug, but i cannot test with other protocols (only with irc).
The big question, when targetID.getName() is url like (ie: irc://xzy@host.com) the username is url encoded, or never will be url like?
Comment 3 Scott Lewis CLA 2007-08-05 16:20:03 EDT 
Adding keywords.  Assigning to myself for verification.
Comment 4 Chris Aniszczyk CLA 2007-08-05 16:35:43 EDT 
dropped off the plate for 1.0.2, will look at in 1.1

Can you come up with a new clean patch? I notice there was some spacing issues.

btw, assigning to bug to you :)
Comment 5 Ferenc Boldog CLA 2007-08-05 17:59:53 EDT 
:))
Ok! I create, updated, fixed patch for that.
Comment 6 Ferenc Boldog CLA 2007-08-11 18:17:32 EDT 
Created attachment 75915 [details]
updated patch for the bug

Hello, 
I update the patch to the current cvs version. Tested in IRC.
Comment 7 Ferenc Boldog CLA 2007-08-21 15:15:22 EDT 
Created attachment 76580 [details]
v1.27 version

updated patch to current cvs version (1.27) of file: ChatRoomManagerView.java
Comment 8 Scott Lewis CLA 2007-10-16 13:40:49 EDT 
Was the patch 76580 for this bug ever applied, tested, and committed?  I don't see any record in the IP log, so I assume not.
Comment 9 Scott Lewis CLA 2008-05-23 15:38:58 EDT 
zx, was this contribution ever tested/applied?
Comment 10 Scott Lewis CLA 2014-02-14 16:27:25 EST 
Resolving as invalid, as it has been many years since zx has been an ECF committer.

If this hasn't been properly committed, and Ferenc is still available in some form, please reopen and attach a git patch and I will apply it.  Thanks.