From 432a97f6f0e408c9c799d16bafed4c3333a8f0b4 Mon Sep 17 00:00:00 2001
From: JasonLiuZhuoCheng <jasonlzc2017@gmail.com>
Date: Mon, 22 Nov 2021 13:22:37 -0500
Subject: [PATCH] Enable complex data type UI (#11989)

* fix byte[] displaying and enable user to specify byte[] value

* use encodeToByteArray and decodeToString

* change variable naming
---
 .../ClusterDetailFragment.kt                  | 49 +++++++++++++------
 1 file changed, 35 insertions(+), 14 deletions(-)

diff --git a/src/android/CHIPTool/app/src/main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt b/src/android/CHIPTool/app/src/main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
index 268841a7cf95e3..5e6f45f0afc717 100644
--- a/src/android/CHIPTool/app/src/main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
+++ b/src/android/CHIPTool/app/src/main/java/com/google/chip/chiptool/clusterclient/clusterinteraction/ClusterDetailFragment.kt
@@ -95,6 +95,8 @@ class ClusterDetailFragment : Fragment() {
     return when (type) {
       Int::class.java -> data.toInt()
       Boolean::class.java -> data.toBoolean()
+      ByteArray::class.java -> data.encodeToByteArray()
+      Long::class.java -> data.toLong()
       else -> data
     }
   }
@@ -169,7 +171,14 @@ class ClusterDetailFragment : Fragment() {
     selectedInteractionInfo.commandParameters.forEach { (paramName, paramInfo) ->
       val param = inflater.inflate(R.layout.cluster_parameter_item, null, false) as ConstraintLayout
       param.clusterParameterNameTv.text = "${paramName}"
-      param.clusterParameterTypeTv.text = "${paramInfo.type}"
+      // byte[].class will be output as class [B, which is not readable, so dynamically change it
+      // to Byte[]. If more custom logic is required, should add a className field in
+      // commandParameterInfo
+      if (paramInfo.type == ByteArray::class.java) {
+        param.clusterParameterTypeTv.text = "Byte[]"
+      } else {
+        param.clusterParameterTypeTv.text = "${paramInfo.type}"
+      }
       parameterList.addView(param)
     }
   }
@@ -181,14 +190,14 @@ class ClusterDetailFragment : Fragment() {
   ) {
     responseValues.forEach { (variableNameType, response) ->
       if (response is List<*>) {
-        createListReadAttributeView(response, inflater, callbackList, variableNameType)
+        createListResponseView(response, inflater, callbackList, variableNameType)
       } else {
-        createBasicReadAttributeView(response, inflater, callbackList, variableNameType)
+        createBasicResponseView(response, inflater, callbackList, variableNameType)
       }
     }
   }
 
-  private fun createBasicReadAttributeView(
+  private fun createBasicResponseView(
     response: Any,
     inflater: LayoutInflater,
     callbackList: LinearLayout,
@@ -197,12 +206,16 @@ class ClusterDetailFragment : Fragment() {
     val callbackItem =
       inflater.inflate(R.layout.cluster_callback_item, null, false) as ConstraintLayout
     callbackItem.clusterCallbackNameTv.text = variableNameType.name
-    callbackItem.clusterCallbackDataTv.text = response.toString()
+    callbackItem.clusterCallbackDataTv.text = if (response.javaClass == ByteArray::class.java) {
+      (response as ByteArray).decodeToString()
+    } else {
+      response.toString()
+    }
     callbackItem.clusterCallbackTypeTv.text = variableNameType.type
     callbackList.addView(callbackItem)
   }
 
-  private fun createListReadAttributeView(
+  private fun createListResponseView(
     response: List<*>,
     inflater: LayoutInflater,
     callbackList: LinearLayout,
@@ -215,21 +228,29 @@ class ClusterDetailFragment : Fragment() {
       callbackList.addView(emptyCallback)
     } else {
       response.forEachIndexed { index, it ->
-        val readAttributeCallbackItem =
+        val attributeCallbackItem =
           inflater.inflate(R.layout.cluster_callback_item, null, false) as ConstraintLayout
-        readAttributeCallbackItem.clusterCallbackNameTv.text = variableNameType.name + "[$index]"
-        val objectString = it.toString()
-        val callbackClassName = it!!.javaClass.toString().split('$').last()
-        readAttributeCallbackItem.clusterCallbackDataTv.text = callbackClassName
-        readAttributeCallbackItem.clusterCallbackDataTv.setOnClickListener {
+        attributeCallbackItem.clusterCallbackNameTv.text = variableNameType.name + "[$index]"
+        val objectString = if (it!!.javaClass == ByteArray::class.java) {
+          (it as ByteArray).contentToString()
+        } else {
+          it.toString()
+        }
+        var callbackClassName = if (it!!.javaClass == ByteArray::class.java) {
+          "Byte[]"
+        } else {
+          it!!.javaClass.toString().split('$').last()
+        }
+        attributeCallbackItem.clusterCallbackDataTv.text = callbackClassName
+        attributeCallbackItem.clusterCallbackDataTv.setOnClickListener {
           AlertDialog.Builder(requireContext())
             .setTitle(callbackClassName)
             .setMessage(objectString)
             .create()
             .show()
         }
-        readAttributeCallbackItem.clusterCallbackTypeTv.text = "List<$callbackClassName>"
-        callbackList.addView(readAttributeCallbackItem)
+        attributeCallbackItem.clusterCallbackTypeTv.text = "List<$callbackClassName>"
+        callbackList.addView(attributeCallbackItem)
       }
     }
   }