Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Updated: Comment #N
Problem/Motivation
The changes to values in massageFormValues() do not persist
Proposed resolution
todo - fix it
Remaining tasks
todo
User interface changes
API changes
none
Original report by @pwolanin
related: https://drupal.org/node/2226493
Comments
Comment #1
Wim LeersComment #2
pwolanin CreditAttribution: pwolanin commentedComment #3
plopescHello,
I tried to reproduce this bug to work on it, maybe I'm missing something :)
In
NumberWidget
I added the code below:Then I created a node with an integer field, and entering 3 as value in the widget, the value stored in the DB and displayed in the node page was 6. So I think it is working as expected.
Could you point me how to reproduce this bug or tell me what I'm missing, please?
I will try to work on it then.
Thank you!
Comment #4
BerdirYes, can't confirm this either, we are using this in a custom widget to change the submitted values, works just fine.
Moving this out of the active/major bug queue.
Comment #5
yched CreditAttribution: yched commentedFYI - Not sure that's related, but @Berdir bumped into this : #2326533: The structure of values received in WidgetInterface::massageFormValues() varies depending on #access TRUE/FALSE
Comment #6
swentel CreditAttribution: swentel commentedThey sound related. Wim hit this during dev days in Szeged and I've been looking along at that time.
Comment #14
pameeela CreditAttribution: pameeela commentedThanks everyone for your contributions on this issue.
As part of the Bug smash initiative, we are triaging issues that are marked 'Postponed (maintainer needs more info)'. This issue is more than 12 months old and no additional information has been provided since the initial replies indicating that contributors were not able to reproduce. As such I am marking it 'Closed (cannot reproduce)'.
If anyone believes this issue should not be closed, please provide specific steps to reproduce when reopening it.