Attaching a patch that fixes them.

#8 2022207-user-to-userinterface-8.patch5.23 KBpcambra
PASSED: [[SimpleTest]]: [MySQL] 58,672 pass(es).
[ View ]
#1 2022207-user-to-userinterface-d8.patch5.86 KBpcambra
PASSED: [[SimpleTest]]: [MySQL] 57,728 pass(es).
[ View ]
user-to-userinterface-d8.patch6.1 KBpcambra
FAILED: [[SimpleTest]]: [MySQL] 57,376 pass(es), 10 fail(s), and 5 exception(s).
[ View ]


Title:User test vars declared as \Drupal\user\Plugin\Core\Entity\User where it should be \Drupal\user\Plugin\Core\Entity\UserInterfaceUser test vars declared as \Drupal\user\Plugin\Core\Entity\User where it should be \Drupal\user\UserInterface
new5.86 KB
PASSED: [[SimpleTest]]: [MySQL] 57,728 pass(es).
[ View ]

It should be actually \Drupal\user\UserInterface

Component:documentation» user.module

This seems reasonable to me, and the documentation syntax is correct. Our documentation/coding standards do say to use interfaces in favor of classes whenever possible... But maybe one of the maintainers of the user module can comment on the accuracy of doing that here?

Looks good to me.

There are a lot more $account and $user variables, arguments which aren't documented at all yet or possibly still as Drupal\user\User and all kinds of broken things. I'm not sure if we should fix that here too, that might end up being a bigger patch or if we should simply make incremental improvements (fine with me, also less conflicts with other, more important patches)

Issue tags:+Novice


Component:user.module» documentation
Assigned:Unassigned» jhodgdon
Status:Needs review» Reviewed & tested by the community

Let's leave this issue as it is now. Thanks for the review, Berdir -- I'll get this one committed soon.

Regarding other spots... If someone wants to file a "meta" issue with a title something like:

[meta] User variables should be documented as UserInterface not Plugin\Core\User

and reference this as a sub-issue, that would not be a bad idea. As a bonus, a grep through core for references to Drupal\user\Plugin\Core\Entity\User in documentation would help us figure out the scope.

Assigned:jhodgdon» Unassigned
Status:Reviewed & tested by the community» Needs work
Issue tags:+Needs reroll

The patch no longer applies in

Status:Needs work» Needs review
Issue tags:-Needs reroll
new5.23 KB
PASSED: [[SimpleTest]]: [MySQL] 58,672 pass(es).
[ View ]

Here we go

Status:Needs review» Fixed

Thanks! Committed to 8.x.

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