diff options
author | marijus <mario@mariovavti.com> | 2014-10-28 11:56:00 +0100 |
---|---|---|
committer | marijus <mario@mariovavti.com> | 2014-10-28 11:56:00 +0100 |
commit | ec8bab7784b531299150ddb2c18ede40d5c5df6c (patch) | |
tree | 43aeaa5714b455730eb77de4419864e0b7e21a00 | |
parent | 9cc76cb33da489c01731ecb5195f8bb3d51ce513 (diff) | |
download | volse-hubzilla-ec8bab7784b531299150ddb2c18ede40d5c5df6c.tar.gz volse-hubzilla-ec8bab7784b531299150ddb2c18ede40d5c5df6c.tar.bz2 volse-hubzilla-ec8bab7784b531299150ddb2c18ede40d5c5df6c.zip |
change the way tagrm works to allow tag removal on the fly
-rw-r--r-- | mod/photos.php | 2 | ||||
-rw-r--r-- | mod/tagrm.php | 91 | ||||
-rwxr-xr-x | view/tpl/photo_view.tpl | 2 |
3 files changed, 66 insertions, 29 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; + } } diff --git a/view/tpl/photo_view.tpl b/view/tpl/photo_view.tpl index 54e15de74..c79cd7d56 100755 --- a/view/tpl/photo_view.tpl +++ b/view/tpl/photo_view.tpl @@ -85,7 +85,7 @@ <div class="photo-item-tools-left" id="in-this-photo"> <span id="in-this-photo-text">{{$tag_hdr}}</span> {{foreach $tags as $t}} - {{$t.0}}{{if $edit}}<span id="tag-remove"> <a href="{{$t.1}}"><i class="icon-remove"></i></a> </span>{{/if}} + {{$t.0}}{{if $edit}}<span id="tag-remove"> <a href="{{$t.1}}" onclick="return confirmDelete();"><i class="icon-remove"></i></a> </span>{{/if}} {{/foreach}} </div> {{/if}} |