Fix checking the index of lists in chart label formatter classes

wordpress-mobile · May 22, 2024 · e31a090 · e31a090
1 parent 930fa44
commit e31a090
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...ess/src/main/java/org/wordpress/android/ui/stats/refresh/utils/LineChartLabelFormatter.kt b/...ess/src/main/java/org/wordpress/android/ui/stats/refresh/utils/LineChartLabelFormatter.kt
@@ -10,7 +10,7 @@ class LineChartLabelFormatter @Inject constructor(
 ) : ValueFormatter() {
     override fun getAxisLabel(value: Float, axis: AxisBase?): String {
         val index = value.toInt()
-        return if (entries.isNotEmpty() && index in 0..entries.size) {
+        return if (entries.isNotEmpty() && index in entries.indices) {
             entries[index].label
         } else {
             ""

diff --git a/.../main/java/org/wordpress/android/ui/stats/refresh/utils/SubscribersChartLabelFormatter.kt b/.../main/java/org/wordpress/android/ui/stats/refresh/utils/SubscribersChartLabelFormatter.kt
@@ -10,7 +10,7 @@ class SubscribersChartLabelFormatter @Inject constructor(
 ) : ValueFormatter() {
     override fun getAxisLabel(value: Float, axis: AxisBase?): String {
         val index = value.toInt()
-        return if (entries.isNotEmpty() && index in 0..entries.size) {
+        return if (entries.isNotEmpty() && index in entries.indices) {
             entries[index].label
         } else {
             ""