aboutsummaryrefslogtreecommitdiffstats
path: root/mod
diff options
context:
space:
mode:
authorRedMatrix <info@friendica.com>2014-10-28 22:13:33 +1100
committerRedMatrix <info@friendica.com>2014-10-28 22:13:33 +1100
commit4891eb5b0111afb15617e125a190ded8d39e3db2 (patch)
tree43aeaa5714b455730eb77de4419864e0b7e21a00 /mod
parent9cc76cb33da489c01731ecb5195f8bb3d51ce513 (diff)
parentec8bab7784b531299150ddb2c18ede40d5c5df6c (diff)
downloadvolse-hubzilla-4891eb5b0111afb15617e125a190ded8d39e3db2.tar.gz
volse-hubzilla-4891eb5b0111afb15617e125a190ded8d39e3db2.tar.bz2
volse-hubzilla-4891eb5b0111afb15617e125a190ded8d39e3db2.zip
Merge pull request #668 from git-marijus/master
change the way tagrm works to allow tag removal on the fly
Diffstat (limited to 'mod')
-rw-r--r--mod/photos.php2
-rw-r--r--mod/tagrm.php91
2 files changed, 65 insertions, 28 deletions
diff --git a/mod/photos.php b/mod/photos.php
index 26a2bf50b..796613e3d 100644
--- a/mod/photos.php
+++ b/mod/photos.php
@@ -918,7 +918,7 @@ function photos_content(&$a) {
foreach($link_item['term'] as $t) {
$tags[$cnt] = array(0 => format_term_for_display($t));
if($can_post && ($ph[0]['uid'] == $owner_uid)) {
- $tags[$cnt][1] = 'tagrm?f=&item=' . $link_item['id'];
+ $tags[$cnt][1] = 'tagrm/drop/' . $link_item['id'] . '/' . bin2hex($t['term']); //?f=&item=' . $link_item['id'];
$tags[$cnt][2] = t('Remove');
}
$cnt ++;
diff --git a/mod/tagrm.php b/mod/tagrm.php
index 930d449b7..38787cb0b 100644
--- a/mod/tagrm.php
+++ b/mod/tagrm.php
@@ -52,53 +52,90 @@ function tagrm_post(&$a) {
function tagrm_content(&$a) {
- $o = '';
-
if(! local_user()) {
goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
// NOTREACHED
}
- $item = intval($_GET['item']);
- if(! $item) {
+ // remove tag on the fly if item and tag are provided
+ if((argc() == 4) && (argv(1) === 'drop') && intval(argv(2))) {
+
+ $item = intval(argv(2));
+ $tag = argv(3);
+
+ $r = q("SELECT * FROM `item` WHERE `id` = %d AND `uid` = %d LIMIT 1",
+ intval($item),
+ intval(local_user())
+ );
+
+ if(! $r)
+ goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
+
+ $r = fetch_post_tags($r,true);
+
+ $item = $r[0];
+
+ $new_tags = array();
+
+ if($item['term']) {
+ for($x = 0; $x < count($item['term']); $x ++) {
+ if($item['term'][$x]['term'] !== hex2bin($tag))
+ $new_tags[] = $item['term'][$x];
+ }
+ }
+
+ if($new_tags)
+ $item['term'] = $new_tags;
+ else
+ unset($item['term']);
+
+ item_store_update($item);
+
+ info( t('Tag removed') . EOL );
goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
- // NOTREACHED
+
}
+ //if we got only the item print a list of tags to select
+ if((argc() == 3) && (argv(1) === 'drop') && intval(argv(2))) {
- $r = q("SELECT * FROM `item` WHERE `id` = %d AND `uid` = %d LIMIT 1",
- intval($item),
- intval(local_user())
- );
+ $o = '';
+ $item = intval(argv(2));
- if(! $r)
- goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
+ $r = q("SELECT * FROM `item` WHERE `id` = %d AND `uid` = %d LIMIT 1",
+ intval($item),
+ intval(local_user())
+ );
- $r = fetch_post_tags($r,true);
+ if(! $r)
+ goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
+ $r = fetch_post_tags($r,true);
- if(! count($r[0]['term']))
- goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
+ if(! count($r[0]['term']))
+ goaway($a->get_baseurl() . '/' . $_SESSION['photo_return']);
- $o .= '<h3>' . t('Remove Item Tag') . '</h3>';
+ $o .= '<h3>' . t('Remove Item Tag') . '</h3>';
- $o .= '<p id="tag-remove-desc">' . t('Select a tag to remove: ') . '</p>';
+ $o .= '<p id="tag-remove-desc">' . t('Select a tag to remove: ') . '</p>';
- $o .= '<form id="tagrm" action="tagrm" method="post" >';
- $o .= '<input type="hidden" name="item" value="' . $item . '" />';
- $o .= '<ul>';
+ $o .= '<form id="tagrm" action="tagrm" method="post" >';
+ $o .= '<input type="hidden" name="item" value="' . $item . '" />';
+ $o .= '<ul>';
- foreach($r[0]['term'] as $x) {
- $o .= '<li><input type="checkbox" name="tag" value="' . bin2hex($x['term']) . '" >' . bbcode($x['term']) . '</input></li>';
- }
+ foreach($r[0]['term'] as $x) {
+ $o .= '<li><input type="checkbox" name="tag" value="' . bin2hex($x['term']) . '" >' . bbcode($x['term']) . '</input></li>';
+ }
+
+ $o .= '</ul>';
+ $o .= '<input id="tagrm-submit" type="submit" name="submit" value="' . t('Remove') .'" />';
+ $o .= '<input id="tagrm-cancel" type="submit" name="submit" value="' . t('Cancel') .'" />';
+ $o .= '</form>';
- $o .= '</ul>';
- $o .= '<input id="tagrm-submit" type="submit" name="submit" value="' . t('Remove') .'" />';
- $o .= '<input id="tagrm-cancel" type="submit" name="submit" value="' . t('Cancel') .'" />';
- $o .= '</form>';
+ return $o;
- return $o;
+ }
}