fix: Fix misuse of CommID

[rrsettgast]

At some point in the past a developer who will remain unnamed made it so that MPI_iCommData could be passed a CommID which was copied in MPI_iCommData. This allows for the case where the CommID is destroyed, and the MPI_iCommData would refer to a value of CommID that has been put back into the set of available CommID. Recall that no two communications can use the same CommID. This PR makes it a one-to-one relationship by grabbing the CommID inside of the constructor of MPI_iCommData as was the case before the unnamed developer changed it. Don't go looking for the culprit....I assure you he is quite remorseful for his blunder 😁

[codecov]

Codecov Report

Attention: Patch coverage is 52.63158% with 9 lines in your changes missing coverage. Please review.

Project coverage is 56.47%. Comparing base (968476f) to head (01deaf0).
Report is 1 commits behind head on develop.

Files with missing lines	Patch %	Lines
...hysicsSolvers/solidMechanics/SolidMechanicsMPM.cpp	0.00%	3 Missing ⚠️
...ration/EmbeddedSurfacesParallelSynchronization.cpp	0.00%	3 Missing ⚠️
...lvers/surfaceGeneration/ParallelTopologyChange.cpp	0.00%	2 Missing ⚠️
...ers/solidMechanics/SolidMechanicsLagrangianFEM.cpp	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3402      +/-   ##
===========================================
- Coverage    56.48%   56.47%   -0.01%     
===========================================
  Files         1074     1074              
  Lines        90502    90504       +2     
===========================================
  Hits         51116    51116              
- Misses       39386    39388       +2     

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