From 3fb2161eb9731db3bf6e8bcfb29ae4e4ea6a82c6 Mon Sep 17 00:00:00 2001
From: Mat Carey <mat@dorightdigital.com>
Date: Wed, 20 Nov 2019 15:07:17 +0000
Subject: [PATCH 1/3] Made empty arrays behave the same as no array.

---
 src/govuk/components/date-input/template.njk   | 2 +-
 src/govuk/components/footer/template.njk       | 6 +++---
 src/govuk/components/summary-list/template.njk | 2 +-
 src/govuk/components/tabs/template.njk         | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/govuk/components/date-input/template.njk b/src/govuk/components/date-input/template.njk
index 8bf462b80e..29b9eadc1c 100644
--- a/src/govuk/components/date-input/template.njk
+++ b/src/govuk/components/date-input/template.njk
@@ -7,7 +7,7 @@
    aria-describedby – for example hints or error messages -#}
 {% set describedBy = params.fieldset.describedBy if params.fieldset.describedBy else "" %}
 
-{% if params.items %}
+{% if params.items | length %}
   {% set dateInputItems = params.items %}
 {% else %}
   {% set dateInputItems = [
diff --git a/src/govuk/components/footer/template.njk b/src/govuk/components/footer/template.njk
index 1e797312a3..745300b4f8 100644
--- a/src/govuk/components/footer/template.njk
+++ b/src/govuk/components/footer/template.njk
@@ -1,12 +1,12 @@
 <footer class="govuk-footer {{ params.classes if params.classes }}" role="contentinfo"
         {%- for attribute, value in params.attributes %} {{attribute}}="{{value}}"{% endfor %}>
   <div class="govuk-width-container {{ params.containerClasses if params.containerClasses }}">
-    {% if params.navigation %}
+    {% if params.navigation | length %}
       <div class="govuk-footer__navigation">
         {% for nav in params.navigation %}
           <div class="govuk-footer__section">
             <h2 class="govuk-footer__heading govuk-heading-m">{{ nav.title }}</h2>
-            {% if nav.items %}
+            {% if nav.items | length %}
               {% set listClasses %}
                 {% if nav.columns %}
                   govuk-footer__list--columns-{{ nav.columns }}
@@ -33,7 +33,7 @@
       <div class="govuk-footer__meta-item govuk-footer__meta-item--grow">
         {% if params.meta %}
           <h2 class="govuk-visually-hidden">{{ params.meta.visuallyHiddenTitle | default("Support links") }}</h2>
-          {% if params.meta.items %}
+          {% if params.meta.items | length %}
             <ul class="govuk-footer__inline-list">
               {% for item in params.meta.items %}
                 <li class="govuk-footer__inline-list-item">
diff --git a/src/govuk/components/summary-list/template.njk b/src/govuk/components/summary-list/template.njk
index 2727b10b2c..8434378922 100644
--- a/src/govuk/components/summary-list/template.njk
+++ b/src/govuk/components/summary-list/template.njk
@@ -10,7 +10,7 @@
 {# Determine if we need 2 or 3 columns #}
 {% set anyRowHasActions = false %}
 {% for row in params.rows %}
-  {% set anyRowHasActions = true if row.actions.items else anyRowHasActions %}
+  {% set anyRowHasActions = true if row.actions.items | length else anyRowHasActions %}
 {% endfor -%}
 
 <dl class="govuk-summary-list {%- if params.classes %} {{ params.classes }}{% endif %}"{% for attribute, value in params.attributes %} {{attribute}}="{{value}}"{% endfor %}>
diff --git a/src/govuk/components/tabs/template.njk b/src/govuk/components/tabs/template.njk
index 5d263f5b20..3786ba5fdd 100644
--- a/src/govuk/components/tabs/template.njk
+++ b/src/govuk/components/tabs/template.njk
@@ -6,7 +6,7 @@
   <h2 class="govuk-tabs__title">
     {{ params.title | default ("Contents") }}
   </h2>
-  {% if(params.items) %}
+  {% if(params.items | length) %}
   <ul class="govuk-tabs__list">
     {% for item in params.items %}
       {% if item %}

From 501e6b626b74d0f917c03f1cd8e498128f6c77a2 Mon Sep 17 00:00:00 2001
From: Mat Carey <mat@dorightdigital.com>
Date: Wed, 20 Nov 2019 15:46:18 +0000
Subject: [PATCH 2/3] Added tests to show how empty arrays behave.

---
 .../components/date-input/template.test.js    |  9 +++++++
 src/govuk/components/footer/template.test.js  | 18 ++++++++++++++
 .../components/summary-list/template.test.js  | 24 ++++++++++++++++++-
 src/govuk/components/tabs/template.test.js    | 14 +++++++++++
 4 files changed, 64 insertions(+), 1 deletion(-)

diff --git a/src/govuk/components/date-input/template.test.js b/src/govuk/components/date-input/template.test.js
index a0b74b24d9..e62094af50 100644
--- a/src/govuk/components/date-input/template.test.js
+++ b/src/govuk/components/date-input/template.test.js
@@ -69,6 +69,15 @@ describe('Date input', () => {
       expect($items.length).toEqual(3)
     })
 
+    it('renders defaults when an empty item array is provided', () => {
+      const $ = render('date-input', {
+        items: []
+      })
+
+      const $items = $('.govuk-date-input__item')
+      expect($items.length).toEqual(3)
+    })
+
     it('renders with default items', () => {
       const $ = render('date-input', {})
 
diff --git a/src/govuk/components/footer/template.test.js b/src/govuk/components/footer/template.test.js
index 45a01427fd..1da94e3aff 100644
--- a/src/govuk/components/footer/template.test.js
+++ b/src/govuk/components/footer/template.test.js
@@ -24,6 +24,15 @@ describe('footer', () => {
     expect($component.attr('role')).toEqual('contentinfo')
   })
 
+  it('no items displayed when no item array is provided', () => {
+    const $ = render('footer', {
+      navigation: []
+    })
+
+    const $component = $('.govuk-footer')
+    expect($component.find('.govuk-footer__navigation').length).toEqual(0)
+  })
+
   it('renders attributes correctly', () => {
     const $ = render('footer', {
       attributes: {
@@ -83,6 +92,15 @@ describe('footer', () => {
       expect($heading.text()).toEqual('Support links')
     })
 
+    it('doesn\'t render footer link list when no items are provided', () => {
+      const $ = render('footer', {
+        meta: { items: [] }
+      })
+
+      const $component = $('.govuk-footer')
+      expect($component.find('.govuk-footer__inline-list').length).toEqual(0)
+    })
+
     it('renders links', () => {
       const $ = render('footer', examples['with meta'])
 
diff --git a/src/govuk/components/summary-list/template.test.js b/src/govuk/components/summary-list/template.test.js
index b6758b2d34..013cb1acef 100644
--- a/src/govuk/components/summary-list/template.test.js
+++ b/src/govuk/components/summary-list/template.test.js
@@ -383,7 +383,7 @@ describe('Data list', () => {
 
         expect($action.hasClass('govuk-link--no-visited-state')).toBeTruthy()
       })
-      it('skips the action column when none are provided', async () => {
+      it('skips the action column when no array is provided', async () => {
         const $ = render('summary-list', {
           rows: [
             {
@@ -402,6 +402,28 @@ describe('Data list', () => {
 
         expect($action.length).toEqual(0)
       })
+      it('skips the action column when no items are in the array provided', async () => {
+        const $ = render('summary-list', {
+          rows: [
+            {
+              key: {
+                text: 'Name'
+              },
+              value: {
+                text: 'Firstname Lastname'
+              },
+              actions: {
+                items: []
+              }
+            }
+          ]
+        })
+
+        const $component = $('.govuk-summary-list')
+        const $action = $component.find('.govuk-summary-list__actions')
+
+        expect($action.length).toEqual(0)
+      })
       it('adds dummy action columns when only some rows have actions', async () => {
         const $ = render('summary-list', {
           rows: [
diff --git a/src/govuk/components/tabs/template.test.js b/src/govuk/components/tabs/template.test.js
index 230499baf5..a7667dca13 100644
--- a/src/govuk/components/tabs/template.test.js
+++ b/src/govuk/components/tabs/template.test.js
@@ -57,6 +57,20 @@ describe('Tabs', () => {
     })
 
     describe('items', () => {
+      it('doesn\'t render a list if items is not defined', () => {
+        const $ = render('tabs', {})
+
+        const $component = $('.govuk-tabs')
+        expect($component.find('.govuk-tabs__list').length).toEqual(0)
+      })
+
+      it('doesn\'t render a list if items is empty', () => {
+        const $ = render('tabs', { items: [] })
+
+        const $component = $('.govuk-tabs')
+        expect($component.find('.govuk-tabs__list').length).toEqual(0)
+      })
+
       it('render a matching tab and panel using item id', () => {
         const $ = render('tabs', {
           items: [

From 8b2cbef0d26ffc456b3c3fb267e3238488cc8d5a Mon Sep 17 00:00:00 2001
From: Mat Carey <mat@dorightdigital.com>
Date: Wed, 20 Nov 2019 15:48:39 +0000
Subject: [PATCH 3/3] Updated changelog.

---
 CHANGELOG.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 53bea20b6a..cc195fd77c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,6 +7,7 @@
 ### Fixes
 
 - [Pull request #1655: Ensure components use public `govuk-media-query` mixin](https://github.com/alphagov/govuk-frontend/pull/1655).
+- [Pull request #1638: Check component item arrays are not empty before outputting markup](https://github.com/alphagov/govuk-frontend/pull/1638).
 
 ## 3.4.0 (Feature release)
 
@@ -58,7 +59,6 @@ You can now add attributes to the `<body>` element of a page, by using the [`bod
 - [Pull request #1620: Only add underline to back link when href exists ](https://github.com/alphagov/govuk-frontend/pull/1620).
 - [Pull request #1631: Fix classes on character count when in error state](https://github.com/alphagov/govuk-frontend/pull/1631).
 
-
 ## 3.3.0 (Feature release)
 
 ### New features