Files: 
CommentFileSizeAuthor
#7 editor-remove-global-user-from-editor-module-2062171-7.patch713 bytesm1r1k
PASSED: [[SimpleTest]]: [MySQL] 58,138 pass(es).
[ View ]
#1 2062171-1.patch851 bytesGarbar
PASSED: [[SimpleTest]]: [MySQL] 57,893 pass(es).
[ View ]

Comments

Assigned:Unassigned» Garbar
Status:Active» Needs review
StatusFileSize
new851 bytes
PASSED: [[SimpleTest]]: [MySQL] 57,893 pass(es).
[ View ]

need reviews

Status:Needs review» Reviewed & tested by the community

Great — thanks! :)

Status:Reviewed & tested by the community» Needs work

Status:Needs work» Postponed

Title:Remove calls to deprecated global $user in editor moduleRemove calls to deprecated global $user in favor of current user service in editor module
Status:Postponed» Active

Garbar, #2062151: Create a current user service to ensure that current account is always available was finished and committed two days ago. Would you like to pick this up again? :)

Issue tags:+php-novice

.

Status:Active» Needs review
StatusFileSize
new713 bytes
PASSED: [[SimpleTest]]: [MySQL] 58,138 pass(es).
[ View ]

Status:Needs review» Reviewed & tested by the community

Looks nice to go

+++ b/core/modules/editor/lib/Drupal/editor/Plugin/InPlaceEditor/Editor.php
@@ -68,7 +68,7 @@ protected function textFormatHasTransformationFilters($format_id) {
+    $user = \Drupal::currentUser();
...
     $manager = \Drupal::service('plugin.manager.editor');

Not sure why the dependencies are not injected here but it was before the patch

This class is from before the proper interfaces for that were finalized IIRC. I'd like to see the conversion happening here, but you're right, that's out of scope for this particular issue.

RTBC+1

Status:Reviewed & tested by the community» Fixed

Committed 51811d8 and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.