@riotjs/compiler - npm Package Compare versions

Comparing version 4.5.2 to 4.5.3

CHANGELOG.md

 # Compiler Changes
 
+### v4.5.3
+- Fix wrong arguments compilation in expression attributes https://github.com/riot/riot/issues/2788
+
 ### v4.5.2
@@ -4,0 +7,0 @@ - Fix avoid to set the `expr` attribute as redundant expressions

package.json

 {
   "name": "@riotjs/compiler",
-  "version": "4.5.2",
+  "version": "4.5.3",
   "description": "Compiler for riot .tag files",
@@ -13,3 +13,2 @@ "main": "dist/index.js",
   "scripts": {
-    "prepare": "npm i pug@2.0.3 node-sass@4.12.0 @babel/core@7 @babel/preset-env@7 --no-save",
     "lint": "eslint src/ test/ build/",
@@ -37,11 +36,15 @@ "cov": "nyc report --reporter=text-lcov | coveralls",
   "devDependencies": {
-    "@riotjs/dom-bindings": "^4.4.6",
+    "@babel/core": "^7.7.5",
+    "@babel/preset-env": "^7.7.5",
+    "@riotjs/dom-bindings": "^4.5.0",
     "chai": "^4.2.0",
-    "coveralls": "^3.0.7",
-    "eslint": "^6.6.0",
+    "coveralls": "^3.0.9",
+    "eslint": "^6.7.2",
     "eslint-config-riot": "^3.0.0",
     "esm": "^3.2.25",
     "mocha": "^6.2.2",
+    "node-sass": "^4.13.0",
     "nyc": "^14.1.1",
-    "rollup": "^1.25.2",
+    "pug": "^2.0.4",
+    "rollup": "^1.27.8",
     "rollup-plugin-alias": "^2.2.0",
@@ -68,3 +71,3 @@ "rollup-plugin-commonjs": "^10.1.0",
     "dom-nodes": "^1.1.3",
-    "globals": "^12.1.1",
+    "globals": "^12.3.0",
     "recast": "^0.18.5",
@@ -71,0 +74,0 @@ "source-map": "^0.7.3"

src/generators/template/utils.js

@@ -19,3 +19,3 @@ import {
 import { hasExpressions, isGlobal, isTagNode, isTextNode, isVoidNode } from './checks'
-import { isBinaryExpression, isIdentifier, isLiteral, isThisExpression } from '../../utils/ast-nodes-checks'
+import { isBinaryExpression, isIdentifier, isLiteral, isMemberExpression, isThisExpression } from '../../utils/ast-nodes-checks'
 import { nullNode, simplePropertyNode } from '../../utils/custom-ast-nodes'
@@ -90,4 +90,3 @@ import addLinesOffset from '../../utils/add-lines-offset'
   const value = path.node.value
-
-  if (isIdentifier(value)) {
+  if (isIdentifier(value) || isMemberExpression(value)) {
     updateNodeScope(path.get('value'))
@@ -94,0 +93,0 @@ } else {

src/utils/ast-nodes-checks.js

@@ -16,2 +16,3 @@ import globalScope from 'globals'
 export const isExportDefaultStatement = n => namedTypes.ExportDefaultDeclaration.check(n)
+export const isMemberExpression = n => namedTypes.MemberExpression.check(n)
 
@@ -18,0 +19,0 @@ export const isBrowserAPI = ({name}) => browserAPIs.includes(name)

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

2015173

increased by2.83%

Lines of code

51850

increased by3.33%

Worsened metrics

Dev dependency count

19

increased by26.67%

Dependency changes

• Updatedglobals@^12.3.0