-
Notifications
You must be signed in to change notification settings - Fork 333
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
Some more deprecated function calls are replaced #14157
Conversation
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.
Good job, it looks like BaseModule
could use a UserSession
dependency, but it will take a fair bit of work to get it in given the inheritance graph.
src/Module/Contact/Profile.php
Outdated
@@ -24,6 +24,7 @@ | |||
use Friendica\App; | |||
use Friendica\BaseModule; | |||
use Friendica\Contact\LocalRelationship; | |||
use Friendica\Contact\LocalRelationship\Entity\LocalRelationship as EntityLocalRelationship; |
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.
Please rename the alias to LocalRelationshipEntity
to satisfy the English order of qualifier.
use Friendica\Contact\LocalRelationship\Entity\LocalRelationship as EntityLocalRelationship; | |
use Friendica\Contact\LocalRelationship\Entity\LocalRelationship as LocalRelationshipEntity; |
Yeah, there will be some work to get rid of |
* @deprecated 2022.03 | ||
* @see IHandleUserSessions::getLocalUserNickname() | ||
*/ | ||
public function getLoggedInUserNickname(): 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.
This seems to be used by the calendar, see #14025 (comment)
Several more function calls are now replaced and some legacy functions are completely removed now.