Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Problem
This is a related issue to https://www.drupal.org/node/2392845. The patch in #69 removed the auto creation code for computed field from ItemList class.
This patch will lead to PHP error as we have function called to get the first item from an ItemList, but the list is empty caused by above change in core patch:
Proposal
If the patch gets in core, we will need to rework on the purge function to remove the pathauto state.
Or we need to rework on the core patch to add the auto creation of the conputered field item back.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | Respect_removal_of_auto_creation_of_the_first_item_in_computered_field-2883191-2.patch | 790 bytes | rli |
| Screen Shot 2017-06-02 at 12.34.39 pm.png | 113.71 KB | rli | |
| Screen Shot 2017-06-02 at 12.30.45 pm.png | 155.83 KB | rli |
Comments
Comment #2
rliTried to make it work with the core patch. Still need to work on it to clean up the purge function etc. But need a decision on this situation first.
Comment #3
rliComment #4
seanbWas breaking my head over this, was using the core patch you mentioned. Thank you!
Comment #5
adammaloneComment #6
adammaloneComment #7
amateescu commentedI've no idea how Pathauto works internally, but my suggestion is to move that
purge()method to the field item list level, instead of having it at the field item level as it is now.Comment #8
berdirThe final core patch looked quite different but still causes some issues, see #2945734: Fix automated test failures in 8.x-1.x.