-
Notifications
You must be signed in to change notification settings - Fork 501
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
[T] OssnUser::getFriends() #2149
Comments
Verify the use on all places where getFriends() used |
used on my 6.2dev without issues at: ./libraries/ossn.lib.users.php:138: |
@githubertus ❤️ |
i just noticed that it's no longer easy to get a list of latest friends ... because the relationships_id isn't part of the new query result |
Latest friends isn't ORDER BY u.guid DESC (limit 10, 20 etc) ? getFriends(id, ['order_by' => 'u.guid DESC']); ? |
I think you are right and I understand about relation id. Let me check. |
One of the way is :
|
Other method
|
I believe I need to replace latest getFriends with above method so it returns relation.* in field list as it works same. I will update method soon and test |
thank you! and I'm sorry I didn't become aware earlier of this. It's a little strange, I thought I tested it with the changes you did, but maybe my users class wasn't really up-to-date that time.... :( |
last one is working almost fine |
Nice, I will try to merge last one with current method because it's useful
to sort out with relation id
…On Tue, Jul 12, 2022, 2:16 PM Michael Zülsdorff ***@***.***> wrote:
last one is working almost fine
i have added 'distinct' => true
because otherwise you may get duplicates
—
Reply to this email directly, view it on GitHub
<#2149 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGERSTWTTE37KDJF7LFPQ3VTUZWTANCNFSM5T7PXWKQ>
.
You are receiving this because you modified the open/close state.Message
ID:
<opensource-socialnetwork/opensource-socialnetwork/issues/2149/1181523510@
github.com>
|
No description provided.
The text was updated successfully, but these errors were encountered: