Skip to content

Commit e79cabf

Browse files
Remove numbers and other invalid characters from the component name field (#39)
1 parent b118a13 commit e79cabf

4 files changed

Lines changed: 41 additions & 41 deletions

File tree

src/server/forms/report-a-terrorist.json

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -54,15 +54,15 @@
5454
"schema": {}
5555
},
5656
{
57-
"name": "3jdOpV",
57+
"name": "ajdOpV",
5858
"options": {},
5959
"type": "Details",
6060
"title": "Help me take a screenshot",
6161
"content": "Try this:Press the Shift key (⇧), command (or Cmd), and 3The screenshot will be saved to your DesktopYou can now upload it to the formTry this:Press the Ctrl key and the switch window keyThe screenshot will be saved to your DownloadsYou can now upload it to the formIf that doesn’t work, try pressing Ctrl and F5.When viewing the material:Press the Prt Scr key (or similar) to take a copy of your screenPaste the image into Microsoft Paint or a similar applicationSave the file to your computerUpload the file to the formIf that doesn’t work, you may need to search for how to take screenshots on your particular computer model.",
6262
"schema": {}
6363
},
6464
{
65-
"name": "LU6RMD",
65+
"name": "LUBRMD",
6666
"options": {},
6767
"type": "RadiosField",
6868
"title": "Do you have any evidence?",
@@ -92,15 +92,15 @@
9292
"title": "Is there anything else you can tell us?",
9393
"components": [
9494
{
95-
"name": "HETM3o",
95+
"name": "HETMBo",
9696
"title": "Html",
9797
"options": {},
9898
"type": "Html",
9999
"content": "<p class=\"govuk-body\">Details may include:who shared the materialwhen it was shareda description, if you haven’t provided a link or evidence</p>",
100100
"schema": {}
101101
},
102102
{
103-
"name": "evZ-IJ",
103+
"name": "evZxIJ",
104104
"options": {
105105
"required": false
106106
},
@@ -141,7 +141,7 @@
141141
"title": "Yes I have evidence",
142142
"components": [
143143
{
144-
"name": "koE_ae",
144+
"name": "koExae",
145145
"options": {
146146
"required": false
147147
},
@@ -230,7 +230,7 @@
230230
"conditions": [
231231
{
232232
"field": {
233-
"name": "PMXq1s.LU6RMD",
233+
"name": "PMXq1s.LUBRMD",
234234
"type": "RadiosField",
235235
"display": "Do you have any evidence? in PMXq1s"
236236
},
@@ -252,7 +252,7 @@
252252
"conditions": [
253253
{
254254
"field": {
255-
"name": "PMXq1s.LU6RMD",
255+
"name": "PMXq1s.LUBRMD",
256256
"type": "RadiosField",
257257
"display": "Do you have any evidence? in PMXq1s"
258258
},

src/server/forms/runner-components-test.json

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"path": "/do-you-own-a-vehicle",
77
"components": [
88
{
9-
"name": "qqbRw1",
9+
"name": "qqbRwx",
1010
"options": {},
1111
"type": "YesNoField",
1212
"title": "Do you own a vehicle?",
@@ -32,7 +32,7 @@
3232
"title": "What address is the vehicle registered to?",
3333
"components": [
3434
{
35-
"name": "sFR4aX",
35+
"name": "sFRxaX",
3636
"options": {},
3737
"type": "UkAddressField",
3838
"title": "What address is the vehicle registered to?",
@@ -46,7 +46,7 @@
4646
"schema": {}
4747
},
4848
{
49-
"name": "Z0Guyn",
49+
"name": "ZxGuyn",
5050
"options": {},
5151
"type": "CheckboxesField",
5252
"title": "Which Clean Air Zones are you claiming an exemption for?",
@@ -66,7 +66,7 @@
6666
"title": "Clean Air Zone (CAZ) Exemption",
6767
"components": [
6868
{
69-
"name": "MOB13t",
69+
"name": "MOBxxt",
7070
"title": "Html",
7171
"options": {},
7272
"type": "Html",
@@ -86,23 +86,23 @@
8686
"title": "Details about your vehicle",
8787
"components": [
8888
{
89-
"name": "0ZVmN_",
89+
"name": "xZVmNx",
9090
"options": {},
9191
"type": "AutocompleteField",
9292
"title": "What is the make of you vehicle?",
9393
"list": "-HMHHj",
9494
"schema": {}
9595
},
9696
{
97-
"name": "gHSgo2",
97+
"name": "gHSgox",
9898
"options": {},
9999
"type": "TextField",
100100
"title": "Vehicle Model",
101101
"hint": "For example A1, 740, Elantra",
102102
"schema": {}
103103
},
104104
{
105-
"name": "4LZ9to",
105+
"name": "xLZxto",
106106
"options": {},
107107
"type": "DatePartsField",
108108
"title": "Date you purchased the vehicle?",
@@ -152,7 +152,7 @@
152152
"schema": {}
153153
},
154154
{
155-
"name": "0zL5bB",
155+
"name": "xzLxbB",
156156
"options": {},
157157
"type": "TelephoneNumberField",
158158
"title": "Contact number",
@@ -171,15 +171,15 @@
171171
"title": "final steps",
172172
"components": [
173173
{
174-
"name": "fkd8av",
174+
"name": "fkdxav",
175175
"options": {},
176176
"type": "List",
177177
"title": "Declaration",
178178
"list": "mJHWaC",
179179
"schema": {}
180180
},
181181
{
182-
"name": "L_2AYe",
182+
"name": "LxxAYe",
183183
"options": {},
184184
"type": "EmailAddressField",
185185
"title": "Your email address",

test/form/definitions/titles.json

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
},
1717
{
1818
"type": "TextField",
19-
"name": "firstName1",
19+
"name": "firstNameOne",
2020
"title": "First name",
2121
"options": {
2222
"required": true
@@ -29,14 +29,14 @@
2929
"optionalText": false
3030
},
3131
"type": "TextField",
32-
"name": "middleName1",
32+
"name": "middleNameOne",
3333
"title": "Middle name",
3434
"hint": "If you have a middle name on your passport you must include it here",
3535
"schema": {}
3636
},
3737
{
3838
"type": "TextField",
39-
"name": "lastName1",
39+
"name": "lastNameOne",
4040
"title": "Surname",
4141
"options": {
4242
"required": true
@@ -56,7 +56,7 @@
5656
"components": [
5757
{
5858
"type": "UkAddressField",
59-
"name": "address1",
59+
"name": "addressOne",
6060
"title": "Address",
6161
"options": {
6262
"required": true
@@ -86,7 +86,7 @@
8686
},
8787
{
8888
"type": "TextField",
89-
"name": "firstName2",
89+
"name": "firstNameTwo",
9090
"title": "First name",
9191
"options": {
9292
"required": true
@@ -99,14 +99,14 @@
9999
"optionalText": false
100100
},
101101
"type": "TextField",
102-
"name": "middleName2",
102+
"name": "middleNameTwo",
103103
"title": "Middle name",
104104
"hint": "If you have a middle name on your passport you must include it here",
105105
"schema": {}
106106
},
107107
{
108108
"type": "TextField",
109-
"name": "lastName2",
109+
"name": "lastNameTwo",
110110
"title": "Surname",
111111
"options": {
112112
"required": true
@@ -126,7 +126,7 @@
126126
"components": [
127127
{
128128
"type": "UkAddressField",
129-
"name": "address2",
129+
"name": "addressTwo",
130130
"title": "Address",
131131
"options": {
132132
"required": false

test/form/titles.test.js

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -45,40 +45,40 @@ describe('Title and section title', () => {
4545
path: '/applicant-one',
4646
payload: {
4747
crumb: csrfToken,
48-
firstName1: 'Firstname',
49-
middleName1: '',
50-
lastName1: 'Lastname'
48+
firstNameOne: 'Firstname',
49+
middleNameOne: '',
50+
lastNameOne: 'Lastname'
5151
}
5252
},
5353
{
5454
path: '/applicant-one-address',
5555
payload: {
5656
crumb: csrfToken,
57-
address1__addressLine1: 'Richard Fairclough House',
58-
address1__addressLine2: 'Knutsford Road',
59-
address1__town: 'Warrington',
60-
address1__county: 'Cheshire',
61-
address1__postcode: 'WA4 1HT'
57+
addressOne__addressLine1: 'Richard Fairclough House',
58+
addressOne__addressLine2: 'Knutsford Road',
59+
addressOne__town: 'Warrington',
60+
addressOne__county: 'Cheshire',
61+
addressOne__postcode: 'WA4 1HT'
6262
}
6363
},
6464
{
6565
path: '/applicant-two',
6666
payload: {
6767
crumb: csrfToken,
68-
firstName2: 'Firstname',
69-
middleName2: '',
70-
lastName2: 'Lastname'
68+
firstNameTwo: 'Firstname',
69+
middleNameTwo: '',
70+
lastNameTwo: 'Lastname'
7171
}
7272
},
7373
{
7474
path: '/applicant-two-address-optional',
7575
payload: {
7676
crumb: csrfToken,
77-
address2__addressLine1: '',
78-
address2__addressLine2: '',
79-
address2__town: '',
80-
address2__county: '',
81-
address2__postcode: ''
77+
addressTwo__addressLine1: '',
78+
addressTwo__addressLine2: '',
79+
addressTwo__town: '',
80+
addressTwo__county: '',
81+
addressTwo__postcode: ''
8282
}
8383
}
8484
]) {

0 commit comments

Comments
 (0)