You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ossie.apache.org by "christianeu-db (via GitHub)" <gi...@apache.org> on 2026/09/18 20:15:49 UTC
[I] wisdom: ONE_TO_MANY relationships lose direction on round-trip [ossie]
christianeu-db opened a new issue, #425:
URL: https://github.com/apache/ossie/issues/425
### Summary
A Wisdom `ONE_TO_MANY` relationship round-trips (Wisdom->Ossie->Wisdom) to `MANY_TO_ONE`. The forward path sets no `ai_context` marker for `ONE_TO_MANY` (unlike ONE_TO_ONE / MANY_TO_MANY), and the reverse path defaults unmarked relationships to `MANY_TO_ONE`.
### Root cause
Forward `wisdom_to_ossie.py`: the `ONE_TO_MANY` branch swaps from/to but leaves `ai_context=None`. Reverse `ossie_to_wisdom.py`: only `one-to-one` / `many-to-many` markers are recognized; default is `MANY_TO_ONE`.
### Repro (real converters, HEAD ab8fc1a — EXECUTED)
Using fixture `tests/fixtures/sample_export.json` through both converters:
```
original wisdom relationshipTypes: ['MANY_TO_ONE','ONE_TO_MANY','MANY_TO_MANY','MANY_TO_ONE','MANY_TO_ONE']
after Wisdom->Ossie->Wisdom: ['MANY_TO_ONE','MANY_TO_ONE','MANY_TO_MANY','MANY_TO_ONE']
```
The `ONE_TO_MANY` at index 1 becomes `MANY_TO_ONE`. (Also note the relationship count dropped 5->4 in this fixture run — worth a look, but out of scope for this issue.) The wrong result is also enshrined in `tests/test_ossie_to_wisdom.py::test_relationship_types_restored`.
### Potential fix
Emit `ai_context = "one-to-many relationship"` in the forward path; add an `elif ...startswith("one-to-many")` branch in reverse; update the test expectation.
### Scope / caveats
ONE_TO_ONE / MANY_TO_MANY unaffected. From/to swap is correct; only the semantic label is lost. Fixing requires the test update.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ossie.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] wisdom: ONE_TO_MANY relationships lose direction on round-trip [ossie]
Posted by "christianeu-db (via GitHub)" <gi...@apache.org>.
christianeu-db commented on issue #425:
URL: https://github.com/apache/ossie/issues/425#issuecomment-5735667279
cc @mwu-wisdom if you want take a first look
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ossie.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] wisdom: ONE_TO_MANY relationships lose direction on round-trip [ossie]
Posted by "tripleaceme (via GitHub)" <gi...@apache.org>.
tripleaceme commented on issue #425:
URL: https://github.com/apache/ossie/issues/425#issuecomment-5803940575
@mwu-wisdom are you planning to pick this up? If not, I'm happy to take it.
My plan would follow the fix in the description: set `ai_context = "one-to-many relationship"` in the `ONE_TO_MANY` branch of `wisdom_to_ossie.py`, add the matching `one-to-many` branch in `ossie_to_wisdom.py`, and correct the expectation in `test_relationship_types_restored`. I'll also add a round-trip test on `sample_export.json` that checks every relationship type survives.
I'll look into the 5 → 4 relationship count drop separately and open its own issue if it turns out to be a real bug.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ossie.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] wisdom: ONE_TO_MANY relationships lose direction on round-trip [ossie]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre closed issue #425: wisdom: ONE_TO_MANY relationships lose direction on round-trip
URL: https://github.com/apache/ossie/issues/425
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@ossie.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org