Fix management edge mapping wrapper

eclipse-capella / capella

Open Source Solution for Model-Based Systems Engineering

https://mbse-capella.org

Eclipse Public License 2.0

222 stars 91 forks source link

Fix management edge mapping wrapper #2848

Closed SteveMonnier closed 3 months ago

sonarcloud[bot] commented 3 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

mPorhel commented 3 months ago

@pdulth @SteveMonnier @ebausson-obeo this seems like an automatic rebased PR, but the behavior has already been corrected and fixed by @pdulth in commit https://github.com/eclipse-capella/capella/commit/165318f5bd9da65faa62a395ebb60746a36aeeea

This PR must not been checked. It would revert several changes making the code check from DiagramServices.isMapping back to mapping equality. @SteveMonnier can you abort/cancel your PR ?

SteveMonnier commented 3 months ago

Hi. My bad, I am not sure why but when I migrated to Capella 7.0 the sample for the validation scenario it somehow broke it. I migrated it again and it works with @pdulth commits. This PR can be abandonned.