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

B2026 053 find unlinked values of selected #795

Merged
mschill merged 28 commits from B2026-053-Find-Unlinked-Values-of-Selected into Development 2026-07-30 15:29:20 -04:00
Conversation 10 Commits 28 Files Changed 1
+18 -29
1 changed files with 18 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit cfc324d31e - Show all commits
PROMS/Volian.Controls.Library/DisplaySearch.cs
+18 -29
View File
@@ -2016,20 +2016,17 @@ namespace Volian.Controls.Library
{ {
ReportTitle = string.Format("Search for '{0}'", TextSearchString); ReportTitle = string.Format("Search for '{0}'", TextSearchString);
// C2020-009: Search - Allow search 'By Word'. string byWordPrefix = string.Empty;
string byWordSuffix = string.Empty;
if (cbxByWord.Checked)
mschill commented 2026-07-30 14:05:49 -04:00
Outdated
Review
Copy Link
Copy Source

Was this if intentionally commented out? -- just checking since doesn't it change the logic around this other search --- for testing would make sure both searches tested with the different options.

Was this if intentionally commented out? -- just checking since doesn't it change the logic around this other search --- for testing would make sure both searches tested with the different options.
plarsen commented 2026-07-30 14:29:58 -04:00
Outdated
Review
Copy Link
Copy Source

Thanks, that code needs to be there and account of that I need to change the logic for updating the pre and post additions.

Thanks, that code needs to be there and account of that I need to change the logic for updating the pre and post additions.
{
byWordPrefix = GetByWordSettings(TextSearchString, true);
jjenko marked this conversation as resolved Outdated
jjenko commented 2026-07-30 14:01:30 -04:00
Outdated
Review
Copy Link
Copy Source

Please move this descriptive comment (the stuff inside the curly braces) into the function you had created.

Then remove the rest of the stuff you had commented out.

Please move this descriptive comment (the stuff inside the curly braces) into the function you had created. Then remove the rest of the stuff you had commented out.
👍 1
byWordSuffix = GetByWordSettings(TextSearchString, false);
}
//if (cbxByWord.Checked) SearchString = TextSearchString;
//{
// // Generate a prefix & suffix to be used in the sql query around the search string.
// // If the search string starts (prefix)/ends(suffix) with a number, then use an expression that
// // does not allow the preceding/following text to have a number, '.', letter or other rtf
// // commands. If the search string starts/ends with a letter, then use an expression that does not
// // find the preceding/following text that is text, i.e. a letter.
SearchString = TextSearchString; SearchResults = ItemInfoList.GetListFromTextSearch(DVISearchList, TypeSearchList, TextSearchString /*.Replace(@"\",@"\u9586?")*/, cbxBooleanTxtSrch.Checked ? 2 : cbxCaseSensitive.Checked ? 1 : 0, cbxProcSectSrch.Checked ? 1 : 0, cbxIncROTextSrch.Checked ? ItemSearchIncludeLinks.Value : ItemSearchIncludeLinks.Nothing, includeRTFformat, includeSpecialChars, unitPrefix, byWordPrefix, byWordSuffix);
// B2022-031 - added a cbxProcSectSrch to filter out procedure and section titles from global search results.
SearchResults = ItemInfoList.GetListFromTextSearch(DVISearchList, TypeSearchList, TextSearchString /*.Replace(@"\",@"\u9586?")*/, cbxBooleanTxtSrch.Checked ? 2 : cbxCaseSensitive.Checked ? 1 : 0, cbxProcSectSrch.Checked ? 1 : 0, cbxIncROTextSrch.Checked ? ItemSearchIncludeLinks.Value : ItemSearchIncludeLinks.Nothing, includeRTFformat, includeSpecialChars, unitPrefix, GetByWordSettings(TextSearchString, true), GetByWordSettings(TextSearchString, false));
cmbResultsStyleIndex = 3; // display step text in results cmbResultsStyleIndex = 3; // display step text in results
//} //}
3
 @@ -2175,23 +2172,15 @@ namespace Volian.Controls.Library
} }
private string GetByWordSettings(string ss, bool checkstart) private string GetByWordSettings(string ss, bool checkstart)
{ {
//if (Regex.IsMatch(ss, @"^[\d\.]")) // starts with a number or '.' decimal pt
//{ // C2020-009: Search - Allow search 'By Word'.
// settings.Add("byWordPrefix", @"[^0-9a-zA-Z.vbpi:\\-]");
//} // // Generate a prefix & suffix to be used in the sql query around the search string.
jjenko marked this conversation as resolved Outdated
jjenko commented 2026-07-30 14:02:15 -04:00
Outdated
Review
Copy Link
Copy Source

Please remove the code you had commented out.

Please remove the code you had commented out.
👍 1
//else if (Regex.IsMatch(ss, @"^[a-zA-Z]")) // starts with a letter // // If the search string starts (prefix)/ends(suffix) with a number, then use an expression that
//{ // // does not allow the preceding/following text to have a number, '.', letter or other rtf
// settings.Add("byWordPrefix", @"[^a-zA-Z]"); // // commands. If the search string starts/ends with a letter, then use an expression that does not
//} // // find the preceding/following text that is text, i.e. a letter.
//if (Regex.IsMatch(ss, @"^[\d\.]")) // ends with a number or decimal
//{
// settings.Add("byWordSuffix", @"[^0-9a-zA-Z.vbpi:\\-]");
//}
//else if (Regex.IsMatch(ss, @"[a-zA-Z]$")) // ends with a letter
//{
// settings.Add("byWordSuffix", @"[^a-zA-Z]");
//}
//return settings;
if (ss.Length > 0) if (ss.Length > 0)
{ {
mschill marked this conversation as resolved Outdated
mschill commented 2026-07-29 13:35:00 -04:00
Outdated
Review
Copy Link
Copy Source

Previous comments regarding Regexes still apply -- also it appears now we have 2 variables that we declared but did not use in string byWordPrefix & string byWordSuffix

Previous comments regarding Regexes still apply -- also it appears now we have 2 variables that we declared but did not use in string byWordPrefix & string byWordSuffix
plarsen commented 2026-07-29 13:54:28 -04:00
Outdated
Review
Copy Link
Copy Source

I removed the two unneeded variable declarations. I used what was previous used for the regex string. If you notice both tests are looking to see what the first character is and not the whole searchstring and from that it selects different regex strings. On account of this I am not sure char.IsNumeric will work because there are searchstrings that start with a number but are a mix of numbers and characters. Example 5% code: "if (Regex.IsMatch(ss, @"^[\d.]")) // starts with a number or '.' decimal pt"

I removed the two unneeded variable declarations. I used what was previous used for the regex string. If you notice both tests are looking to see what the first character is and not the whole searchstring and from that it selects different regex strings. On account of this I am not sure char.IsNumeric will work because there are searchstrings that start with a number but are a mix of numbers and characters. Example 5% code: "if (Regex.IsMatch(ss, @"^[\d\.]")) // starts with a number or '.' decimal pt"
//checking start //checking start
8