Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[orchestra2md] invalid handling of more than one appinfo #66

Open
donmendelson opened this issue May 3, 2022 · 3 comments
Open

[orchestra2md] invalid handling of more than one appinfo #66

donmendelson opened this issue May 3, 2022 · 3 comments
Assignees
Labels
bug Something isn't working fixed Issue has been resolved, awaits testing

Comments

@donmendelson
Copy link
Member

Reported by Hanno Klein:
XML output generated by md2orchestra:

        <fixr:group id="2071" name="SecAltIDGrp" scenario="EXXXXX">
            <fixr:numInGroup id="454"/>
            <fixr:fieldRef id="455">
              <fixr:annotation>
                <fixr:appinfo purpose="P1">Hanno</fixr:appinfo>
                <fixr:appinfo purpose="P2">Klein</fixr:appinfo>
              </fixr:annotation>
            </fixr:fieldRef>
            <fixr:fieldRef id="456" scenario="EXXXXX"/>
        </fixr:group>

md output generated by orchestra2md:

### Group SecAltIDGrp scenario EXXXXX (2071)
| Name                | Tag | Scenario | Presence | P1    |
|---------------------|----:|----------|----------|-------|
| NoSecurityAltID     | 454 |          | optional |       |
| SecurityAltID       | 455 |          | optional | Klein |
| SecurityAltIDSource | 456 | EXXXXX   | optional |       |

Expected separate columns for P1 and P2.

@donmendelson
Copy link
Member Author

For documentation, some headings are considered equivalent so they are combined, e.g. "documentation" and "description". Furthermore, they are ranked so synopsis always comes before elaboration, etc. Anything with unknown purpose is given the lowest rank. However, that logic does not apply to appinfo; every purpose must be considered unique, and there is no preset ranking.

@donmendelson
Copy link
Member Author

Setting sequence so all appinfo follows documentation is their set order by purpose.

@donmendelson
Copy link
Member Author

Corrected appinfo columns:

| Tag | Scenario | Presence | P1      | P2      |
|----:|----------|----------|---------|---------|
| 454 |          | optional |         |         |
| 455 |          | optional | [Hanno] | [Klein] |
| 456 | EXXXXX   | optional |         |         |

@donmendelson donmendelson added the bug Something isn't working label May 3, 2022
@kleihan kleihan added the fixed Issue has been resolved, awaits testing label Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed Issue has been resolved, awaits testing
Projects
Status: No status
Development

No branches or pull requests

2 participants