Skip to content

Commit 638d8f7

Browse files
committed
Fixes after rebase
1 parent d341b26 commit 638d8f7

File tree

6 files changed

+13
-17
lines changed

6 files changed

+13
-17
lines changed

compiler/test/data/typescriptBodies/constructorWithPropertyDeclaration.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// [test] withPropertyDeclaration.kt
1+
// [test] constructorWithPropertyDeclaration.kt
22
@file:Suppress("INTERFACE_WITH_SUPERCLASS", "OVERRIDING_FINAL_MEMBER", "RETURN_TYPE_MISMATCH_ON_OVERRIDE", "CONFLICTING_OVERLOADS", "EXTERNAL_DELEGATION")
33

44
import kotlin.js.*

model-lowerings-common/src/org/jetbrains/dukat/model/commonLowerings/ModelWithOwnerTypeLowering.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ interface ModelWithOwnerTypeLowering : ModelWithOwnerLowering {
186186
val declaration = ownerContext.node
187187
return declaration.copy(
188188
primaryConstructor = declaration.primaryConstructor?.let { constructorModel ->
189-
constructorModel.copy(parameters = constructorModel.parameters.map { lowerParameterModel(ownerContext.wrap(it)) })
189+
constructorModel.copy(parameters = constructorModel.parameters.map { lowerCallableParameterModel(ownerContext.wrap(it)) })
190190
},
191191
members = declaration.members.mapNotNull { member -> lowerMemberModel(NodeOwner(member, ownerContext), parentModule) },
192192
typeParameters = declaration.typeParameters.map { typeParameterModel -> lowerTypeParameterModel(ownerContext.wrap(typeParameterModel)) },

typescript/ts-node-lowering/src/org/jetrbains/dukat/nodeLowering/NodeTypeLowering.kt

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package org.jetrbains.dukat.nodeLowering
22

33
import org.jetbrains.dukat.ast.model.nodes.ClassNode
44
import org.jetbrains.dukat.ast.model.nodes.ConstructorNode
5+
import org.jetbrains.dukat.ast.model.nodes.ConstructorParameterNode
56
import org.jetbrains.dukat.ast.model.nodes.ModuleNode
67
import org.jetbrains.dukat.ast.model.nodes.FunctionNode
78
import org.jetbrains.dukat.ast.model.nodes.FunctionTypeNode
@@ -143,13 +144,21 @@ interface NodeTypeLowering : TopLevelNodeLowering {
143144
)
144145
}
145146

146-
override fun lowerPropertyParameterNode(declaration: PropertyParameterNode): PropertyParameterNode {
147+
fun lowerPropertyParameterNode(declaration: PropertyParameterNode): PropertyParameterNode {
147148
return declaration.copy(
148149
name = lowerIdentificator(declaration.name),
149150
type = lowerType(declaration.type)
150151
)
151152
}
152153

154+
fun lowerConstructorParameterNode(declaration: ConstructorParameterNode): ConstructorParameterNode {
155+
return when (declaration) {
156+
is ParameterNode -> lowerParameterNode(declaration)
157+
is PropertyParameterNode -> lowerPropertyParameterNode(declaration)
158+
else -> declaration
159+
}
160+
}
161+
153162
override fun lowerVariableNode(declaration: VariableNode): VariableNode {
154163
return declaration.copy(name = lowerIdentificator(declaration.name), type = lowerType(declaration.type))
155164
}

typescript/ts-node-lowering/src/org/jetrbains/dukat/nodeLowering/TopLevelNodeLowering.kt

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,11 @@ package org.jetrbains.dukat.nodeLowering
33
import org.jetbrains.dukat.ast.model.duplicate
44
import org.jetbrains.dukat.ast.model.nodes.ClassLikeNode
55
import org.jetbrains.dukat.ast.model.nodes.ClassNode
6-
import org.jetbrains.dukat.ast.model.nodes.ConstructorParameterNode
7-
import org.jetbrains.dukat.ast.model.nodes.ModuleNode
86
import org.jetbrains.dukat.ast.model.nodes.EnumNode
97
import org.jetbrains.dukat.ast.model.nodes.FunctionNode
108
import org.jetbrains.dukat.ast.model.nodes.InterfaceNode
119
import org.jetbrains.dukat.ast.model.nodes.ModuleNode
1210
import org.jetbrains.dukat.ast.model.nodes.ObjectNode
13-
import org.jetbrains.dukat.ast.model.nodes.ParameterNode
14-
import org.jetbrains.dukat.ast.model.nodes.PropertyParameterNode
1511
import org.jetbrains.dukat.ast.model.nodes.TopLevelNode
1612
import org.jetbrains.dukat.ast.model.nodes.TypeAliasNode
1713
import org.jetbrains.dukat.ast.model.nodes.VariableNode
@@ -25,14 +21,6 @@ interface TopLevelNodeLowering {
2521
fun lowerEnumNode(declaration: EnumNode, owner: ModuleNode): EnumNode = declaration
2622
fun lowerTypeAliasNode(declaration: TypeAliasNode, owner: ModuleNode): TypeAliasNode = declaration
2723

28-
fun lowerConstructorParameterNode(declaration: ConstructorParameterNode): ConstructorParameterNode {
29-
return when (declaration) {
30-
is ParameterNode -> lowerParameterNode(declaration)
31-
is PropertyParameterNode -> lowerPropertyParameterNode(declaration)
32-
else -> declaration
33-
}
34-
}
35-
3624
fun lowerClassLikeNode(declaration: ClassLikeNode, owner: ModuleNode): ClassLikeNode {
3725
return when (declaration) {
3826
is InterfaceNode -> lowerInterfaceNode(declaration)

typescript/ts-node-lowering/src/org/jetrbains/dukat/nodeLowering/lowerings/introduceModels.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -337,7 +337,7 @@ internal class DocumentConverter(private val moduleNode: ModuleNode, private val
337337
// TODO: how ClassModel end up here?
338338
return when (this) {
339339
is ConstructorNode -> ConstructorModel(
340-
parameters = parameters.map { param -> param.process().copy() },
340+
parameters = parameters.map { param -> param.process() },
341341
typeParameters = convertTypeParams(typeParameters)
342342
)
343343
is MethodNode -> process()

typescript/ts-node-lowering/src/org/jetrbains/dukat/nodeLowering/lowerings/specifyUnionType.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import org.jetbrains.dukat.ast.model.makeNullable
66
import org.jetbrains.dukat.ast.model.nodes.ClassNode
77
import org.jetbrains.dukat.ast.model.nodes.ConstructorNode
88
import org.jetbrains.dukat.ast.model.nodes.ConstructorParameterNode
9-
import org.jetbrains.dukat.ast.model.nodes.ModuleNode
109
import org.jetbrains.dukat.ast.model.nodes.FunctionNode
1110
import org.jetbrains.dukat.ast.model.nodes.InterfaceNode
1211
import org.jetbrains.dukat.ast.model.nodes.MethodNode

0 commit comments

Comments
 (0)