diff options
author | Dave Arter <davea@mysociety.org> | 2019-10-15 15:14:59 +0100 |
---|---|---|
committer | Dave Arter <davea@mysociety.org> | 2019-10-15 15:14:59 +0100 |
commit | 0eedb623219ab87b86024a10529ce2cb5c7f86a7 (patch) | |
tree | 1fbb09890f5ddfbe1991b8f4eb7e7f1590634578 | |
parent | dfe260a3f3d20ed8877b156d7daece93c771efd1 (diff) |
Restore debug toolbar on non-contact admin pages
sortable() returns undefined if no elements match, so on admin pages
that weren't the contact editing page a TypeError was being thrown by
the [0] line which resulted in the debug toolbar not being shown.
-rw-r--r-- | web/cobrands/fixmystreet/admin.js | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/web/cobrands/fixmystreet/admin.js b/web/cobrands/fixmystreet/admin.js index d3df27f33..8210f002f 100644 --- a/web/cobrands/fixmystreet/admin.js +++ b/web/cobrands/fixmystreet/admin.js @@ -89,7 +89,6 @@ $(function(){ selected_perms['permissions[' + p + ']'] = 1; }); }); - console.log(selected_perms); $perms.css('color', '#666'); $perms.find('a').css('color', '#666'); $perms.find('input').each(function() { @@ -139,15 +138,18 @@ $(function(){ $(this).parents('.js-metadata-item').remove(); }).on('change', '.js-metadata-item', updateMetadataItemTitle); - sortable('.js-metadata-items', { + var items = sortable('.js-metadata-items', { forcePlaceholderSize: true, handle: '.js-metadata-item-header-grab', placeholder: '<div class="extra-metadata-item-placeholder"></div>' - })[0].addEventListener('sortupdate', function(e) { - $(e.detail.destination.items).each(function(i){ - $(this).find('.js-sort-order input').val(i); - }); }); + if (items.length) { + items[0].addEventListener('sortupdate', function(e) { + $(e.detail.destination.items).each(function(i){ + $(this).find('.js-sort-order input').val(i); + }); + }); + } $('.js-sort-order').addClass('hidden-js'); function reloadSortableMetadataItems(){ |