Application crash in ios device when I use ColumnChart

[Shunfeii]

On Android, it runs well.And I found that only the Column Chart causes the program to crash, while all others function normally.

[658jjh]

Any update for this @ehsannarmani ?

[ehsannarmani]

Hi @658jjh , not yet.
I don't have ios and mac device to test and debug, if you have, you can debug code and after fixing that you can make pr.

[658jjh]

@ehsannarmani It happend because of receiver context - the API was removed in Kotlin 2.x
I don't know why I can't create pull request, so please help to update it

Subject: [PATCH] Fix column/row chart crash on iOS
---
Index: compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/models/Bars.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/models/Bars.kt b/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/models/Bars.kt
--- a/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/models/Bars.kt	(revision 236af61927068fc072526e6291e9d9ed892ed934)
+++ b/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/models/Bars.kt	(date 1738840058975)
@@ -4,7 +4,7 @@
 import androidx.compose.animation.core.AnimationSpec
 import androidx.compose.animation.core.AnimationVector1D
 import androidx.compose.ui.graphics.Brush
-import androidx.compose.ui.graphics.Color
+import androidx.compose.ui.unit.Density
 import androidx.compose.ui.unit.Dp
 import androidx.compose.ui.unit.dp
 import kotlin.random.Random
@@ -57,5 +57,31 @@
                 }
             }
         }
+
+        sealed class RadiusPx {
+            data object None : RadiusPx()
+            data class Circular(val radius: Float) : RadiusPx()
+            data class Rectangle(
+              val topLeft: Float = 0f,
+              val topRight: Float = 0f,
+              val bottomLeft: Float = 0f,
+              val bottomRight: Float = 0f
+            ) : RadiusPx()
+        }
     }
-}
\ No newline at end of file
+}
+
+fun Bars.Data.Radius.asRadiusPx(density: Density): Bars.Data.RadiusPx {
+  with(density) {
+    return when (this@asRadiusPx) {
+      is Bars.Data.Radius.None -> Bars.Data.RadiusPx.None
+      is Bars.Data.Radius.Circular -> Bars.Data.RadiusPx.Circular(radius.toPx())
+      is Bars.Data.Radius.Rectangle -> Bars.Data.RadiusPx.Rectangle(
+        topLeft = topLeft.toPx(),
+        topRight = topRight.toPx(),
+        bottomLeft = bottomLeft.toPx(),
+        bottomRight = bottomRight.toPx()
+      )
+    }
+  }
+}
Index: compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/RowChart.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/RowChart.kt b/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/RowChart.kt
--- a/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/RowChart.kt	(revision 236af61927068fc072526e6291e9d9ed892ed934)
+++ b/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/RowChart.kt	(date 1738840058970)
@@ -49,6 +49,7 @@
 import ir.ehsannarmani.compose_charts.models.PopupProperties
 import ir.ehsannarmani.compose_charts.models.SelectedBar
 import ir.ehsannarmani.compose_charts.models.VerticalIndicatorProperties
+import ir.ehsannarmani.compose_charts.models.asRadiusPx
 import ir.ehsannarmani.compose_charts.utils.ImplementRCAnimation
 import ir.ehsannarmani.compose_charts.utils.VerticalLabels
 import ir.ehsannarmani.compose_charts.utils.calculateOffset
@@ -312,7 +313,7 @@
                                     radius = radius.reverse(horizontal = true)
                                 }
 
-                                path.addRoundRect(rect = rect, radius = radius)
+                                path.addRoundRect(rect = rect, radius = radius.asRadiusPx(this))
 
                                 val alpha = if (rect == selectedBar.value?.rect) {
                                     1f - (barAlphaDecreaseOnPopup * popupAnimation.value)
Index: compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/ColumnChart.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/ColumnChart.kt b/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/ColumnChart.kt
--- a/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/ColumnChart.kt	(revision 236af61927068fc072526e6291e9d9ed892ed934)
+++ b/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/ColumnChart.kt	(date 1738840058960)
@@ -49,6 +49,7 @@
 import ir.ehsannarmani.compose_charts.models.LabelProperties
 import ir.ehsannarmani.compose_charts.models.PopupProperties
 import ir.ehsannarmani.compose_charts.models.SelectedBar
+import ir.ehsannarmani.compose_charts.models.asRadiusPx
 import ir.ehsannarmani.compose_charts.utils.HorizontalLabels
 import ir.ehsannarmani.compose_charts.utils.ImplementRCAnimation
 import ir.ehsannarmani.compose_charts.utils.calculateOffset
@@ -331,7 +332,7 @@
                                     radius = radius.reverse()
                                 }
 
-                                path.addRoundRect(rect = rect, radius = radius)
+                                path.addRoundRect(rect = rect, radius = radius.asRadiusPx(this))
                                 val alpha = if (rect == selectedValue.value?.rect) {
                                     1f - (barAlphaDecreaseOnPopup * popupAnimation.value)
                                 } else {
Index: compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/extensions/CornerRadius.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/extensions/CornerRadius.kt b/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/extensions/CornerRadius.kt
--- a/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/extensions/CornerRadius.kt	(revision 236af61927068fc072526e6291e9d9ed892ed934)
+++ b/compose-charts/src/commonMain/kotlin/ir/ehsannarmani/compose_charts/extensions/CornerRadius.kt	(date 1738840058980)
@@ -4,51 +4,48 @@
 import androidx.compose.ui.geometry.Rect
 import androidx.compose.ui.geometry.RoundRect
 import androidx.compose.ui.graphics.Path
-import androidx.compose.ui.graphics.drawscope.DrawScope
 import ir.ehsannarmani.compose_charts.models.Bars
 
-
-context(DrawScope)
 fun Path.addRoundRect(
     rect: Rect,
-    radius: Bars.Data.Radius
+    radius: Bars.Data.RadiusPx
 ){
     when (radius) {
-        is Bars.Data.Radius.None -> {
+        is Bars.Data.RadiusPx.None -> {
             addRect(rect)
         }
 
-        is Bars.Data.Radius.Circular -> {
+        is Bars.Data.RadiusPx.Circular -> {
             addRoundRect(
                 roundRect = RoundRect(
                     rect = rect,
                     cornerRadius = CornerRadius(
-                        x = radius.radius.toPx(),
-                        y = radius.radius.toPx()
+                        x = radius.radius,
+                        y = radius.radius
                     )
                 )
             )
         }
 
-        is Bars.Data.Radius.Rectangle -> {
+        is Bars.Data.RadiusPx.Rectangle -> {
             addRoundRect(
                 roundRect = RoundRect(
                     rect = rect,
                     topLeft = CornerRadius(
-                        x = radius.topLeft.toPx(),
-                        y = radius.topLeft.toPx()
+                        x = radius.topLeft,
+                        y = radius.topLeft
                     ),
                     topRight = CornerRadius(
-                        x = radius.topRight.toPx(),
-                        y = radius.topRight.toPx()
+                        x = radius.topRight,
+                        y = radius.topRight
                     ),
                     bottomLeft = CornerRadius(
-                        x = radius.bottomLeft.toPx(),
-                        y = radius.bottomLeft.toPx()
+                        x = radius.bottomLeft,
+                        y = radius.bottomLeft
                     ),
                     bottomRight = CornerRadius(
-                        x = radius.bottomRight.toPx(),
-                        y = radius.bottomRight.toPx()
+                        x = radius.bottomRight,
+                        y = radius.bottomRight
                     ),
                 )
             )

[ehsannarmani]

Hi @658jjh , thanks for following the issue.
What's the problem you cant create pr?

[658jjh]

@ehsannarmani #71