Skip to content

Fix LT-22354: FW crashes when deleting a category#707

Merged
jasonleenaylor merged 2 commits intomainfrom
LT-22354
Feb 27, 2026
Merged

Fix LT-22354: FW crashes when deleting a category#707
jasonleenaylor merged 2 commits intomainfrom
LT-22354

Conversation

@jtmaxwell3
Copy link
Collaborator

@jtmaxwell3 jtmaxwell3 commented Feb 25, 2026

This fixes https://jira.sil.org/browse/LT-22354.


This change is Reviewable

@github-actions
Copy link

github-actions bot commented Feb 25, 2026

NUnit Tests

    1 files  ±0      1 suites  ±0   5m 24s ⏱️ - 1m 8s
4 407 tests ±0  4 320 ✅ ±0  87 💤 ±0  0 ❌ ±0 
4 416 runs  ±0  4 329 ✅ ±0  87 💤 ±0  0 ❌ ±0 

Results for commit cee031c. ± Comparison against base commit cb374e2.

♻️ This comment has been updated with latest results.

@jasonleenaylor
Copy link
Contributor

Src/xWorks/DTMenuHandler.cs line 656 at r1 (raw file):

			if (template.Owner == selectedPOS)
			{
				return;

A comment here would be nice.

Copy link
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@jasonleenaylor reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on jtmaxwell3).

@jasonleenaylor jasonleenaylor merged commit aa45c37 into main Feb 27, 2026
8 of 9 checks passed
@jasonleenaylor jasonleenaylor deleted the LT-22354 branch February 27, 2026 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants