[Swift] Fix unwrapRequired behavior so it actually works (!1527) · Merge requests · OpenAPI Tools / openapi-generator

Closed Administrator requested to merge github/fork/joeboyscout04/issue-228 into master Nov 22, 2018

Created by: joeboyscout04

PR checklist

Read the contribution guidelines.
Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{LANG}-petstore.sh if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in .\bin\windows\.
Filed the PR against the correct branch: master, 3.4.x, 4.0.x. Default: master.
Copied the technical committee to review the pull request if your PR is targeting a particular programming language. @clasnake @jimschubert @shijinkui @ramzimaalej

Description of the PR

Previously the unwrapRequired config option didn't exist for swift4 generator (see #228 (closed)), but the previous PR (#229) didn't fully fix the issue due to some faulty logic and a missing default value. That has now been corrected so this option should work again.