[#599] Review: Apply the rule which says that mandatory parameters mu…

…st be before optional parameters in a function signature
Orange-OpenSource · Sep 5, 2023 · a90978f · a90978f
1 parent 76255f9
commit a90978f
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/src/main/java/com/orange/ods/compose/component/button/OdsIconToggleButtonsRow.kt b/lib/src/main/java/com/orange/ods/compose/component/button/OdsIconToggleButtonsRow.kt
@@ -76,9 +76,9 @@ fun OdsIconToggleButtonsRow(
             )
     ) {
         icons.forEachIndexed { index, icon ->
-            icon.Content(index = index, displaySurface = displaySurface, selected = selectedIndex == index) { clickedButtonIndex ->
+            icon.Content(index = index, displaySurface = displaySurface, selected = selectedIndex == index, onClick = { clickedButtonIndex ->
                 onSelectedIndexChange(clickedButtonIndex)
-            }
+            })
             if (index < icons.size) {
                 Divider(
                     modifier = Modifier
@@ -137,11 +137,11 @@ class OdsIconToggleButtonsRowIcon : OdsComponentIcon {
         }
 
     @Composable
-    internal fun Content(index: Int, displaySurface: OdsDisplaySurface = OdsDisplaySurface.Default, selected: Boolean = false, onClick: (Int) -> Unit) {
+    internal fun Content(index: Int, onClick: (Int) -> Unit, displaySurface: OdsDisplaySurface = OdsDisplaySurface.Default, selected: Boolean = false) {
         this.index = index
+        this.onClick = onClick
         this.displaySurface = displaySurface
         this.selected = selected
-        this.onClick = onClick
         Content()
     }