MantisBT - Doomseeker |
View Issue Details |
|
ID | Project | Category | View Status | Date Submitted | Last Update |
0003193 | Doomseeker | [All Projects] Bug | public | 2017-07-21 23:53 | 2018-10-27 22:55 |
|
Reporter | Zom-B | |
Assigned To | Zalewa | |
Priority | low | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | |
Platform | Microsoft | OS | Windows | OS Version | XP/Vista/7 |
Product Version | 1.1 | |
Target Version | 1.2 | Fixed in Version | 1.2 | |
|
Summary | 0003193: Buddies: "\c[" is not a "valid" standard color sequence (was: Server owner does not show up in buddies list) |
Description | Extended colors from third-party "NewTextColors_260.pk3" (for Doom) are not recognized/escaped correctly in the Buddies interface of Doomseeker.
--
(Original description below)
The owner of a server, when plaing on his own server, does not show up in buddies list. All other buddies on same server do. I had the owner change his name, added the new name in the buddies list, and he still didn't show up. I found another server with presumably a local player (ping 1) but after adding him he does show up, so I don't know what's going on. |
Steps To Reproduce | See comment 0018437
--
(Original steps to reproduce below)
Add "Johnny Blaze" to your buddies list, when he's playing in one of the "MOP + Chillax" servers (usually fri/sat around midnight GMT, sometimes other times too) |
Additional Information | Relevant info on NewTextColors:'https://zandronum.com/forum/viewtopic.php?t=3502 [^]' |
Tags | No tags attached. |
Relationships | |
Attached Files | |
|
Issue History |
Date Modified | Username | Field | Change |
2017-07-21 23:53 | Zom-B | New Issue | |
2017-07-22 16:38 | Zalewa | Note Added: 0018075 | |
2017-07-29 20:01 | Zom-B | Note Added: 0018117 | |
2017-10-04 20:13 | WubTheCaptain | Note Added: 0018435 | |
2017-10-04 20:26 | WubTheCaptain | Note Added: 0018436 | |
2017-10-04 20:26 | WubTheCaptain | Note Edited: 0018436 | bug_revision_view_page.php?rev_id=11045 |
2017-10-04 20:33 | WubTheCaptain | Note Edited: 0018436 | bug_revision_view_page.php?rev_id=11046 |
2017-10-04 20:52 | WubTheCaptain | Note Deleted: 0018436 | |
2017-10-04 20:59 | WubTheCaptain | Note Added: 0018437 | |
2017-10-04 21:00 | WubTheCaptain | Note Edited: 0018437 | bug_revision_view_page.php?bugnote_id=18437#r11048 |
2017-10-04 21:01 | WubTheCaptain | Note Edited: 0018437 | bug_revision_view_page.php?bugnote_id=18437#r11049 |
2017-10-04 21:08 | WubTheCaptain | Status | new => acknowledged |
2017-10-04 21:09 | WubTheCaptain | Note Edited: 0018437 | bug_revision_view_page.php?bugnote_id=18437#r11050 |
2017-10-04 21:10 | WubTheCaptain | Note Edited: 0018437 | bug_revision_view_page.php?bugnote_id=18437#r11051 |
2017-10-04 21:10 | WubTheCaptain | Product Version | => 1.1 |
2017-10-04 21:31 | WubTheCaptain | Note Edited: 0018437 | bug_revision_view_page.php?bugnote_id=18437#r11052 |
2017-10-04 21:58 | WubTheCaptain | Note Added: 0018438 | |
2017-10-05 00:36 | Blzut3 | Note Added: 0018442 | |
2018-02-19 22:20 | WubTheCaptain | Status | acknowledged => confirmed |
2018-02-19 22:21 | WubTheCaptain | Summary | Server owner does not show up in buddies list => Buddies: "\c[" is not a "valid" standard color sequence (was: Server owner does not show up in buddies list) |
2018-02-19 22:24 | WubTheCaptain | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=11411#r11411 |
2018-02-19 22:24 | WubTheCaptain | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=11412#r11412 |
2018-02-19 22:24 | WubTheCaptain | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=11413#r11413 |
2018-02-19 22:25 | WubTheCaptain | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=11414#r11414 |
2018-02-19 22:26 | WubTheCaptain | Description Updated | bug_revision_view_page.php?rev_id=11416#r11416 |
2018-02-19 22:29 | WubTheCaptain | Description Updated | bug_revision_view_page.php?rev_id=11417#r11417 |
2018-02-20 09:29 | Zalewa | Note Added: 0019077 | |
2018-02-20 09:30 | Zalewa | Note Edited: 0019077 | bug_revision_view_page.php?bugnote_id=19077#r11421 |
2018-02-21 07:36 | Zalewa | Assigned To | => Zalewa |
2018-02-21 07:36 | Zalewa | Status | confirmed => assigned |
2018-02-21 07:40 | Zalewa | Additional Information Updated | bug_revision_view_page.php?rev_id=11428#r11428 |
2018-03-03 20:41 | Zalewa | Note Added: 0019116 | |
2018-03-03 20:41 | Zalewa | Status | assigned => needs testing |
2018-03-03 21:16 | WubTheCaptain | Note Added: 0019121 | |
2018-03-03 21:16 | WubTheCaptain | Status | needs testing => assigned |
2018-03-03 21:17 | WubTheCaptain | Note Edited: 0019121 | bug_revision_view_page.php?bugnote_id=19121#r11472 |
2018-03-03 21:20 | Zalewa | Note Added: 0019122 | |
2018-03-03 21:20 | Zalewa | Status | assigned => needs testing |
2018-03-03 21:47 | WubTheCaptain | Target Version | => 1.2 |
2018-03-04 10:53 | WubTheCaptain | Note Added: 0019131 | |
2018-03-04 11:31 | WubTheCaptain | Note Added: 0019132 | |
2018-03-04 11:40 | WubTheCaptain | Note Added: 0019133 | |
2018-03-04 11:41 | WubTheCaptain | Status | needs testing => resolved |
2018-03-04 11:41 | WubTheCaptain | Fixed in Version | => 1.2 |
2018-03-04 11:41 | WubTheCaptain | Resolution | open => fixed |
2018-03-04 12:23 | WubTheCaptain | Note Added: 0019134 | |
2018-10-27 22:55 | WubTheCaptain | Status | resolved => closed |
Notes |
|
(0018075)
|
Zalewa
|
2017-07-22 16:38
|
|
Doomseeker doesn't know if player is local or not (in fact, all players in Zandronum are technically remote, because the servers can only be run in a dedicated mode).
Maybe "Johnny Blaze" is using a hidden character in their nickname? Something that is there but isn't printed on screen.
Try adding this player in "Basic" mode as "*Johnny*Blaze*". |
|
|
(0018117)
|
Zom-B
|
2017-07-29 20:01
|
|
He shows up as "b1]Johnny Blaze". It's not stripping this extended color code quite right for some reason. |
|
|
|
I'm unable to reproduce. I tried names with colors, spaces, basic wildcard matching, they all show up in the buddy list as intended.
Only thing I noticed was at some point Doomseeker's buddy list for names I had added just reset. This happens sometimes when a server started from Doomseeker is open and Doomseeker is restarted. |
|
|
(0018437)
|
WubTheCaptain
|
2017-10-04 20:59
(edited on: 2017-10-04 21:31) |
|
Okay, I think I understand what's going on in here. It's an escape sequence/third-party issue. This affects extended colors from third-party "NewTextColors_260.pk3", but not regular color sequences.
Ingame, I had my name set to "b1]Wub". Just those characters, no colors. "name b1]Wub" in console. Attempting to match "b1]*" in Doomseeker finds no results. Attempting to match "*Wub" finds it. An escaped sequence "b1\]*" also finds it.
With standard colors: "name \cfWub" in console. This is displayed as "Wub" with a color in the "server details" dialog in Doomseeker, but as "Wub" in the buddies dialog. It uses a standard color ingame. "Wub" matches this in buddy list.
With extended colors from NewTextColors_260.pk3: "name \c[b1]Wub" in console. This is displayed as "Wub" with no colors in the "server details" dialog in Doomseeker, but as "b1]Wub" in buddies dialog. It uses an extended color ingame. "Wub" does not match this in buddy list, but "*Wub" does. "b1]Wub" or "b1\]Wub" does not match. "\[b1\]Wub" matches.
I also tried using a non-breaking space (" ") as a "hidden character". To my awareness, it's treated the same as a regular space character (like case-insensitive).
In other words: Because "\c[" is not a "valid" standard color sequence ("[" is not a recognized color), it will be discarded and you get this result. Special characters must also be escaped with a backwards slash ("\") or matched with a wildcard asterisk ("*").
Tested with Zandronum 3.0 and Doomseeker 1.1.
|
|
|
|
Personally, I also think Doomseeker to be working as intended. If game data (WAD) authors want to break existing conventions, that's their fault. I admit I'd like to have a color sequence standardized for custom colors, but until then I wouldn't make a hacked up patch to this issue. The correct way to handle those broken convention cases is to match "*Wub".
I've set this to acknowledged, but not confirmed. I'll leave it up to the developers (Zalewa and/or Blzut3) to decide what to do with this. |
|
|
(0018442)
|
Blzut3
|
2017-10-05 00:36
|
|
The "\c[*]" is a valid color sequence for a long time now, so if that's what's tripping things up then this is a bug. |
|
|
(0019077)
|
Zalewa
|
2018-02-20 09:29
(edited on: 2018-02-20 09:30) |
|
Wub, thanks for clarifying this further. I'll have a look at this sometime this week.
|
|
|
(0019116)
|
Zalewa
|
2018-03-03 20:41
|
|
|
|
(0019121)
|
WubTheCaptain
|
2018-03-03 21:16
(edited on: 2018-03-03 21:17) |
|
Quote from src/core/serverapi/player.cpp:L101
// Lets only remove the following character on \c.
This old comment is now outdated (and slightly misleading). Please update it.
Quote from src/core/serverapi/player.cpp:L105
int colorCodeIdx = i + 1;
Could condense this to int colorCodeIdx = i++, but meh.
|
|
|
(0019122)
|
Zalewa
|
2018-03-03 21:20
|
|
|
|
|
Rant: This one is quite difficult to test, because the buddies list is not updated every time the specific server is refreshed, a new nickname is added or deleted. I get confused a lot (like previously in this ticket) when something matches, then the other time it seemingly doesn't. Cornercases are difficult or time consuming to test because of this seperate issue. As a workaround I sometimes resort to restarting Doomseeker often, but that may also lead to master server response throttling.
"name \c[b1]Wub" ingame matches to "Wub" in Doomseeker buddies now in 1.2. |
|
|
|
Doomseeker 1.1 and 1.2~beta-1 both discard brackets in basic pattern type silently.
"name b1]Wub" ingame has these basic search results:
- The name is displayed as b1]Wub in 1.1 and 1.2 in Doomseeker buddies list.
- b1* matches in 1.1 and 1.2.
- *Wub matches in 1.1 and 1.2.
- Wub doesn't match in 1.1 and 1.2.
- b1\]Wub doesn't match with basic pattern type, but matches with advanced pattern type in 1.1 and 1.2.
- b1]Wub doesn't match and is not saved to configuration, thus lost before next launch of Doomseeker. (Rejected by advanced pattern type as invalid regex, but not rejected by basic.)
"name \c[b1]Wubeeeeeeeeeeeeeeeeeeeeeee" is the max length I could do.
]name b1]Wubeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee
b1]Wub is now known as b1]Wubeeeeeeeeeeeeeeeeeeeeeee
(This is misleading: The \c[ part was stripped from stdout by Zandronum?)
- The name is displayed as b1]Wubeeeeeeeeeeeeeeeeeeeeeee in 1.1 and as Wubeeeeeeeeeeeeeeeeeeeeeee in 1.2 in Doomseeker buddies list.
- b1* matches in 1.1, doesn't match in 1.2.
- *Wubeeeeeeeeeeeeeeeeeeeeeee matches in 1.1 and 1.2.
- Wubeeeeeeeeeeeeeeeeeeeeeee doesn't match in 1.1, matches in 1.2.
- b1\]Wubeeeeeeeeeeeeeeeeeeeeeee matches with 1.1, and doesn't match in 1.2 with advanced pattern type.
- b1]Wubeeeeeeeeeeeeeeeeeeeeeee is again silently dropped in 1.1 and 1.2, and rejected by advanced pattern type as invalid regex.
I think this is resolved. |
|
|
|
Also worth noting something garbage like "name \c\c[]Wub" will result in ingame name of "[]Wub", which the basic pattern type will never find with "[]Wub" (due to "[]Wub" still showing in buddies match list, but silently discarded on next launch) but advanced pattern type "\[\]Wub" will match. Anyway, this is a seperate concern and not necessarily an issue... |
|
|
|
Note: Confusion in this ticket was created mainly by issues 0003398 and 0003399. |
|