Bugfix/ie11 fix rebased on refactor work#7
Conversation
wingleung
commented
Dec 19, 2018
wingleung
commented
- IE11 bugfix
- basic styling index.html
| var formDataUrlParams = 'exp=' + inputExp.value + '&subject=' + subjectInput.value; | ||
| domains.querySelectorAll('input').forEach(function(input) { | ||
| if (input.checked) { | ||
| formDataUrlParams = formDataUrlParams + '&' + input.name + '=' + input.value |
There was a problem hiding this comment.
Moet hier geen escaping gebeuren?
There was a problem hiding this comment.
Je zou knn opperen van wel, maar bogus input moet toch op de backend gechecked worden. frontend input is niet te vertrouwen :)
There was a problem hiding this comment.
De backend checkt de data die aangeleverd wordt; dat is het probleem niet.
Het probleem is dat de gebruiker rare foutmeldingen gaat krijgen als hij een "ongewoon" karakter in zijn Subject tikt. Met spatie ook als ongewoon...
There was a problem hiding this comment.
@nielslaukens wordt automatisch gedaan op lijn 164 door new URLSearchParams(formDataUrlParams)
There was a problem hiding this comment.
I'm pretty sure dat een ingevulde subject van foo&dummy=bar de boel om zeep gaat helpen...
There was a problem hiding this comment.
3 participants
