- Issue created by @mayur-sose
- πΊπΈUnited States bnjmnm Ann Arbor, MI
- On initial load, the node author field has a value formatted like this:
username (USER_ID)
- After the first autosave, the field value becomes just the user id.
- Now, when the form reloads (such as a refresh that uses autosave data) there is just a 2 in the field and when the validators are called as part of the build,
\Drupal\Core\Entity\Element\EntityAutocomplete::validateEntityAutocomplete
will call$match = static::matchEntityByTitle($handler, $input, $element, $form_state, !$autocreate);
and there is no title to match, because$input
is just the user ID, not the name.
I suspect this will be addressed if we address the process that converts the field value from
username (USER_ID)
to just the id. - On initial load, the node author field has a value formatted like this:
- πΊπΈUnited States bnjmnm Ann Arbor, MI
MR has a few bits that are necessary to get this working, but nothing touching on comment #2 yet. Some way of calling
EntityAutocomplete::valueCallback
or doing the equivalent will likely be needed. - πΊπΈUnited States bnjmnm Ann Arbor, MI
@tedbow ran into what is likely this issue and narrowed it down to the changes made in π Media Library dialogs triggered from page data do not have buttons yet Active
The work done so far in the MR is probably worth keeping, even though they were not related to #3506467 (autocomplete should be adding value, not label, and details contents should be in the DOM).
The parts still not working are probabl related to the #3506467 changes so now we have a place to target our efforts
- πΊπΈUnited States bnjmnm Ann Arbor, MI
The not-yet-addressed part of this is addressed by π Get Options as buttons in Page Data form working Active , so that should be prioritized
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
This blocks β¨ [PP-1] Make link widget autocomplete work (for uri and uri-reference props) Postponed .
#6: #3515294 is in now :)
- First commit to issue fork.
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Attempted to fix the failing tests.
One was a legitimate regression in details elements. Went with a CSS solution instead of using the hidden attribute - π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Needs @bnjmnm's final sign-off.
-
larowlan β
committed e7a0c603 on 0.x authored by
bnjmnm β
Issue #3515227 by larowlan, bnjmnm, wim leers: "There are no users...
-
larowlan β
committed e7a0c603 on 0.x authored by
bnjmnm β
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Unpostponed β¨ [PP-1] Make link widget autocomplete work (for uri and uri-reference props) Postponed π₯³