Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
drupal_container() is deprecated, and all calls in the editor 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)
Comment | File | Size | Author |
---|---|---|---|
#9 | 2003616-replace-drupal_container-editor-module.patch | 3.71 KB | ddrozdik |
#3 | 2003616-replace-drupal_container-editor-module.patch | 3.7 KB | ddrozdik |
#1 | 2003616-1.patch | 3.71 KB | ebeyrent |
Comments
Comment #1
ebeyrent CreditAttribution: ebeyrent commentedComment #3
ddrozdik CreditAttribution: ddrozdik commentedComment #4
podarok#3 clean conversion
RTBC
Comment #5
ebeyrent CreditAttribution: ebeyrent commentedShould be \Drupal::service()?
Should be \Drupal::service()?
Comment #6
ddrozdik CreditAttribution: ddrozdik commentedI just think we loosely agreed on omitting in procedural code, but It's not wrong in any way though.
Comment #7
ebeyrent CreditAttribution: ebeyrent commentedWe should be consistent then, especially in the same piece of code.
Comment #8
ParisLiakos CreditAttribution: ParisLiakos commentedleading backslash is only needed for namespaced code.
any code in .module files is not
Comment #9
ddrozdik CreditAttribution: ddrozdik commentedreroll patch
Comment #10
ParisLiakos CreditAttribution: ParisLiakos commentedComment #11
webchickCommitted and pushed to 8.x. Thanks!
Comment #12
ddrozdik CreditAttribution: ddrozdik commented