Import

langcode: und
status: true
dependencies: {  }
name: test
label: 'Test browser'
display: iframe
display_configuration:
  width: 650
  height: 500
  link_text: 'Select entities'
selection_display: no_display
selection_display_configuration: {  }
widget_selector: single
widget_selector_configuration: {  }
widgets:
  8dcc81c7-e24c-4d0b-85a1-bbd03a56dbe3:
    id: upload
    label: 'View widget nr. 1'
    weight: 0
    uuid: 8dcc81c7-e24c-4d0b-85a1-bbd03a56dbe3
    settings: {  }

and try to export. Notice "location_url" default setting appearing on the wrong place.

Comments

slashrsm’s picture

Issue summary: View changes
slashrsm’s picture

Issue tags: +Media Initiative, +sprint, +Novice

We need to check if this is still relevant.

slashrsm’s picture

Issue tags: -sprint +D8Media
slashrsm’s picture

Status: Active » Fixed

I can't reproduce this bug any more.

Status: Fixed » Closed (fixed)

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

boobaa’s picture

Assigned: Unassigned » boobaa
Status: Closed (fixed) » Active

Widget configurations are not saved because their values are one level deeper in $form_state in \Drupal\entity_browser\PluginConfigurationFormTrait::submitConfigurationForm(). (Currently there's only support for the upload widget's configuration, but that one doesn't work either because of this, and #2671810: Implement configuration form for View widget won't work either before this gets fixed.)

boobaa’s picture

Assigned: boobaa » Unassigned
Status: Active » Needs review

Pull Request is sent on GitHub.

slashrsm’s picture

Status: Needs review » Needs work

Few comments on pull.

EDIT: Wrong issue.

  • slashrsm committed 10c2fa9 on 8.x-1.x authored by Boobaa
    Issue #2366315 by Boobaa: Widget configuration is not saved correctly
    
slashrsm’s picture

Status: Needs work » Fixed
slashrsm’s picture

Merged. Thanks!

Status: Fixed » Closed (fixed)

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