Clean up backend Word/Sense models

[imnasnainaec]

This change is 

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (48cedae) 72.32% compared to head (62c945c) 72.29%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2867      +/-   ##
==========================================
- Coverage   72.32%   72.29%   -0.04%     
==========================================
  Files         257      257              
  Lines        9521     9524       +3     
  Branches     1097     1096       -1     
==========================================
- Hits         6886     6885       -1     
- Misses       2308     2311       +3     
- Partials      327      328       +1     

Flag	Coverage Δ
backend	82.86% <100.00%> (-0.08%)	⬇️
frontend	62.58% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jmgrady]

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @imnasnainaec)

---

Backend/Models/Sense.cs line 134 at r1 (raw file):

        /// <summary> Adds all semantic domains from other Sense. </summary>
        public int AddNewDomains(Sense other, string userId)

CopyDomains would be a better name. AddNewDomains suggests (to me) that I am providing a list of domains to be added - unrelated to another sense.

Code quote:

AddNewDomains

---

Backend/Models/Sense.cs line 134 at r1 (raw file):

        /// <summary> Adds all semantic domains from other Sense. </summary>
        public int AddNewDomains(Sense other, string userId)

Method should not return int. It is never used.

Code quote:

int

[imnasnainaec]

Reviewable status: 1 of 4 files reviewed, 1 unresolved discussion (waiting on @jmgrady)

---

Backend/Models/Sense.cs line 134 at r1 (raw file):

Previously, jmgrady (Jim Grady) wrote…

Method should not return int. It is never used.

Done.

[jmgrady]

Reviewed 3 of 3 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @imnasnainaec)