Skip to content

Commit dc57748

Browse files
using exception class
1 parent 0040315 commit dc57748

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed
Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,8 @@
11
package org.jetbrains.kotlinx.dataframe.exceptions
22

3-
public class ColumnsWithDifferentParentException() :
3+
public class ColumnsWithDifferentParentException(message: String) :
44
IllegalArgumentException(),
55
DataFrameError {
66

7-
override val message: String
8-
get() = "Cannot move columns to an index remaining inside group if they have different parent"
7+
override val message: String = message
98
}

core/src/main/kotlin/org/jetbrains/kotlinx/dataframe/impl/api/move.kt

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,18 +18,17 @@ import org.jetbrains.kotlinx.dataframe.api.move
1818
import org.jetbrains.kotlinx.dataframe.api.replace
1919
import org.jetbrains.kotlinx.dataframe.api.to
2020
import org.jetbrains.kotlinx.dataframe.api.toDataFrame
21-
import org.jetbrains.kotlinx.dataframe.api.toPath
2221
import org.jetbrains.kotlinx.dataframe.api.with
2322
import org.jetbrains.kotlinx.dataframe.columns.ColumnPath
2423
import org.jetbrains.kotlinx.dataframe.columns.ColumnWithPath
2524
import org.jetbrains.kotlinx.dataframe.columns.UnresolvedColumnsPolicy
2625
import org.jetbrains.kotlinx.dataframe.columns.toColumnSet
26+
import org.jetbrains.kotlinx.dataframe.exceptions.ColumnsWithDifferentParentException
2727
import org.jetbrains.kotlinx.dataframe.impl.DataFrameReceiver
2828
import org.jetbrains.kotlinx.dataframe.impl.asList
2929
import org.jetbrains.kotlinx.dataframe.impl.columns.toColumnWithPath
3030
import org.jetbrains.kotlinx.dataframe.impl.columns.tree.ColumnPosition
3131
import org.jetbrains.kotlinx.dataframe.impl.columns.tree.getOrPut
32-
import org.jetbrains.kotlinx.dataframe.impl.last
3332
import org.jetbrains.kotlinx.dataframe.path
3433
import kotlin.collections.first
3534

@@ -144,7 +143,7 @@ internal fun <T, C> MoveClause<T, C>.moveToImpl(columnIndex: Int, insideGroup: B
144143
val columnsToMoveParents = columnsToMove.map { it.path.dropLast() }
145144
val parentOfFirst = columnsToMoveParents.first()
146145
if (columnsToMoveParents.any { it != parentOfFirst }) {
147-
throw IllegalArgumentException(
146+
throw ColumnsWithDifferentParentException(
148147
"Cannot move columns to an index remaining inside group if they have different parent",
149148
)
150149
}

0 commit comments

Comments
 (0)