babel-plugin-transform-modules-ui5 7.1.4 → 7.1.5

package/CHANGELOG.md

@@ -3,6 +3,19 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [7.1.5](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/compare/v7.1.4...v7.1.5) (2023-05-30)
+
+
+### Bug Fixes
+
+* enable support for TS param props ([#99](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/issues/99)) ([2119b41](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/commit/2119b41fbd5fb41ade7e096190cbebad03eecf7c)), closes [#65](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/issues/65)
+* ensure copyright comments to be leading comments ([#97](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/issues/97)) ([39ab194](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/commit/39ab1946f08d3fc33609bd6b9ac6cda9a985c5d7))
+* sap.ui.define without callback function ([#98](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/issues/98)) ([c1cd6a8](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/commit/c1cd6a8f158de9b5036d9c768c48972d66ae20aa)), closes [#50](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/issues/50)
+
+
+
+
+
 ## [7.1.4](https://github.com/ui5-community/babel-plugin-transform-modules-ui5/compare/v7.1.3...v7.1.4) (2023-04-11)
 
 

package/dist/classes/helpers/classes.js

@@ -54,7 +54,32 @@ function convertClassToUI5Extend(path, node, classInfo, extraStaticProps, opts)
   for (const member of node.body.body) {
     const memberName = member.key.name;
     if (_core.types.isClassMethod(member)) {
-      const func = _core.types.functionExpression(member.key, member.params, member.body, member.generator, member.async);
+      var _member$params;
+      const isConstructor = member.kind === "constructor";
+      const membersToAssign = [];
+      const params = isConstructor ? (_member$params = member.params) === null || _member$params === void 0 ? void 0 : _member$params.map(param => {
+        // handling of parameter properties for constructors (TypeScript):
+        // https://www.typescriptlang.org/docs/handbook/2/classes.html#parameter-properties
+        //   -> extracting the real parameters and store the members to assign
+        if (param.type === "TSParameterProperty") {
+          membersToAssign.push(param.parameter);
+          return param.parameter;
+        }
+        return param;
+      }) : member.params;
+      const func = _core.types.functionExpression(member.key, params, member.body, member.generator, member.async);
+      if (isConstructor) {
+        // handling of parameter properties for constructors (TypeScript):
+        //   -> assigning parameter properties as members to the instance
+        const newMembers = membersToAssign.map(member => buildMemberAssignmentStatement(_core.types.thisExpression(), {
+          key: member,
+          computed: false,
+          value: member
+        }));
+        _core.types.isSuperCallExpression;
+        const idx = member.body.body.findIndex(m => _core.types.isExpressionStatement(m) && _core.types.isCallExpression(m.expression) && _core.types.isSuper(m.expression.callee));
+        member.body.body.splice(idx === -1 ? member.body.body.length : idx + 1, 0, ...newMembers);
+      }
       if (member.static) {
         staticMembers.push(buildMemberAssignmentStatement(classNameIdentifier, _objectSpread(_objectSpread({}, member), {}, {
           value: func

package/dist/classes/visitor.js

@@ -130,7 +130,7 @@ function isSuperApply(callee) {
 function getRequiredParamsOfSAPUIDefine(path, node) {
   const defineArgs = node.arguments;
   const callbackNode = defineArgs.find(argNode => _core.types.isFunction(argNode));
-  return callbackNode.params; // Identifier
+  return (callbackNode === null || callbackNode === void 0 ? void 0 : callbackNode.params) || []; // Identifier
 }
 
 /**

package/dist/modules/helpers/wrapper.js

@@ -11,6 +11,7 @@ var ast = _interopRequireWildcard(require("../../utils/ast"));
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 function wrap(visitor, programNode, opts) {
+  var _body, _body$, _body$$leadingComment;
   let {
     defaultExport,
     exportGlobal,
@@ -25,6 +26,14 @@ function wrap(visitor, programNode, opts) {
   let {
     body
   } = programNode;
+
+  // find the copyright comment from the original program body
+  const copyright = (_body = body) === null || _body === void 0 ? void 0 : (_body$ = _body[0]) === null || _body$ === void 0 ? void 0 : (_body$$leadingComment = _body$.leadingComments) === null || _body$$leadingComment === void 0 ? void 0 : _body$$leadingComment.find((comment, idx, arr) => {
+    if (comment.value.startsWith("!")) {
+      arr.splice(idx, 1);
+      return true;
+    }
+  });
   let allExportHelperAdded = false;
   let extendAdded = false;
   opts.collapse = !opts.noExportCollapse;
@@ -108,6 +117,11 @@ function wrap(visitor, programNode, opts) {
     body.unshift(th.buildDefaultImportInterop());
   }
   programNode.body = [...preDefine, generateDefine(body, imports, exportGlobal || opts.exportAllGlobal)];
+
+  // if a copyright comment is present we append it to the new program node
+  if (copyright) {
+    programNode.body[0].leadingComments = [copyright];
+  }
 }
 function hasUseStrict(node) {
   return (node.directives || []).some(directive => directive.value.value === "use strict");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "babel-plugin-transform-modules-ui5",
-  "version": "7.1.4",
+  "version": "7.1.5",
   "description": "An unofficial babel plugin for SAP UI5.",
   "main": "dist/index.js",
   "repository": {
@@ -92,5 +92,5 @@
     "prettier": "^2.8.4",
     "rimraf": "^4.1.2"
   },
-  "gitHead": "4a3fb836d05e71f8a4b89f7597b8d234edf18e55"
+  "gitHead": "a34cbc1fe2fc53fe5ace60ac710d9579e93034bd"
 }