diff --git a/languages/c/Types.mjs b/languages/c/Types.mjs index 72912930..e9e922e8 100644 --- a/languages/c/Types.mjs +++ b/languages/c/Types.mjs @@ -223,13 +223,13 @@ function getSchemaType(schema, module, { name, prefix = '', destination, resultS return info.type } -function getSchemaTypeInfo(module = {}, json = {}, name = '', schemas = {}, prefix = '', options = {level: 0, descriptions: true, title: false, resultSchema: false, event: false}) { +function getSchemaTypeInfo(module = {}, json = {}, name = '', schemas = {}, prefix = '', options = { level: 0, descriptions: true, title: false, resultSchema: false, event: false}) { if (json.schema) { json = json.schema } - let stringAsHandle = options.resultSchema || options.event + let fireboltString = options.resultSchema || options.event let structure = {} structure["type"] = '' @@ -257,7 +257,7 @@ function getSchemaTypeInfo(module = {}, json = {}, name = '', schemas = {}, pref } } else if (json.const) { - structure.type = getNativeType(json, stringAsHandle) + structure.type = getNativeType(json, fireboltString) structure.json = json return structure } @@ -314,7 +314,7 @@ function getSchemaTypeInfo(module = {}, json = {}, name = '', schemas = {}, pref return getSchemaTypeInfo(module, union, '', schemas, '', options) } else if (json.oneOf) { - structure.type = 'char*' + structure.type = fireboltString ? getFireboltStringType() : 'char*' structure.json.type = 'string' return structure } @@ -331,7 +331,7 @@ function getSchemaTypeInfo(module = {}, json = {}, name = '', schemas = {}, pref structure.namespace = (json.namespace ? json.namespace : getModuleName(module)) } else { - structure.type = 'char*' + structure.type = fireboltString ? getFireboltStringType() : 'char*' } if (name) { structure.name = capitalize(name) @@ -340,7 +340,7 @@ function getSchemaTypeInfo(module = {}, json = {}, name = '', schemas = {}, pref return structure } else if (json.type) { - structure.type = getNativeType(json, stringAsHandle) + structure.type = getNativeType(json, fireboltString) structure.json = json if (name || json.title) { structure.name = capitalize(name || json.title) @@ -371,13 +371,12 @@ function getSchemaShapeInfo(json, module, schemas = {}, { name = '', prefix = '' if (json['$ref'][0] === '#') { //Ref points to local schema //Get Path to ref in this module and getSchemaType - const schema = getPath(json['$ref'], module, schemas) - const tname = schema.title || json['$ref'].split('/').pop() + let schema = getPath(json['$ref'], module, schemas) + const tName = schema.title || json['$ref'].split('/').pop() if (json['$ref'].includes('x-schemas')) { schema = (getRefModule(json['$ref'].split('/')[2])) } - - shape = getSchemaShapeInfo(schema, module, schemas, { name, prefix, merged, level, title, summary, descriptions, destination, section, enums }) + shape = getSchemaShapeInfo(schema, module, schemas, { name: tName, prefix, merged, level, title, summary, descriptions, destination, section, enums }) } } //If the schema is a const, @@ -423,7 +422,6 @@ function getSchemaShapeInfo(json, module, schemas = {}, { name = '', prefix = '' let info = getSchemaTypeInfo(module, items, items.name || pname, schemas, prefix, {level : level, descriptions: descriptions, title: true}) if (info.type && info.type.length > 0) { let objName = tName + '_' + capitalize(prop.title || pname) - let moduleName = info.namespace info.json.namespace = info.namespace let moduleProperty = getJsonTypeInfo(module, json, json.title || name, schemas, prefix) let prefixName = ((prefix.length > 0) && items['$ref']) ? '' : prefix @@ -533,10 +531,14 @@ function getSchemaShapeInfo(json, module, schemas = {}, { name = '', prefix = '' if (info.type && info.type.length > 0) { let type = getArrayElementSchema(json, module, schemas, info.name) - let arrayName = capitalize(name) + capitalize(type.type) - let objName = getTypeName(info.namespace, arrayName, prefix) + let arrayName = capitalize(info.name) + capitalize(type.type) + let namespace = info.namespace + if (type && type.type === 'object') { + namespace = getModuleName(module) + } + let objName = getTypeName(namespace, arrayName, prefix) let tName = objName + 'Array' - let moduleName = info.namespace + info.json.namespace = info.namespace let moduleProperty = getJsonTypeInfo(module, json, json.title || name, schemas, prefix) let subModuleProperty = getJsonTypeInfo(module, j, j.title, schemas, prefix) @@ -706,6 +708,9 @@ function getJsonTypeInfo(module = {}, json = {}, name = '', schemas, prefix = '' structure.type = getJsonNativeType(json) return structure } + else { + structure.type = 'JsonObject' + } return structure } diff --git a/languages/c/src/types/ImplHelpers.mjs b/languages/c/src/types/ImplHelpers.mjs index 6509375a..8bd950c2 100644 --- a/languages/c/src/types/ImplHelpers.mjs +++ b/languages/c/src/types/ImplHelpers.mjs @@ -463,7 +463,7 @@ function getResultInstantiation (name, nativeType, container, indentLevel = 3) { impl += `${' '.repeat(indentLevel)}*resultPtr = WPEFramework::Core::ProxyType<${container}>::Create();\n` impl += `${' '.repeat(indentLevel)}*(*resultPtr) = jsonResult;\n` impl += `${' '.repeat(indentLevel)}*${name} = static_cast<${nativeType}>(resultPtr);` - } else { + } else if (nativeType) { impl += `${' '.repeat(indentLevel)}*${name} = jsonResult.Value();` } diff --git a/languages/c/src/types/NativeHelpers.mjs b/languages/c/src/types/NativeHelpers.mjs index 022437e8..1460e22e 100644 --- a/languages/c/src/types/NativeHelpers.mjs +++ b/languages/c/src/types/NativeHelpers.mjs @@ -115,12 +115,12 @@ const getArrayElementSchema = (json, module, schemas = {}, name) => { return result } -const getNativeType = (json, stringAsHandle = false) => { +const getNativeType = (json, fireboltString = false) => { let type let jsonType = json.const ? typeof json.const : json.type if (jsonType === 'string') { type = 'char*' - if (stringAsHandle) { + if (fireboltString) { type = getFireboltStringType() } } diff --git a/languages/c/templates/codeblocks/setter.c b/languages/c/templates/codeblocks/setter.c index 9f496543..342a98d6 100644 --- a/languages/c/templates/codeblocks/setter.c +++ b/languages/c/templates/codeblocks/setter.c @@ -1,9 +1,7 @@ /* ${method.name} - ${method.description} */ -uint32_t ${info.title}_${method.Name}( ${method.signature.params} ) +uint32_t ${info.Title}_${method.Name}( ${method.signature.params} ) { const string method = _T("${info.title}.${method.name}"); - ${if.params} -${method.params.serialization} - ${end.if.params} +${if.params}${method.params.serialization}${end.if.params} return FireboltSDK::Properties::Set(method, jsonParameters); } diff --git a/languages/c/templates/declarations/default.c b/languages/c/templates/declarations/default.c index e69de29b..ac4e7f78 100644 --- a/languages/c/templates/declarations/default.c +++ b/languages/c/templates/declarations/default.c @@ -0,0 +1,3 @@ +/* ${method.name} - ${method.description} +${method.params.annotations}${if.deprecated} * @deprecated ${method.deprecation}${end.if.deprecated} */ +uint32_t ${info.Title}_${method.Name}( ${method.signature.params}${if.result}${if.params}, ${end.if.params}${method.result.type}* ${method.result.name}${end.if.result}${if.signature.empty}void${end.if.signature.empty} ); diff --git a/languages/c/templates/methods/default.c b/languages/c/templates/methods/default.c index 6a023534..6fbca47e 100644 --- a/languages/c/templates/methods/default.c +++ b/languages/c/templates/methods/default.c @@ -1,24 +1,21 @@ /* ${method.name} - ${method.description} */ -uint32_t ${info.title}_${method.Name}(${method.params.list}${if.params}, ${end.if.params}${method.result.type}* ${method.result.name}) { - uint32_t status = FireboltSDKErrorUnavailable; - FireboltSDK::Transport* transport = FireboltSDK::Accessor::Instance().GetTransport(); - if (transport != nullptr) { - - JsonObject jsonParameters; +uint32_t ${info.Title}_${method.Name}( ${method.signature.params}${if.result}${if.params}, ${end.if.params}${method.result.type}* ${method.result.name}${end.if.result}${if.signature.empty}void${end.if.signature.empty} ) { - ${if.params} -${method.params.json} - ${end.if.params} + uint32_t status = FireboltSDKErrorUnavailable; + FireboltSDK::Transport* transport = FireboltSDK::Accessor::Instance().GetTransport(); + if (transport != nullptr) { - WPEFramework::Core::JSON::Boolean jsonResult; - status = transport->Invoke("${info.title}.${method.name}", jsonParameters, jsonResult); - if (status == FireboltSDKErrorNone) { - *success = jsonResult.Value(); - } + ${method.params.serialization.with.indent} + ${method.result.json.type} jsonResult; + status = transport->Invoke("${info.title}.${method.name}", jsonParameters, jsonResult); + if (status == FireboltSDKErrorNone) { + FIREBOLT_LOG_INFO(FireboltSDK::Logger::Category::OpenRPC, FireboltSDK::Logger::Module(), "${info.Title}.${method.name} is successfully invoked"); +${method.result.instantiation} + } - } else { - FIREBOLT_LOG_ERROR(FireboltSDK::Logger::Category::OpenRPC, FireboltSDK::Logger::Module(), "Error in getting Transport err = %d", status); - } + } else { + FIREBOLT_LOG_ERROR(FireboltSDK::Logger::Category::OpenRPC, FireboltSDK::Logger::Module(), "Error in getting Transport err = %d", status); + } - return status; + return status; } diff --git a/languages/c/templates/methods/property.c b/languages/c/templates/methods/property.c index c7510da9..9a92dce4 100644 --- a/languages/c/templates/methods/property.c +++ b/languages/c/templates/methods/property.c @@ -1,5 +1,5 @@ /* ${method.name} - ${method.description} */ -uint32_t ${info.title}_Get${method.Name}( ${method.signature.params}${if.params}, ${end.if.params}${method.result.type}* ${method.result.name} ) +uint32_t ${info.Title}_Get${method.Name}( ${method.signature.params}${if.params}, ${end.if.params}${method.result.type}* ${method.result.name} ) { const string method = _T("${info.title}.${method.name}"); ${if.params}${method.params.serialization}${end.if.params} diff --git a/src/macrofier/engine.mjs b/src/macrofier/engine.mjs index b167f4d4..ebe34460 100644 --- a/src/macrofier/engine.mjs +++ b/src/macrofier/engine.mjs @@ -1109,9 +1109,12 @@ function insertMethodMacros(template, methodObj, json, templates, examples = {}) .replace(/\$\{method\.params\.array\}/g, JSON.stringify(methodObj.params.map(p => p.name))) .replace(/\$\{method\.params\.count}/g, methodObj.params ? methodObj.params.length : 0) .replace(/\$\{if\.params\}(.*?)\$\{end\.if\.params\}/gms, method.params.length ? '$1' : '') - .replace(/\$\{if\.params.empty\}(.*?)\$\{end\.if\.params.empty\}/gms, method.params.length === 0 ? '$1' : '') + .replace(/\$\{if\.result\}(.*?)\$\{end\.if\.result\}/gms, resultType ? '$1' : '') + .replace(/\$\{if\.params\.empty\}(.*?)\$\{end\.if\.params\.empty\}/gms, method.params.length === 0 ? '$1' : '') + .replace(/\$\{if\.signature\.empty\}(.*?)\$\{end\.if\.signature\.empty\}/gms, (method.params.length === 0 && resultType === '') ? '$1' : '') .replace(/\$\{if\.context\}(.*?)\$\{end\.if\.context\}/gms, event && event.params.length ? '$1' : '') .replace(/\$\{method\.params\.serialization\}/g, serializedParams) + .replace(/\$\{method\.params\.serialization\.with\.indent\}/g, indent(serializedParams, ' ')) // Typed signature stuff .replace(/\$\{method\.signature\}/g, types.getMethodSignature(methodObj, json, { isInterface: false, destination: state.destination, section: state.section })) .replace(/\$\{method\.signature\.params\}/g, types.getMethodSignatureParams(methodObj, json, { destination: state.destination, section: state.section })) @@ -1129,7 +1132,7 @@ function insertMethodMacros(template, methodObj, json, templates, examples = {}) .replace(/\$\{event\.callback\.params\.serialization\}/g, callbackSerializedParams) .replace(/\$\{event\.callback\.result\.instantiation\}/g, callbackResultInst) .replace(/\$\{event\.callback\.response\.instantiation\}/g, callbackResponseInst) - .replace(/\$\{info\.title\}/g, info.title) + .replace(/\$\{info\.title\}/g, info.title.toLowerCase()) .replace(/\$\{info\.Title\}/g, capitalize(info.title)) .replace(/\$\{info\.TITLE\}/g, info.title.toUpperCase()) .replace(/\$\{method\.property\.immutable\}/g, hasTag(methodObj, 'property:immutable')) @@ -1149,6 +1152,7 @@ function insertMethodMacros(template, methodObj, json, templates, examples = {}) .replace(/\$\{event\.result\.type\}/, isEventMethod(methodObj) ? types.getSchemaType(result.schema, json, { name: result.name, destination: state.destination, event: true, description: methodObj.result.summary, asPath: false }) : '') .replace(/\$\{event\.result\.json\.type\}/g, resultJsonType) .replace(/\$\{method\.result\}/g, generateResult(result.schema, json, templates, { name: result.name })) + .replace(/\$\{method\.result\.json\.type\}/g, resultJsonType) .replace(/\$\{method\.result\.instantiation\}/g, resultInst) .replace(/\$\{method\.example\.value\}/g, JSON.stringify(methodObj.examples[0].result.value)) .replace(/\$\{method\.alternative\}/g, method.alternative)