We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Original ticket http://trac.elgg.org/ticket/1391 on 39917955-12-18 by trac user alivin70, assigned to cash.
Elgg version: 1.6
Sometimes widget are not referred to users but to other entities.
An example is che custom_index_widgets plugin, where widgets should be referred to a site entity, not to the admin.
I propose to change the function add_widget() in engine/lib/widget.php at line 232 change:
if ($user = get_user($user_guid)) {
with
if ($user = get_entity($user_guid)) {
In my knowledge that's the only point where the $user_guid is cheked to by the guid of a ElggUser instance.
I made that change and everything works as before.
In a later clean up of the code the word "user" can be removed in order to make the code clearer.
The text was updated successfully, but these errors were encountered:
Milestone changed to Elgg 1.7.2 by cash on 40524327-07-19
Elgg 1.7.2
Sorry, something went wrong.
cash wrote on 40524331-01-01
(In [svn:6683]) Fixes #1391 - widget ownership is not limited to users
No branches or pull requests
Original ticket http://trac.elgg.org/ticket/1391 on 39917955-12-18 by trac user alivin70, assigned to cash.
Elgg version: 1.6
Sometimes widget are not referred to users but to other entities.
An example is che custom_index_widgets plugin, where widgets should be referred to a site entity, not to the admin.
I propose to change the function add_widget() in engine/lib/widget.php
at line 232 change:
with
In my knowledge that's the only point where the $user_guid is cheked to by the guid of a ElggUser instance.
I made that change and everything works as before.
In a later clean up of the code the word "user" can be removed in order to make the code clearer.
The text was updated successfully, but these errors were encountered: