mirror of
https://github.com/friendica/friendica
synced 2025-02-23 06:38:18 +00:00
extract Item::prepareItemD() into own method
This commit is contained in:
parent
2a7f7af02c
commit
e93ff4243f
1 changed files with 24 additions and 17 deletions
|
@ -882,23 +882,7 @@ class Item
|
||||||
$item['network'] = trim(($item['network'] ?? '') ?: Protocol::PHANTOM);
|
$item['network'] = trim(($item['network'] ?? '') ?: Protocol::PHANTOM);
|
||||||
}
|
}
|
||||||
|
|
||||||
$uid = intval($item['uid']);
|
$item = self::prepareItemData($item, (bool) $notify);
|
||||||
|
|
||||||
$item['guid'] = self::guid($item, $notify);
|
|
||||||
$item['uri'] = substr(trim($item['uri'] ?? '') ?: self::newURI($item['guid']), 0, 255);
|
|
||||||
|
|
||||||
// Store URI data
|
|
||||||
$item['uri-id'] = ItemURI::insert(['uri' => $item['uri'], 'guid' => $item['guid']]);
|
|
||||||
|
|
||||||
// Backward compatibility: parent-uri used to be the direct parent uri.
|
|
||||||
// If it is provided without a thr-parent, it probably is the old behavior.
|
|
||||||
if (empty($item['thr-parent']) || empty($item['parent-uri'])) {
|
|
||||||
$item['thr-parent'] = trim($item['thr-parent'] ?? $item['parent-uri'] ?? $item['uri']);
|
|
||||||
$item['parent-uri'] = $item['thr-parent'];
|
|
||||||
}
|
|
||||||
|
|
||||||
$item['thr-parent-id'] = ItemURI::getIdByURI($item['thr-parent']);
|
|
||||||
$item['parent-uri-id'] = ItemURI::getIdByURI($item['parent-uri']);
|
|
||||||
|
|
||||||
// Store conversation data
|
// Store conversation data
|
||||||
$source = $item['source'] ?? '';
|
$source = $item['source'] ?? '';
|
||||||
|
@ -928,6 +912,8 @@ class Item
|
||||||
|
|
||||||
$defined_permissions = isset($item['allow_cid']) && isset($item['allow_gid']) && isset($item['deny_cid']) && isset($item['deny_gid']) && isset($item['private']);
|
$defined_permissions = isset($item['allow_cid']) && isset($item['allow_gid']) && isset($item['deny_cid']) && isset($item['deny_gid']) && isset($item['private']);
|
||||||
|
|
||||||
|
$uid = intval($item['uid']);
|
||||||
|
|
||||||
// Communities aren't working with the Diaspora protocol
|
// Communities aren't working with the Diaspora protocol
|
||||||
if (($uid != 0) && ($item['network'] == Protocol::DIASPORA)) {
|
if (($uid != 0) && ($item['network'] == Protocol::DIASPORA)) {
|
||||||
$user = User::getById($uid, ['account-type']);
|
$user = User::getById($uid, ['account-type']);
|
||||||
|
@ -1425,6 +1411,27 @@ class Item
|
||||||
return $post_user_id;
|
return $post_user_id;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private static function prepareItemData(array $item, bool $notify): array
|
||||||
|
{
|
||||||
|
$item['guid'] = self::guid($item, $notify);
|
||||||
|
$item['uri'] = substr(trim($item['uri'] ?? '') ?: self::newURI($item['guid']), 0, 255);
|
||||||
|
|
||||||
|
// Store URI data
|
||||||
|
$item['uri-id'] = ItemURI::insert(['uri' => $item['uri'], 'guid' => $item['guid']]);
|
||||||
|
|
||||||
|
// Backward compatibility: parent-uri used to be the direct parent uri.
|
||||||
|
// If it is provided without a thr-parent, it probably is the old behavior.
|
||||||
|
if (empty($item['thr-parent']) || empty($item['parent-uri'])) {
|
||||||
|
$item['thr-parent'] = trim($item['thr-parent'] ?? $item['parent-uri'] ?? $item['uri']);
|
||||||
|
$item['parent-uri'] = $item['thr-parent'];
|
||||||
|
}
|
||||||
|
|
||||||
|
$item['thr-parent-id'] = ItemURI::getIdByURI($item['thr-parent']);
|
||||||
|
$item['parent-uri-id'] = ItemURI::getIdByURI($item['parent-uri']);
|
||||||
|
|
||||||
|
return $item;
|
||||||
|
}
|
||||||
|
|
||||||
private static function validateItemData(array $item): array
|
private static function validateItemData(array $item): array
|
||||||
{
|
{
|
||||||
$item['wall'] = intval($item['wall'] ?? 0);
|
$item['wall'] = intval($item['wall'] ?? 0);
|
||||||
|
|
Loading…
Add table
Reference in a new issue