@Isabelle: I can't confirm the issues 1-3 you are describing. Which version of Tidypics are you using for testing on Elgg 1.9? Is it 1.9.1beta13? The $vars["url"] deprecated issue can definitely not be caused by Tidypics, because it's not used within the code of Tidypics anymore anywhere. Do you have any other 3rd party plugins enabled? Other plugins might result in the issues you are describing if they are interferring with the views and pages of the Tidypics plugins. Please try again with no other 3rd party plugins enabled. Does the same problem happen again.
Issue 4: the popup will only show the albums of the logged in user - no exception for admins. I don't think it would make sense to display all albums of the site for admins because the dropdown menu might contain a very high number of albums then without any chance of seeing who is the album owner. If you want to add images to an album of another user please go this the page of this album and use "Upload photos to this album".
Any feedback regarding the upgrade script - if you are indeed testing beta13 on Elgg 1.9 and have already existing Tidypics content on this test installation?
@iionly : sorry for the wrong reporting on error 1. I was just going to post that, before you made this post.
I am using https://github.com/iionly/Elgg_1.9_tidypics version number 2014040201.
I haven't tested it on an upgrade installation. I dont have any big installation of 1.8 at the moment.
#2: images of a certain user page is called either by http://site.url/photos/siteimagesowner or http://site.url/photos/siteimagesowner/<user_guid>
If there's a user guid in the url (in case you want to see the images of this other user), the guid is retrieved in the pagehandler function in start.php:
if (isset($page[1])) {
set_input('guid', $page[1]);
}
Then in siteimagesowner.php:
$owner_guid = get_input('guid');
$owner = get_entity($owner_guid);
if(!$owner || !(elgg_instanceof($owner, 'user'))) {
$owner = elgg_get_logged_in_user_entity();
$filter = elgg_view('filter_override/siteimages', array('selected' => 'mine'));
} else {
$filter = '';
}
Here the user's guid is retrieved. If there's no guid in input or it isn't a valid user guid the owner (page viewer) is assumed to be the currently logged in user in session. The page has indeed no "owner" retrieved by elgg_get_pageowner* or set by elgg_set_pageowner*.
info@elgg.org
Security issues should be reported to security@elgg.org!
©2014 the Elgg Foundation
Elgg is a registered trademark of Thematic Networks.
Cover image by Raül Utrera is used under Creative Commons license.
Icons by Flaticon and FontAwesome.