Merge pull request #6746 from nupplaphil/issue/6338-notices
Fixing notices
This commit is contained in:
commit
efe358c617
2 changed files with 25 additions and 15 deletions
|
@ -19,10 +19,10 @@ use Friendica\Model\Contact;
|
||||||
use Friendica\Model\Group;
|
use Friendica\Model\Group;
|
||||||
use Friendica\Model\Item;
|
use Friendica\Model\Item;
|
||||||
use Friendica\Model\Profile;
|
use Friendica\Model\Profile;
|
||||||
|
use Friendica\Module\Objects;
|
||||||
use Friendica\Protocol\ActivityPub;
|
use Friendica\Protocol\ActivityPub;
|
||||||
use Friendica\Protocol\DFRN;
|
use Friendica\Protocol\DFRN;
|
||||||
use Friendica\Util\Strings;
|
use Friendica\Util\Strings;
|
||||||
use Friendica\Module\Objects;
|
|
||||||
|
|
||||||
function display_init(App $a)
|
function display_init(App $a)
|
||||||
{
|
{
|
||||||
|
@ -283,22 +283,25 @@ function display_content(App $a, $update = false, $update_uid = 0)
|
||||||
$is_remote_contact = false;
|
$is_remote_contact = false;
|
||||||
$item_uid = local_user();
|
$item_uid = local_user();
|
||||||
|
|
||||||
$parent = Item::selectFirst(['uid'], ['uri' => $item_parent_uri, 'wall' => true]);
|
if (isset($item_parent_uri)) {
|
||||||
if (DBA::isResult($parent)) {
|
$parent = Item::selectFirst(['uid'], ['uri' => $item_parent_uri, 'wall' => true]);
|
||||||
$a->profile['uid'] = defaults($a->profile, 'uid', $parent['uid']);
|
if (DBA::isResult($parent)) {
|
||||||
$a->profile['profile_uid'] = defaults($a->profile, 'profile_uid', $parent['uid']);
|
$a->profile['uid'] = defaults($a->profile, 'uid', $parent['uid']);
|
||||||
$is_remote_contact = Contact::isFollower(remote_user(), $a->profile['profile_uid']);
|
$a->profile['profile_uid'] = defaults($a->profile, 'profile_uid', $parent['uid']);
|
||||||
}
|
$is_remote_contact = Contact::isFollower(remote_user(), $a->profile['profile_uid']);
|
||||||
|
|
||||||
if ($is_remote_contact) {
|
if ($is_remote_contact) {
|
||||||
$cdata = Contact::getPublicAndUserContacID(remote_user(), $a->profile['profile_uid']);
|
$cdata = Contact::getPublicAndUserContacID(remote_user(), $a->profile['profile_uid']);
|
||||||
if (!empty($cdata['user'])) {
|
if (!empty($cdata['user'])) {
|
||||||
$groups = Group::getIdsByContactId($cdata['user']);
|
$groups = Group::getIdsByContactId($cdata['user']);
|
||||||
$remote_cid = $cdata['user'];
|
$remote_cid = $cdata['user'];
|
||||||
$item_uid = $parent['uid'];
|
$item_uid = $parent['uid'];
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
$page_contact = DBA::selectFirst('contact', [], ['self' => true, 'uid' => $a->profile['uid']]);
|
$page_contact = DBA::selectFirst('contact', [], ['self' => true, 'uid' => $a->profile['uid']]);
|
||||||
if (DBA::isResult($page_contact)) {
|
if (DBA::isResult($page_contact)) {
|
||||||
$a->page_contact = $page_contact;
|
$a->page_contact = $page_contact;
|
||||||
|
|
|
@ -505,8 +505,15 @@ class PortableContact
|
||||||
$last_updated = "";
|
$last_updated = "";
|
||||||
|
|
||||||
foreach ($entries as $entry) {
|
foreach ($entries as $entry) {
|
||||||
$published = DateTimeFormat::utc($xpath->query('atom:published/text()', $entry)->item(0)->nodeValue);
|
$published_item = $xpath->query('atom:published/text()', $entry)->item(0);
|
||||||
$updated = DateTimeFormat::utc($xpath->query('atom:updated/text()' , $entry)->item(0)->nodeValue);
|
$updated_item = $xpath->query('atom:updated/text()' , $entry)->item(0);
|
||||||
|
$published = isset($published_item->nodeValue) ? DateTimeFormat::utc($published_item->nodeValue) : null;
|
||||||
|
$updated = isset($updated_item->nodeValue) ? DateTimeFormat::utc($updated_item->nodeValue) : null;
|
||||||
|
|
||||||
|
if (!isset($published) || !isset($updated)) {
|
||||||
|
Logger::notice('Invalid entry for XPath.', ['entry' => $entry, 'profile' => $profile]);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
if ($last_updated < $published) {
|
if ($last_updated < $published) {
|
||||||
$last_updated = $published;
|
$last_updated = $published;
|
||||||
|
|
Loading…
Reference in a new issue