compiler: support arbitrary string js tags

myitcv · myitcv · commit 89aa286cf0ab · 2018-03-24T19:20:32.000Z
Javascript allows for arbitrary strings to be used as index values on an Object. A good example of where this comes up is React where "data-*" and "aria-*" indexes are used on objects (https://reactjs.org/docs/dom-elements.html). By switching from property selectors to the index operator for *js.Object special field access we can support arbitrary string values in js tags. Fixes #778
diff --git a/compiler/expressions.go b/compiler/expressions.go
@@ -516,9 +516,9 @@ func (c *funcContext) translateExpr(expr ast.Expr) *expression {
 			fields, jsTag := c.translateSelection(sel, e.Pos())
 			if jsTag != "" {
 				if _, ok := sel.Type().(*types.Signature); ok {
-					return c.formatExpr("$internalize(%1e.%2s.%3s, %4s, %1e.%2s)", e.X, strings.Join(fields, "."), jsTag, c.typeName(sel.Type()))
+					return c.formatExpr("$internalize(%1e.%2s[\"%3s\"], %4s, %1e.%2s)", e.X, strings.Join(fields, "."), jsTag, c.typeName(sel.Type()))
 				}
-				return c.internalize(c.formatExpr("%e.%s.%s", e.X, strings.Join(fields, "."), jsTag), sel.Type())
+				return c.internalize(c.formatExpr("%e.%s[\"%s\"]", e.X, strings.Join(fields, "."), jsTag), sel.Type())
 			}
 			return c.formatExpr("%e.%s", e.X, strings.Join(fields, "."))
 		case types.MethodVal:
@@ -669,7 +669,7 @@ func (c *funcContext) translateExpr(expr ast.Expr) *expression {
 			case types.FieldVal:
 				fields, jsTag := c.translateSelection(sel, f.Pos())
 				if jsTag != "" {
-					call := c.formatExpr("%e.%s.%s(%s)", f.X, strings.Join(fields, "."), jsTag, externalizeArgs(e.Args))
+					call := c.formatExpr("%e.%s[\"%s\"](%s)", f.X, strings.Join(fields, "."), jsTag, externalizeArgs(e.Args))
 					switch sig.Results().Len() {
 					case 0:
 						return call
diff --git a/compiler/statements.go b/compiler/statements.go
@@ -703,7 +703,7 @@ func (c *funcContext) translateAssign(lhs, rhs ast.Expr, define bool) string {
 		}
 		fields, jsTag := c.translateSelection(sel, l.Pos())
 		if jsTag != "" {
-			return fmt.Sprintf("%s.%s.%s = %s;", c.translateExpr(l.X), strings.Join(fields, "."), jsTag, c.externalize(rhsExpr.String(), sel.Type()))
+			return fmt.Sprintf("%s.%s[\"%s\"] = %s;", c.translateExpr(l.X), strings.Join(fields, "."), jsTag, c.externalize(rhsExpr.String(), sel.Type()))
 		}
 		return fmt.Sprintf("%s.%s = %s;", c.translateExpr(l.X), strings.Join(fields, "."), rhsExpr)
 	case *ast.StarExpr:
diff --git a/js/js_test.go b/js/js_test.go
@@ -597,3 +597,26 @@ func TestTypeSwitchJSObject(t *testing.T) {
 		}
 	}
 }
+
+type StructWithNonIdentifierJsTags struct {
+	object *js.Object
+	Name   string `js:"my name"`
+}
+
+func TestStructWithNonIdentifierJsTags(t *testing.T) {
+	s := StructWithNonIdentifierJsTags{
+		object: js.Global.Get("Object").New(),
+	}
+
+	const want = "Paul"
+
+	// externalise a value
+	s.Name = want
+
+	// internalise again
+	got := s.Name
+
+	if want != got {
+		t.Errorf("Value via non-identifier js tag field gave %q, want %q", got, want)
+	}
+}

Original file line number	Diff line number	Diff line change
`@@ -703,7 +703,7 @@ func (c *funcContext) translateAssign(lhs, rhs ast.Expr, define bool) string {`
`703`	`703`	`}`
`704`	`704`	`fields, jsTag := c.translateSelection(sel, l.Pos())`
`705`	`705`	`if jsTag != "" {`
`706`		`- return fmt.Sprintf("%s.%s.%s = %s;", c.translateExpr(l.X), strings.Join(fields, "."), jsTag, c.externalize(rhsExpr.String(), sel.Type()))`
	`706`	`+ return fmt.Sprintf("%s.%s[\"%s\"] = %s;", c.translateExpr(l.X), strings.Join(fields, "."), jsTag, c.externalize(rhsExpr.String(), sel.Type()))`
`707`	`707`	`}`
`708`	`708`	`return fmt.Sprintf("%s.%s = %s;", c.translateExpr(l.X), strings.Join(fields, "."), rhsExpr)`
`709`	`709`	`case *ast.StarExpr:`