drupal_container() is deprecated, and all calls in the search module need to be replaced with Drupal::service(), except for where the module_handler service is requested, which needs to be replaced with Drupal::moduleHandler() (see #1957154: Replace calls to drupal_container()->get('module_handler') service with Drupal::moduleHandler())
| Comment | File | Size | Author |
|---|---|---|---|
| #20 | 2011126-20-search.patch | 2.42 KB | dcam |
| #15 | search-2011126-15.patch | 2.41 KB | kgoel |
| #13 | search-2011126-13.patch | 2.42 KB | kgoel |
| #9 | search-2011126-9.patch | 2.42 KB | kgoel |
| #9 | interdiff.txt | 2.42 KB | kgoel |
Comments
Comment #1
kgoel commentedComment #2
kgoel commentedComment #4
ddrozdik commentedThe same problem I have described in the task #2011094-4: Replace drupal_container() with Drupal::service() in the layout module
Comment #5
kgoel commentedComment #7
kgoel commentedComment #9
kgoel commentedComment #11
kgoel commented#9: search-2011126-9.patch queued for re-testing.
Comment #12
dcam commented#9 does not apply locally.
Comment #13
kgoel commentedComment #15
kgoel commentedComment #17
dcam commented#15: search-2011126-15.patch queued for re-testing.
Comment #19
kgoel commentedI am not sure why patch failed as it appears to be simple change.
Comment #20
dcam commentedRerolled #9.
Comment #21
ddrozdik commentedApplied, looks good.
Comment #22
catchCommitted/pushed to 8.x, thanks!