Merge pull request #76 from advanced-rest-client/fix/W-13536254/required-fields

[W-13536254] Required fields

Author: carowright

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "@api-components/api-type-document",
   "description": "A documentation table for type (resource) properties. Works with AMF data model",
-  "version": "4.2.26",
+  "version": "4.2.27",
   "license": "Apache-2.0",
   "main": "index.js",
   "module": "index.js",

package.json

@@ -390,6 +390,13 @@ export class PropertyShapeDocument extends PropertyDocumentMixin(LitElement) {
     return !!parentTypeName;
   }
 
+  _toBoolean(value) {
+    if (typeof value === 'boolean') {
+      return value
+    }
+    return value === 'true'
+  }
+
   /**
    * Computes value for `isRequired` property.
    * In AMF model a property is required when `http://www.w3.org/ns/shacl#minCount`
@@ -404,10 +411,11 @@ export class PropertyShapeDocument extends PropertyDocumentMixin(LitElement) {
     }
     shape = this._resolve(shape);
     if (this._hasType(shape, this.ns.aml.vocabularies.apiContract.Parameter)) {
-      return /** @type boolean */ Boolean(this._getValue(
+      const required = this._getValue(
         shape,
         this.ns.aml.vocabularies.apiContract.required
-      ));
+      )
+      return this._toBoolean(required);
     }
     const data = this._getValue(shape, this.ns.w3.shacl.minCount);
     return data !== undefined && Number(data) !== 0;