Volian/SourceCode
8
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

B2026-028_Selecting_to_Go_To_on_an_RO_causes_PROMS_to_crash #730

Merged
jjenko merged 3 commits from B2026-028_Selecting_to_Go_To_on_an_RO_causes_PROMS_to_crash into Development 2026-03-09 14:41:04 -04:00
Conversation 4 Commits 3 Files Changed 1 +2 -2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 95737d05e1 - Show all commits

4
PROMS/Volian.Controls.Library/DisplayRO.cs
View File

@@ -379,8 +379,8 @@ namespace Volian.Controls.Library
if (chld.value != null)
{
chld.value = Regex.Replace(chld.value, @"\\u([0-9]{1,4})\?", m => Convert.ToChar(int.Parse(m.Groups[1].Value)).ToString());
RoUsageInfo SavROLink = null;
chld.value = Regex.Replace(chld.value, @"\\u([0-9]{1,4})\?", m => int.TryParse(m?.Groups[1]?.Value, out int result) ? Convert.ToChar(result).ToString() : "");
mschill marked this conversation as resolved Outdated
mschill commented 2026-03-09 07:16:42 -04:00
Outdated
Review
Copy Link

Could there be an exception here with the int.Parse?

Should this be something like:

chld.value = Regex.Replace(chld.value, @"\u([0-9]{1,4})?", m => int.TryParse(m?.Groups[1]?.Value, out int result) ? Convert.ToChar(result).ToString() : "");

Could there be an exception here with the int.Parse? Should this be something like: chld.value = Regex.Replace(chld.value, @"\\u([0-9]{1,4})\?", m => int.TryParse(m?.Groups[1]?.Value, out int result) ? Convert.ToChar(result).ToString() : "");
jjenko commented 2026-03-09 08:27:44 -04:00
Outdated
Review
Copy Link

I agree. It would be safter to do the TryParse as suggested

I agree. It would be safter to do the TryParse as suggested
plarsen commented 2026-03-09 08:49:08 -04:00
Outdated
Review
Copy Link

I can add the tryparse, but since the match is only looking for integers [0-9]{1,4} why would the tryparese be needed?

I can add the tryparse, but since the match is only looking for integers [0-9]{1,4} why would the tryparese be needed?
RoUsageInfo SavROLink = null;
if (_savCurROLink != null) SavROLink = _savCurROLink;
// Set the Display Text to the AccPageID