1. Goto admin/structure/types/manage/article/display/default
2. Select a layout
3. Save

The admin/structure/types/manage/article/display/default page will got this Javascript issue:
drupal.js?v=8.3.0-dev:59 Uncaught TypeError: rowHandlers[data.rowHandler] is not a constructor
(anonymous function) @ field_ui.js?v=8.3.0-dev:114
each @ jquery.min.js?v=2.2.3:2
each @ jquery.min.js?v=2.2.3:2
attach @ field_ui.js?v=8.3.0-dev:105
(anonymous function) @ field_ui.js?v=8.3.0-dev:75
each @ jquery.min.js?v=2.2.3:2
each @ jquery.min.js?v=2.2.3:2
attach @ field_ui.js?v=8.3.0-dev:74
Drupal.attachBehaviors @ drupal.js?v=8.3.0-dev:160
(anonymous function) @ drupal.init.js?v=8.3.0-dev:17
t @ ready.min.js?v=1.0.8:4

CommentFileSizeAuthor
#2 js_uncaught-2839309-2.patch605 bytesalex_optim

Comments

jian he created an issue. See original summary.

alex_optim’s picture

alex_optim
Location Tallinn
commented
StatusFileSize
new js_uncaught-2839309-2.patch605 bytes

Fixed error js.
Create patch. Please check.

alex_optim’s picture

alex_optim
Location Tallinn
commented
Status: Active » Needs review
aspilicious’s picture

aspilicious commented
Issue tags: +drupalmountaincamp

I need to verify this.

swentel’s picture

swentel commented
Status: Needs review » Needs work 
+++ b/includes/field_ui.inc
@@ -211,7 +211,7 @@ function ds_field_ui_regions($form, FormStateInterface $form_state) {
+      $row['#js_settings'] = array('rowHandler' => 'fireld');

this can't be right ?

aspilicious’s picture

aspilicious commented

The patch should be "field".
But that's not the real problem, the real problem is that ds.admin.js contains code to add a custom row handler but that one is commented. (probably because it doesn't work)

The region column is also missing in the UI...

aspilicious’s picture

aspilicious commented

Never mind, I think I can fix it... :)

aspilicious’s picture

aspilicious commented
Status: Needs work » Fixed

In the end the patch was ok, (after I changed fireld to field).
I removed our custom handling as it isn't needed anymore.

I had some troubles with testing because of cached javascript.

Thnx for all the help!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.