- Issue created by @mingsong
- Merge request !460Issue #3441406: Source entity type and id are not populated for a View page β (Closed) created by mingsong
- last update
9 months ago 532 pass, 2 fail - Status changed to Needs review
9 months ago 7:15am 17 April 2024 - last update
9 months ago 532 pass, 2 fail - last update
9 months ago 536 pass - πΊπΈUnited States jrockowitz Brooklyn, NY
We should add some test coverage to get this committed.
- Status changed to Needs work
9 months ago 9:41am 23 April 2024 - Open on Drupal.org βCore: 10.1.4 + Environment: PHP 8.2 & MySQL 8last update
9 months ago Waiting for branch to pass - Status changed to Needs review
9 months ago 3:35am 25 April 2024 - π¦πΊAustralia mingsong π¦πΊ
It seems that the CI test failure with 'phpunit (previous minor)' is related to π Test Webform against upcoming 10.2.x on GitLab CI β and use concurrency to make tests 6x faster (on GitLab CI) Needs work .
- Merge request !502Issue #3441406 by Mingsong, jrockowitz: Source entity type and id are not populated for a View page β (Open) created by mingsong
- π¦πΊAustralia mingsong π¦πΊ
Open merge request !502 aiming for 6.3.x branch.
The merge requests are ready for review.
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
Thanks for the patch. Please add the property with constructor property promotion and include a type declaration.
In future, you can re-use the same merge request by rebasing and force-pushing its branch.
- π¦πΊAustralia mingsong π¦πΊ
Thanks @Liam for reviewing my patch.
I wonder if you meant adding property promotion in the following line?
https://git.drupalcode.org/project/webform/-/blob/6c74b9d9347a61fda605b3...
Because the original didn't have that, so I am not sure which line you referred to in #14.
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
Sorry, I misread the code. There is no constructor so there can't be constructor property promotion.
- π¦πΊAustralia mingsong π¦πΊ
It is alright. Thanks for prompt response.