Contrib.social

Reload entities in DraggableListController::submitForm()

Open on Drupal.org →
Created on 27 August 2013, over 11 years ago
Updated 24 January 2023, almost 2 years ago

Updated: Comment #0

Problem/Motivation

Follow-up for #1855402-110: Add generic weighting (tabledrag) support for config entities (DraggableListController) →
The entities are serialized with all injections

<berdir> andypost: I think what it does right now is confusing yes. and scary
<berdir> andypost: try this debug(serialize($form), $form_id); in a hook_form_alter() for a draggable form
<berdir> andypost: there's *everything* in there ;)
<berdir> request, full container, module lists, database connection, ...

Proposed resolution

TBD

Remaining tasks

TBD

User interface changes

-

API changes

-

Related Issues

#2004282: Injected dependencies and serialization hell →

📌 Task
Status

Needs work

Version

10.1 ✨

Component
Configuration entity  →

Last updated 3 days ago

Created by

🇫🇷France andypost

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

  • Needs change record

    A change record needs to be drafted before an issue is committed. Note: Change records used to be called change notifications.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Comment almost 2 years ago →
    🇮🇳India sahil.goyal

    Updating the patch against #42 trying to resolve the CCF errors.

  • Status changed to Needs review almost 2 years ago
  • Comment almost 2 years ago →
    🇮🇳India ankithashetty Karnataka, India

    Thanks, @sahil.goyal for the fix. But the patch in #43 has additional changes like StringTranslation which is out-of-scope of this issue.

    Compared the old patches in #42, #25, #20... The change in DraggableListBuilder.php affected LanguageListBuilder.php as well.

    Rerolled the old patch, and made some additional changes to the patch to fix the errors in dependent classes.

    Thanks!

  • Status changed to Needs work almost 2 years ago
  • Comment almost 2 years ago →
    🇺🇸United States smustgrave

    @ankithashetty thank you for working on this and good work!

    Brought this up on #contribute with @andypost and agree that a change record would be best.
    Also an IS update for the proposed solution.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024