Closed (fixed)
Project:
Drupal core
Version:
8.9.x-dev
Component:
views.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
10 Jun 2020 at 08:45 UTC
Updated:
29 Jun 2020 at 12:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
munish.kumar commentedComment #3
jungleThanks, @munish.kumar for picking it up, just a reminder, the comment "A view executable instance, from the loaded entity." should be updated as well.
Comment #4
munish.kumar commentedThanks @jungle for pointing these changes as well. As discussed I have updated the comment in the patch. Please review.
Comment #6
jungleA random failure, re-queued.
Comment #7
siddhant.bhosale commentedComment #8
jungleAdded a test "Tests the getView() method against a non-existent view.", but the test-only patch will pass, as by default if the return statement is missing, it returns NULL.
Comment #9
siddhant.bhosale commentedHi, I have tested the patch and looks good to be merged.
Comment #10
alexpottCommitted and pushed 19d2c39d0d to 9.1.x and e8cdb1d61f to 9.0.x and fd046df3d9 to 8.9.x. Thanks!