mirror of
https://github.com/swc-project/swc.git
synced 2024-11-23 17:54:15 +03:00
fix(es/codegen): Fix codegen of ts specific syntax in class props (#8426)
**Description:** This fixes the codegen of private properties with `is_optional` or `definite` and class properties with `definite`.
This commit is contained in:
parent
4bd15908c1
commit
7566ddf0b7
@ -1533,6 +1533,12 @@ where
|
|||||||
|
|
||||||
emit!(n.key);
|
emit!(n.key);
|
||||||
if let Some(type_ann) = &n.type_ann {
|
if let Some(type_ann) = &n.type_ann {
|
||||||
|
if n.is_optional {
|
||||||
|
punct!("?");
|
||||||
|
}
|
||||||
|
if n.definite {
|
||||||
|
punct!("!");
|
||||||
|
}
|
||||||
punct!(":");
|
punct!(":");
|
||||||
space!();
|
space!();
|
||||||
emit!(type_ann);
|
emit!(type_ann);
|
||||||
@ -1583,6 +1589,9 @@ where
|
|||||||
emit!(n.key);
|
emit!(n.key);
|
||||||
|
|
||||||
if let Some(ty) = &n.type_ann {
|
if let Some(ty) = &n.type_ann {
|
||||||
|
if n.definite {
|
||||||
|
punct!("!");
|
||||||
|
}
|
||||||
punct!(":");
|
punct!(":");
|
||||||
space!();
|
space!();
|
||||||
emit!(ty);
|
emit!(ty);
|
||||||
|
@ -0,0 +1,8 @@
|
|||||||
|
class MyClass {
|
||||||
|
prop1?: string;
|
||||||
|
prop2!: string;
|
||||||
|
#prop3?: string;
|
||||||
|
#prop4?: string = "test";
|
||||||
|
static readonly prop5!: string;
|
||||||
|
readonly #prop6 = "asdf";
|
||||||
|
}
|
@ -0,0 +1,8 @@
|
|||||||
|
class MyClass {
|
||||||
|
prop1?: string;
|
||||||
|
prop2!: string;
|
||||||
|
#prop3?: string;
|
||||||
|
#prop4?: string = "test";
|
||||||
|
static readonly prop5!: string;
|
||||||
|
readonly #prop6 = "asdf";
|
||||||
|
}
|
1
crates/swc_ecma_codegen/tests/fixture/typescript/class_prop/output.min.js
vendored
Normal file
1
crates/swc_ecma_codegen/tests/fixture/typescript/class_prop/output.min.js
vendored
Normal file
@ -0,0 +1 @@
|
|||||||
|
class MyClass{prop1?: string;prop2!: string;#prop3?: string;#prop4?: string="test";static readonly prop5!: string;readonly #prop6="asdf"}
|
@ -3,7 +3,7 @@ enum MyEnum {
|
|||||||
y = "yyy"
|
y = "yyy"
|
||||||
}
|
}
|
||||||
class Xpto {
|
class Xpto {
|
||||||
value: MyEnum;
|
value!: MyEnum;
|
||||||
}
|
}
|
||||||
_ts_decorate([
|
_ts_decorate([
|
||||||
Decorator(),
|
Decorator(),
|
||||||
|
@ -1,5 +1,5 @@
|
|||||||
class User {
|
class User {
|
||||||
currency: "usd" | "eur" | "yen";
|
currency!: "usd" | "eur" | "yen";
|
||||||
}
|
}
|
||||||
_ts_decorate([
|
_ts_decorate([
|
||||||
column(),
|
column(),
|
||||||
|
@ -1,10 +1,10 @@
|
|||||||
export class Product extends TimestampedEntity {
|
export class Product extends TimestampedEntity {
|
||||||
id: string;
|
id!: string;
|
||||||
price: number;
|
price!: number;
|
||||||
type: ProductType;
|
type!: ProductType;
|
||||||
productEntityId: string;
|
productEntityId!: string;
|
||||||
/* ANCHOR: Relations ------------------------------------------------------ */ orders: Order[];
|
/* ANCHOR: Relations ------------------------------------------------------ */ orders!: Order[];
|
||||||
discounts: Discount[];
|
discounts!: Discount[];
|
||||||
}
|
}
|
||||||
_ts_decorate([
|
_ts_decorate([
|
||||||
PrimaryGeneratedColumn("uuid")
|
PrimaryGeneratedColumn("uuid")
|
||||||
|
@ -1,5 +1,5 @@
|
|||||||
export class Product extends TimestampedEntity {
|
export class Product extends TimestampedEntity {
|
||||||
id: string;
|
id!: string;
|
||||||
}
|
}
|
||||||
_ts_decorate([
|
_ts_decorate([
|
||||||
PrimaryGeneratedColumn("uuid")
|
PrimaryGeneratedColumn("uuid")
|
||||||
|
@ -144,6 +144,7 @@ impl VisitMut for StripType {
|
|||||||
prop.readonly = false;
|
prop.readonly = false;
|
||||||
prop.is_override = false;
|
prop.is_override = false;
|
||||||
prop.is_optional = false;
|
prop.is_optional = false;
|
||||||
|
prop.definite = false;
|
||||||
prop.accessibility = None;
|
prop.accessibility = None;
|
||||||
prop.visit_mut_children_with(self);
|
prop.visit_mut_children_with(self);
|
||||||
}
|
}
|
||||||
@ -152,6 +153,7 @@ impl VisitMut for StripType {
|
|||||||
prop.readonly = false;
|
prop.readonly = false;
|
||||||
prop.is_override = false;
|
prop.is_override = false;
|
||||||
prop.is_optional = false;
|
prop.is_optional = false;
|
||||||
|
prop.definite = false;
|
||||||
prop.accessibility = None;
|
prop.accessibility = None;
|
||||||
prop.visit_mut_children_with(self);
|
prop.visit_mut_children_with(self);
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user