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

Instance validator NPE on itemtype #886

Merged
merged 4 commits into from May 24, 2018

Conversation

Projects
None yet
3 participants
@hdconradi

hdconradi commented Mar 13, 2018

When a questionnaire choice (or openchoice) item has an embedded item a NPE can be triggered.

Heinz-Dieter Conradi added some commits Mar 12, 2018

Heinz-Dieter Conradi Heinz-Dieter Conradi
Test questionnaire with embedded items for DSTU3.
The test for an item embedded in a text item works as expected.
However, the embedded item in a choice or openchoice case
triggers an NPE.
Heinz-Dieter Conradi Heinz-Dieter Conradi
Test questionnaire with embedded items for R4.
    The test for an item embedded in a text item works as expected.
    However, the embedded item in a choice or openchoice case
    triggers an NPE.
@hdconradi

This comment has been minimized.

hdconradi commented Apr 3, 2018

Rebased after 3.3

@coveralls

This comment has been minimized.

coveralls commented Apr 3, 2018

Coverage Status

Coverage increased (+0.02%) to 72.533% when pulling baf53ef on hdconradi:instance-validator-npe-on-itemtype into f766156 on jamesagnew:master.

@jamesagnew jamesagnew merged commit 6d8ebe2 into jamesagnew:master May 24, 2018

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

jamesagnew added a commit that referenced this pull request May 24, 2018

@hdconradi hdconradi deleted the hdconradi:instance-validator-npe-on-itemtype branch Jun 5, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment