Skip to content

Commit 751a5df

Browse files
authored
Merge pull request #2145 from kobotoolbox/2137-nested-repeat-submission-detail-crash
Nested repeat submission detail crash - quickfix
2 parents e05c088 + ddd1a9c commit 751a5df

File tree

3 files changed

+11
-10
lines changed

3 files changed

+11
-10
lines changed

jsapp/js/components/modalForms/submission.es6

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -263,6 +263,13 @@ class Submission extends React.Component {
263263
case 'video':
264264
return this.renderAttachment(submissionValue, q.type);
265265
break;
266+
case 'begin_repeat':
267+
const list = submissionValue.map((r) => {
268+
const stringified = JSON.stringify(r);
269+
return <li key={stringified}>{stringified}</li>
270+
});
271+
return <ul>{list}</ul>
272+
break;
266273
default:
267274
return submissionValue;
268275
break;

jsapp/xlform/src/view.row.coffee

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ $acceptedFilesView = require './view.acceptedFiles'
1313
$viewRowDetail = require './view.rowDetail'
1414
renderKobomatrix = require('js/formbuild/renderInBackbone').renderKobomatrix
1515
_t = require('utils').t
16+
alertify = require 'alertifyjs'
1617

1718
module.exports = do ->
1819
class BaseRowView extends Backbone.View
@@ -193,7 +194,7 @@ module.exports = do ->
193194
$appearanceField.find('input:checkbox').prop('checked', false)
194195
appearanceModel = @model.get('appearance')
195196
if appearanceModel.getValue()
196-
@surveyView.ngScope.miscUtils.alert(_t("You can't display nested groups on the same screen - the setting has been removed from the parent group"))
197+
alertify.warning(_t("You can't display nested groups on the same screen - the setting has been removed from the parent group"))
197198
appearanceModel.set('value', '')
198199

199200
@model.on 'remove', (row) =>

jsapp/xlform/src/view.surveyApp.coffee

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ $rowView = require './view.row'
1111
$baseView = require './view.pluggedIn.backboneView'
1212
$viewUtils = require './view.utils'
1313
_t = require('utils').t
14+
alertify = require 'alertifyjs'
1415

1516
module.exports = do ->
1617
surveyApp = {}
@@ -681,16 +682,8 @@ module.exports = do ->
681682
enketoServer: window.koboConfigs?.enketoServer or false
682683
enketoPreviewUri: window.koboConfigs?.enketoPreviewUri or false
683684
onSuccess: => @onEscapeKeydown = $viewUtils.enketoIframe.close
684-
onError: (message, opts)=> @alert message, opts
685+
onError: (message)=> alertify.error(message)
685686
return
686-
687-
alert: (message, opts={}) ->
688-
title = opts.title or 'Error'
689-
$('.alert-modal').html(message).dialog('option', {
690-
title: title,
691-
width: 500,
692-
dialogClass: 'surveyapp__alert'
693-
}).dialog 'open'
694687
downloadButtonClick: (evt)->
695688
# Download = save a CSV file to the disk
696689
surveyCsv = @survey.toCSV()

0 commit comments

Comments
 (0)