X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=mod%2Ftagrm.php;h=267ee718cc04280ab01a5e163dbf8ba42c7cdeab;hb=6e2880c6799e5c98e41ea40e1bdbac7bfe6ae326;hp=b9991d68da5b5ff0ea4c9a7a360dfe2fe1464ed8;hpb=9e3bae5caa6725737c2a1221a6c499e06ea0077c;p=friendica.git diff --git a/mod/tagrm.php b/mod/tagrm.php index b9991d68da..267ee718cc 100644 --- a/mod/tagrm.php +++ b/mod/tagrm.php @@ -6,80 +6,97 @@ use Friendica\App; use Friendica\Content\Text\BBCode; use Friendica\Core\L10n; -use Friendica\Core\System; -use Friendica\Database\DBM; +use Friendica\Database\DBA; +use Friendica\DI; use Friendica\Model\Item; +use Friendica\Model\Term; +use Friendica\Util\Strings; -function tagrm_post(App $a) { - +function tagrm_post(App $a) +{ if (!local_user()) { - goaway(System::baseUrl() . '/' . $_SESSION['photo_return']); + DI::baseUrl()->redirect($_SESSION['photo_return']); } - if ((x($_POST,'submit')) && ($_POST['submit'] === L10n::t('Cancel'))) { - goaway(System::baseUrl() . '/' . $_SESSION['photo_return']); + if (!empty($_POST['submit']) && ($_POST['submit'] === L10n::t('Cancel'))) { + DI::baseUrl()->redirect($_SESSION['photo_return']); } - $tag = ((x($_POST,'tag')) ? hex2bin(notags(trim($_POST['tag']))) : ''); - $item = ((x($_POST,'item')) ? intval($_POST['item']) : 0 ); + $tags = []; + foreach ($_POST['tag'] ?? [] as $tag) { + $tags[] = hex2bin(Strings::escapeTags(trim($tag))); + } - $r = q("SELECT * FROM `item` WHERE `id` = %d AND `uid` = %d LIMIT 1", - intval($item), - intval(local_user()) - ); + $item_id = $_POST['item'] ?? 0; + update_tags($item_id, $tags); + info(L10n::t('Tag(s) removed') . EOL); - if (!DBM::is_result($r)) { - goaway(System::baseUrl() . '/' . $_SESSION['photo_return']); - } + DI::baseUrl()->redirect($_SESSION['photo_return']); + // NOTREACHED +} - $arr = explode(',', $r[0]['tag']); - for ($x = 0; $x < count($arr); $x ++) { - if ($arr[$x] === $tag) { - unset($arr[$x]); - break; - } +/** + * Updates tags from an item + * + * @param $item_id + * @param $tags array + * @throws Exception + */ +function update_tags($item_id, $tags){ + if (empty($item_id) || empty($tags)){ + return; } - $tag_str = implode(',',$arr); + $item = Item::selectFirst(['tag'], ['id' => $item_id, 'uid' => local_user()]); + if (!DBA::isResult($item)) { + return; + } - Item::update(['tag' => $tag_str], ['id' => $item]); + $old_tags = explode(',', $item['tag']); - info(L10n::t('Tag removed') . EOL ); - goaway(System::baseUrl() . '/' . $_SESSION['photo_return']); + foreach ($tags as $new_tag) { + foreach ($old_tags as $index => $old_tag) { + if (strcmp($old_tag, $new_tag) == 0) { + unset($old_tags[$index]); + break; + } + } + } - // NOTREACHED + $tag_str = implode(',', $old_tags); + Term::insertFromTagFieldByItemId($item_id, $tag_str); } - - -function tagrm_content(App $a) { - +function tagrm_content(App $a) +{ $o = ''; if (!local_user()) { - goaway(System::baseUrl() . '/' . $_SESSION['photo_return']); + DI::baseUrl()->redirect($_SESSION['photo_return']); // NOTREACHED } - $item = (($a->argc > 1) ? intval($a->argv[1]) : 0); - if (!$item) { - goaway(System::baseUrl() . '/' . $_SESSION['photo_return']); - // NOTREACHED + if ($a->argc == 3) { + update_tags($a->argv[1], [Strings::escapeTags(trim(hex2bin($a->argv[2])))]); + DI::baseUrl()->redirect($_SESSION['photo_return']); } - $r = q("SELECT * FROM `item` WHERE `id` = %d AND `uid` = %d LIMIT 1", - intval($item), - intval(local_user()) - ); + $item_id = (($a->argc > 1) ? intval($a->argv[1]) : 0); + if (!$item_id) { + DI::baseUrl()->redirect($_SESSION['photo_return']); + // NOTREACHED + } - if (!DBM::is_result($r)) { - goaway(System::baseUrl() . '/' . $_SESSION['photo_return']); + $item = Item::selectFirst(['tag'], ['id' => $item_id, 'uid' => local_user()]); + if (!DBA::isResult($item)) { + DI::baseUrl()->redirect($_SESSION['photo_return']); } - $arr = explode(',', $r[0]['tag']); + $arr = explode(',', $item['tag']); - if (!count($arr)) { - goaway(System::baseUrl() . '/' . $_SESSION['photo_return']); + + if (empty($item['tag'])) { + DI::baseUrl()->redirect($_SESSION['photo_return']); } $o .= '
' . L10n::t('Select a tag to remove: ') . '
'; $o .= ''; return $o; - }