MS Contactor improvements

[andrewlee94]

Fixes None

Summary/Motivation:

This PR fixes some bugs in the MS Contactor model

Changes proposed in this PR:

• Adds check to only build energy transfer terms if energy balances are being used.
• Fixes issue with closures in rules similar to Function Scoping Fixes #1242

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

1. I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

[codecov]

Codecov Report

Patch coverage: 88.49% and no project coverage change.

Comparison is base (b128971) 76.83% compared to head (863a658) 76.84%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1243   +/-   ##
=======================================
  Coverage   76.83%   76.84%           
=======================================
  Files         390      390           
  Lines       61942    61963   +21     
  Branches    11398    11398           
=======================================
+ Hits        47595    47613   +18     
- Misses      11878    11882    +4     
+ Partials     2469     2468    -1     

Files Changed	Coverage Δ
idaes/models/unit_models/mscontactor.py	91.20% <88.49%> (+0.53%)	⬆️

... and 1 file with indirect coverage changes

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

[bpaul4]

Looks fine to me. The pylint issue seems to be a leftover phase_list variable on line 597 that is redefined in the new methods so it's not used outside them, taking out that line should make pylint happy.