-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update directory commands to use preferred names #5455
Update directory commands to use preferred names #5455
Conversation
c487c92
to
53b53ae
Compare
Adding the display name and replacing output of the first and last name with the preferred version as well as saving the preferred version for new students synced into a school to match the behavior of the user sync command.
53b53ae
to
e08d308
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this the new convention for phpDoc?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I remove the doc expressions when the property is typed, don't like to keep them both so we don't have to keep them in sync.
@@ -51,7 +51,7 @@ public function testFindByCampusId(): void | |||
$this->config->shouldReceive('get')->once()->with('ldap_directory_campus_id_property')->andReturn('campusId'); | |||
$this->ldapManager->shouldReceive('search')->with('(campusId=1234)')->andReturn([['id' => 1]]); | |||
|
|||
$result = $this->obj->findByCampusId(1234); | |||
$result = $this->obj->findByCampusId('1234'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was the reason for changing this from a number to a string?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I typed the parameter because it's always a string in production. Just had to update the test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bueno!
Adding the display name and replacing output of the first and last name with the preferred version as well as saving the preferred version for new students synced into a school to match the behavior of the user sync command.
Refs #5453