PrimaryActionList size check is removed as secondary action can exist without primary. (#6319)

karthikgopal36 · almedina-ms · web-flow · commit ce47f4911e6e · 2021-09-16T14:41:59.000-07:00
Co-authored-by: almedina-ms &lt;35784165+almedina-ms@users.noreply.github.com&gt;
diff --git a/source/android/adaptivecards/src/main/java/io/adaptivecards/renderer/ActionLayoutRenderer.java b/source/android/adaptivecards/src/main/java/io/adaptivecards/renderer/ActionLayoutRenderer.java
@@ -49,12 +49,14 @@ public View renderActions(
         HostConfig hostConfig,
         RenderArgs renderArgs) throws AdaptiveFallbackException
     {
-        long size;
-        if (baseActionElementList == null || (size = baseActionElementList.size()) <= 0)
+
+        if (baseActionElementList == null)
         {
             return null;
         }
 
+        long size = baseActionElementList.size();
+
         LinearLayout actionButtonsLayout = new LinearLayout(context);
         actionButtonsLayout.setLayoutParams(new LinearLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.WRAP_CONTENT));
         int alignment = hostConfig.GetActions().getActionAlignment().swigValue();

Original file line number	Diff line number	Diff line change
`@@ -49,12 +49,14 @@ public View renderActions(`
`49`	`49`	`HostConfig hostConfig,`
`50`	`50`	`RenderArgs renderArgs) throws AdaptiveFallbackException`
`51`	`51`	`{`
`52`		`- long size;`
`53`		`- if (baseActionElementList == null \|\| (size = baseActionElementList.size()) <= 0)`
	`52`	`+`
	`53`	`+ if (baseActionElementList == null)`
`54`	`54`	`{`
`55`	`55`	`return null;`
`56`	`56`	`}`
`57`	`57`
	`58`	`+ long size = baseActionElementList.size();`
	`59`	`+`
`58`	`60`	`LinearLayout actionButtonsLayout = new LinearLayout(context);`
`59`	`61`	`actionButtonsLayout.setLayoutParams(new LinearLayout.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.WRAP_CONTENT));`
`60`	`62`	`int alignment = hostConfig.GetActions().getActionAlignment().swigValue();`