]> git.mxchange.org Git - friendica.git/commitdiff
Merge pull request #983 from annando/master
authortobiasd <tobias.diekershoff@gmx.net>
Sun, 18 May 2014 18:01:29 +0000 (20:01 +0200)
committertobiasd <tobias.diekershoff@gmx.net>
Sun, 18 May 2014 18:01:29 +0000 (20:01 +0200)
"red"-bookmarks and better plaintext export

1  2 
include/bbcode.php
include/items.php

diff --combined include/bbcode.php
index 33c0721b08e9cda041b7b151d72d8226368791ea,b036a5dcdf0045ddf7aac9654cda4a735b9b85e2..4ccab8db5d9ffb276a17874a1ff3bd9edd680296
@@@ -40,11 -40,11 +40,11 @@@ function bb_remove_share_information($T
  }
  
  function bb_cleanup_share($shared) {
-         if ($shared[2] != "type-link")
-                 return($shared[3]);
+       if (!in_array($shared[2], array("type-link", "type-video")))
+                 return($shared[0]);
  
          if (!preg_match_all("/\[bookmark\=([^\]]*)\](.*?)\[\/bookmark\]/ism",$shared[3], $bookmark))
-                 return($shared[3]);
+                 return($shared[0]);
  
          $title = "";
          $link = "";
@@@ -454,9 -454,10 +454,9 @@@ function bb_ShareAttributesForExport($m
        $userid = GetProfileUsername($profile,$author);
  
        $headline = '<div class="shared_header">';
 -      $headline .= sprintf(t('<span><b>'.
 -                      html_entity_decode("&#x2672; ", ENT_QUOTES, 'UTF-8').
 -                      '<a href="%s" target="_blank">%s</a>%s:</b></span>'), $link, $userid, $posted);
 -        $headline .= "</div>";
 +      $headline .= '<span><b>'.html_entity_decode("&#x2672; ", ENT_QUOTES, 'UTF-8');
 +      $headline .= sprintf(t('<a href="%1$s" target="_blank">%2$s</a> %3$s'), $link, $userid, $posted);
 +        $headline .= ":</b></span></div>";
  
        $text = trim($match[1]);
  
@@@ -746,9 -747,20 +746,20 @@@ function bbcode($Text,$preserve_nl = fa
        // Set up the parameters for a MAIL search string
        $MAILSearchString = $URLSearchString;
  
+       // Remove all hashtag addresses
+       if (!$tryoembed OR $simplehtml)
+               $Text = preg_replace("/([#@])\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", '$1$3', $Text);
        // Bookmarks in red - will be converted to bookmarks in friendica
        $Text = preg_replace("/#\^\[url\]([$URLSearchString]*)\[\/url\]/ism", '[bookmark=$1]$1[/bookmark]', $Text);
        $Text = preg_replace("/#\^\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", '[bookmark=$1]$2[/bookmark]', $Text);
+       $Text = preg_replace("/#\[url\=[$URLSearchString]*\]\^\[\/url\]\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/i",
+                               "[bookmark=$1]$2[/bookmark]", $Text);
+       if ($simplehtml == 2) {
+               $Text = preg_replace("/[^#@]\[url\=([^\]]*)\](.*?)\[\/url\]/ism",' $2 [url]$1[/url]',$Text);
+               $Text = preg_replace("/\[bookmark\=([^\]]*)\](.*?)\[\/bookmark\]/ism",' $2 [url]$1[/url]',$Text);
+       }
  
        if ($simplehtml == 5)
                $Text = preg_replace("/[^#@]\[url\=([$URLSearchString]*)\](.*?)\[\/url\]/ism", '[url]$1[/url]', $Text);
        //      $Text = str_replace('<br /><ul','<ul ', $Text);
  
        // Remove all hashtag addresses
      if (!$tryoembed AND get_config("system", "remove_hashtags_on_export")) {
/*    if (!$tryoembed AND get_config("system", "remove_hashtags_on_export")) {
                $pattern = '/#<a.*?href="(.*?)".*?>(.*?)<\/a>/is';
                $Text = preg_replace($pattern, '#$2', $Text);
        }
+ */
        call_hooks('bbcode',$Text);
  
        $a->save_timestamp($stamp1, "parser");
diff --combined include/items.php
index 2f03818a76bae0e2099df620a75016a04ec6f1b5,94285e61baff57a3609867a2e442e409ad7c794c..76bcbc7b24120d39216ce49a42140427a7e19436
@@@ -176,7 -176,7 +176,7 @@@ function get_feed_for(&$a, $dfrn_id, $o
                '$thumb'        => xmlify($owner['thumb']),
                '$picdate'      => xmlify(datetime_convert('UTC','UTC',$owner['avatar-date'] . '+00:00' , ATOM_TIME)) ,
                '$uridate'      => xmlify(datetime_convert('UTC','UTC',$owner['uri-date']    . '+00:00' , ATOM_TIME)) ,
 -              '$namdate'      => xmlify(datetime_convert('UTC','UTC',$owner['name-date']   . '+00:00' , ATOM_TIME)) , 
 +              '$namdate'      => xmlify(datetime_convert('UTC','UTC',$owner['name-date']   . '+00:00' , ATOM_TIME)) ,
                '$birthday'     => ((strlen($birthday)) ? '<dfrn:birthday>' . xmlify($birthday) . '</dfrn:birthday>' : ''),
                '$community'    => (($owner['page-flags'] == PAGE_COMMUNITY) ? '<dfrn:community>1</dfrn:community>' : '')
        ));
@@@ -261,7 -261,7 +261,7 @@@ function construct_activity_object($ite
        }
  
        return '';
 -} 
 +}
  
  function construct_activity_target($item) {
  
@@@ -425,7 -425,7 +425,7 @@@ function get_atom_elements($feed, $item
        $res = array();
  
        $author = $item->get_author();
 -      if($author) { 
 +      if($author) {
                $res['author-name'] = unxmlify($author->get_name());
                $res['author-link'] = unxmlify($author->get_link());
        }
  
        $res['body'] = limit_body_size($res['body']);
  
 -      // It isn't certain at this point whether our content is plaintext or html and we'd be foolish to trust 
 -      // the content type. Our own network only emits text normally, though it might have been converted to 
 +      // It isn't certain at this point whether our content is plaintext or html and we'd be foolish to trust
 +      // the content type. Our own network only emits text normally, though it might have been converted to
        // html if we used a pubsubhubbub transport. But if we see even one html tag in our text, we will
        // have to assume it is all html and needs to be purified.
  
 -      // It doesn't matter all that much security wise - because before this content is used anywhere, we are 
 -      // going to escape any tags we find regardless, but this lets us import a limited subset of html from 
 -      // the wild, by sanitising it and converting supported tags to bbcode before we rip out any remaining 
 +      // It doesn't matter all that much security wise - because before this content is used anywhere, we are
 +      // going to escape any tags we find regardless, but this lets us import a limited subset of html from
 +      // the wild, by sanitising it and converting supported tags to bbcode before we rip out any remaining
        // html.
  
        if((strpos($res['body'],'<') !== false) && (strpos($res['body'],'>') !== false)) {
                        if(! $type)
                                $type = 'application/octet-stream';
  
 -                      $att_arr[] = '[attach]href="' . $link . '" length="' . $len . '" type="' . $type . '" title="' . $title . '"[/attach]'; 
 +                      $att_arr[] = '[attach]href="' . $link . '" length="' . $len . '" type="' . $type . '" title="' . $title . '"[/attach]';
                }
                $res['attach'] = implode(',', $att_arr);
        }
@@@ -989,27 -989,10 +989,27 @@@ function item_store($arr,$force_parent 
        if(! x($arr,'type'))
                $arr['type']      = 'remote';
  
 +
 +
 +      /* check for create  date and expire time */
 +      $uid = intval($arr['uid']);
 +      $r = q("SELECT expire FROM user WHERE uid = %d", $uid);
 +      if(count($r)) {
 +              $expire_interval = $r[0]['expire'];
 +              if ($expire_interval>0) {
 +                      $expire_date =  new DateTime( '- '.$expire_interval.' days', new DateTimeZone('UTC'));
 +                      $created_date = new DateTime($arr['created'], new DateTimeZone('UTC'));
 +                      if ($created_date < $expire_date) {
 +                              logger('item-store: item created ('.$arr['created'].') before expiration time ('.$expire_date->format(DateTime::W3C).'). ignored. ' . print_r($arr,true), LOGGER_DEBUG);
 +                              return 0;
 +                      }
 +              }
 +      }
 +
        // Shouldn't happen but we want to make absolutely sure it doesn't leak from a plugin.
-       if((strpos($arr['body'],'<') !== false) || (strpos($arr['body'],'>') !== false))
-               $arr['body'] = strip_tags($arr['body']);
+       // Deactivated, since the bbcode parser can handle with it - and it destroys posts with some smileys that contain "<"
+       //if((strpos($arr['body'],'<') !== false) || (strpos($arr['body'],'>') !== false))
+       //      $arr['body'] = strip_tags($arr['body']);
  
  
        if (version_compare(PHP_VERSION, '5.3.0', '>=')) {
@@@ -1664,7 -1647,7 +1664,7 @@@ function dfrn_deliver($owner,$contact,$
        $final_dfrn_id = '';
  
        if($perm) {
 -              if((($perm == 'rw') && (! intval($contact['writable']))) 
 +              if((($perm == 'rw') && (! intval($contact['writable'])))
                || (($perm == 'r') && (intval($contact['writable'])))) {
                        q("update contact set writable = %d where id = %d",
                                intval(($perm == 'rw') ? 1 : 0),
                }
        }
  
 -      if(($contact['duplex'] && strlen($contact['pubkey'])) 
 +      if(($contact['duplex'] && strlen($contact['pubkey']))
                || ($owner['page-flags'] == PAGE_COMMUNITY && strlen($contact['pubkey']))
                || ($contact['rel'] == CONTACT_IS_SHARING && strlen($contact['pubkey']))) {
                openssl_public_decrypt($sent_dfrn_id,$final_dfrn_id,$contact['pubkey']);
  
        if($final_dfrn_id != $orig_id) {
                logger('dfrn_deliver: wrong dfrn_id.');
 -              // did not decode properly - cannot trust this site 
 +              // did not decode properly - cannot trust this site
                return 3;
        }
  
  
        if($page)
                $postvars['page'] = $page;
 -      
 +
        if($rino && $rino_allowed && (! $dissolve)) {
                $key = substr(random_string(),0,16);
                $data = bin2hex(aes_encrypt($postvars['data'],$key));
                $postvars['data'] = $data;
 -              logger('rino: sent key = ' . $key, LOGGER_DEBUG);       
 +              logger('rino: sent key = ' . $key, LOGGER_DEBUG);
  
  
 -              if($dfrn_version >= 2.1) {      
 -                      if(($contact['duplex'] && strlen($contact['pubkey'])) 
 +              if($dfrn_version >= 2.1) {
 +                      if(($contact['duplex'] && strlen($contact['pubkey']))
                                || ($owner['page-flags'] == PAGE_COMMUNITY && strlen($contact['pubkey']))
                                || ($contact['rel'] == CONTACT_IS_SHARING && strlen($contact['pubkey']))) {
  
  
        $res = parse_xml_string($xml);
  
 -      return $res->status; 
 +      return $res->status;
  }
  
  
@@@ -1808,12 -1791,12 +1808,12 @@@ function edited_timestamp_is_newer($exi
   * $importer = the contact_record (joined to user_record) of the local user who owns this relationship.
   *             It is this person's stuff that is going to be updated.
   * $contact =  the person who is sending us stuff. If not set, we MAY be processing a "follow" activity
 - *             from an external network and MAY create an appropriate contact record. Otherwise, we MUST 
 + *             from an external network and MAY create an appropriate contact record. Otherwise, we MUST
   *             have a contact record.
 - * $hub = should we find a hub declation in the feed, pass it back to our calling process, who might (or 
 + * $hub = should we find a hub declation in the feed, pass it back to our calling process, who might (or
   *        might not) try and subscribe to it.
   * $datedir sorts in reverse order
 - * $pass - by default ($pass = 0) we cannot guarantee that a parent item has been 
 + * $pass - by default ($pass = 0) we cannot guarantee that a parent item has been
   *      imported prior to its children being seen in the stream unless we are certain
   *      of how the feed is arranged/ordered.
   * With $pass = 1, we only pull parent items out of the stream.
@@@ -1974,7 -1957,7 +1974,7 @@@ function consume_feed($xml,$importer,&$
                         *
                         * $bdtext is just a readable placeholder in case the event is shared
                         * with others. We will replace it during presentation to our $importer
 -                       * to contain a sparkle link and perhaps a photo. 
 +                       * to contain a sparkle link and perhaps a photo.
                         *
                         */
  
                        );
  
                        // This function is called twice without reloading the contact
 -                      // Make sure we only create one event. This is why &$contact 
 +                      // Make sure we only create one event. This is why &$contact
                        // is a reference var in this function
  
                        $contact['bdyear'] = substr($birthday,0,4);
                                        $when = datetime_convert('UTC','UTC','now','Y-m-d H:i:s');
                        }
                        if($deleted && is_array($contact)) {
 -                              $r = q("SELECT `item`.*, `contact`.`self` FROM `item` INNER JOIN `contact` on `item`.`contact-id` = `contact`.`id` 
 +                              $r = q("SELECT `item`.*, `contact`.`self` FROM `item` INNER JOIN `contact` on `item`.`contact-id` = `contact`.`id`
                                        WHERE `uri` = '%s' AND `item`.`uid` = %d AND `contact-id` = %d AND NOT `item`.`file` LIKE '%%[%%' LIMIT 1",
                                        dbesc($uri),
                                        intval($importer['uid']),
                                $datarray['contact-id'] = $contact['id'];
  
                                if(! link_compare($datarray['owner-link'],$contact['url'])) {
 -                                      // The item owner info is not our contact. It's OK and is to be expected if this is a tgroup delivery, 
 +                                      // The item owner info is not our contact. It's OK and is to be expected if this is a tgroup delivery,
                                        // but otherwise there's a possible data mixup on the sender's system.
                                        // the tgroup delivery code called from item_store will correct it if it's a forum,
 -                                      // but we're going to unconditionally correct it here so that the post will always be owned by our contact. 
 +                                      // but we're going to unconditionally correct it here so that the post will always be owned by our contact.
                                        logger('consume_feed: Correcting item owner.', LOGGER_DEBUG);
                                        $datarray['owner-name']   = $contact['name'];
                                        $datarray['owner-link']   = $contact['url'];
                                        $datarray['owner-avatar'] = $contact['thumb'];
                                }
  
 -                              // We've allowed "followers" to reach this point so we can decide if they are 
 +                              // We've allowed "followers" to reach this point so we can decide if they are
                                // posting an @-tag delivery, which followers are allowed to do for certain
 -                              // page types. Now that we've parsed the post, let's check if it is legit. Otherwise ignore it. 
 +                              // page types. Now that we've parsed the post, let's check if it is legit. Otherwise ignore it.
  
                                if(($contact['rel'] == CONTACT_IS_FOLLOWER) && (! tgroup_check($importer['uid'],$datarray)))
                                        continue;
@@@ -2828,7 -2811,7 +2828,7 @@@ function local_delivery($importer,$data
  
                dbesc_array($msg);
  
 -              $r = dbq("INSERT INTO `mail` (`" . implode("`, `", array_keys($msg)) 
 +              $r = dbq("INSERT INTO `mail` (`" . implode("`, `", array_keys($msg))
                        . "`) VALUES ('" . implode("', '", array_values($msg)) . "')" );
  
                // send notifications.
                                        }
                                        else
                                                $sql_extra = " and contact.self = 1 and item.wall = 1 ";
 - 
 -                                      // was the top-level post for this reply written by somebody on this site? 
 -                                      // Specifically, the recipient? 
 +
 +                                      // was the top-level post for this reply written by somebody on this site?
 +                                      // Specifically, the recipient?
  
                                        $is_a_remote_delete = false;
  
                                        // POSSIBLE CLEANUP --> Why select so many fields when only forum_mode and wall are used?
 -                                      $r = q("select `item`.`id`, `item`.`uri`, `item`.`tag`, `item`.`forum_mode`,`item`.`origin`,`item`.`wall`, 
 -                                              `contact`.`name`, `contact`.`url`, `contact`.`thumb` from `item` 
 -                                              INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id` 
 +                                      $r = q("select `item`.`id`, `item`.`uri`, `item`.`tag`, `item`.`forum_mode`,`item`.`origin`,`item`.`wall`,
 +                                              `contact`.`name`, `contact`.`url`, `contact`.`thumb` from `item`
 +                                              INNER JOIN `contact` ON `contact`.`id` = `item`.`contact-id`
                                                WHERE `item`.`uri` = '%s' AND (`item`.`parent-uri` = '%s' or `item`.`thr-parent` = '%s')
 -                                              AND `item`.`uid` = %d 
 +                                              AND `item`.`uid` = %d
                                                $sql_extra
                                                LIMIT 1",
                                                dbesc($parent_uri),
                                                $is_a_remote_delete = true;
  
                                        // Does this have the characteristics of a community or private group comment?
 -                                      // If it's a reply to a wall post on a community/prvgroup page it's a 
 -                                      // valid community comment. Also forum_mode makes it valid for sure. 
 +                                      // If it's a reply to a wall post on a community/prvgroup page it's a
 +                                      // valid community comment. Also forum_mode makes it valid for sure.
                                        // If neither, it's not.
  
                                        if($is_a_remote_delete && $community) {
                        }
  
                        // Does this have the characteristics of a community or private group comment?
 -                      // If it's a reply to a wall post on a community/prvgroup page it's a 
 -                      // valid community comment. Also forum_mode makes it valid for sure. 
 +                      // If it's a reply to a wall post on a community/prvgroup page it's a
 +                      // valid community comment. Also forum_mode makes it valid for sure.
                        // If neither, it's not.
  
                        if($is_a_remote_comment && $community) {
                                                                'link'             => $a->get_baseurl() . '/display/' . $importer['nickname'] . '/' . $posted_id,
                                                                'source_name'  => stripslashes($datarray['author-name']),
                                                                'source_link'  => $datarray['author-link'],
 -                                                              'source_photo' => ((link_compare($datarray['author-link'],$importer['url'])) 
 +                                                              'source_photo' => ((link_compare($datarray['author-link'],$importer['url']))
                                                                        ? $importer['thumb'] : $datarray['author-avatar']),
                                                                'verb'         => ACTIVITY_POST,
                                                                'otype'        => 'item',
                                                                        'link'             => $a->get_baseurl() . '/display/' . $importer['nickname'] . '/' . $posted_id,
                                                                        'source_name'  => stripslashes($datarray['author-name']),
                                                                        'source_link'  => $datarray['author-link'],
 -                                                                      'source_photo' => ((link_compare($datarray['author-link'],$importer['url'])) 
 +                                                                      'source_photo' => ((link_compare($datarray['author-link'],$importer['url']))
                                                                                ? $importer['thumb'] : $datarray['author-avatar']),
                                                                        'verb'         => ACTIVITY_POST,
                                                                        'otype'        => 'item',
  
  
                        if(! link_compare($datarray['owner-link'],$importer['url'])) {
 -                              // The item owner info is not our contact. It's OK and is to be expected if this is a tgroup delivery, 
 +                              // The item owner info is not our contact. It's OK and is to be expected if this is a tgroup delivery,
                                // but otherwise there's a possible data mixup on the sender's system.
                                // the tgroup delivery code called from item_store will correct it if it's a forum,
 -                              // but we're going to unconditionally correct it here so that the post will always be owned by our contact. 
 +                              // but we're going to unconditionally correct it here so that the post will always be owned by our contact.
                                logger('local_delivery: Correcting item owner.', LOGGER_DEBUG);
                                $datarray['owner-name']   = $importer['senderName'];
                                $datarray['owner-link']   = $importer['url'];
                                foreach($links->link as $l) {
                                $atts = $l->attributes();
                                switch($atts['rel']) {
 -                                      case "alternate": 
 +                                      case "alternate":
                                                                $Blink = $atts['href'];
                                                                break;
                                                        default:
                                                        'link'             => $a->get_baseurl() . '/display/' . $importer['nickname'] . '/' . $posted_id,
                                                        'source_name'  => stripslashes($datarray['author-name']),
                                                        'source_link'  => $datarray['author-link'],
 -                                                      'source_photo' => ((link_compare($datarray['author-link'],$importer['url'])) 
 +                                                      'source_photo' => ((link_compare($datarray['author-link'],$importer['url']))
                                                                ? $importer['thumb'] : $datarray['author-avatar']),
                                                        'verb'         => $datarray['verb'],
                                                        'otype'        => 'person',
                                                ));
                                        }
                                }
 -                      }                       
 +                      }
  
                        continue;
                }
@@@ -3654,7 -3637,7 +3654,7 @@@ function new_follower($importer,$contac
  
                // create contact record
  
 -              $r = q("INSERT INTO `contact` ( `uid`, `created`, `url`, `nurl`, `name`, `nick`, `photo`, `network`, `rel`, 
 +              $r = q("INSERT INTO `contact` ( `uid`, `created`, `url`, `nurl`, `name`, `nick`, `photo`, `network`, `rel`,
                        `blocked`, `readonly`, `pending`, `writable` )
                        VALUES ( %d, '%s', '%s', '%s', '%s', '%s', '%s', '%s', %d, 0, 0, 1, 1 ) ",
                        intval($importer['uid']),
                                        '$siteurl' => $a->get_baseurl(),
                                        '$sitename' => $a->config['sitename']
                                ));
 -                              $res = mail($r[0]['email'], 
 +                              $res = mail($r[0]['email'],
                                        email_header_encode((($sharing) ? t('A new person is sharing with you at ') : t("You have a new follower at ")) . $a->config['sitename'],'UTF-8'),
                                        $email,
                                        'From: ' . 'Administrator' . '@' . $_SERVER['SERVER_NAME'] . "\n"
@@@ -3755,7 -3738,7 +3755,7 @@@ function subscribe_to_hub($url,$importe
                );
        }
  
 -      // Diaspora has different message-ids in feeds than they do 
 +      // Diaspora has different message-ids in feeds than they do
        // through the direct Diaspora protocol. If we try and use
        // the feed, we'll get duplicates. So don't.
  
@@@ -3951,7 -3934,7 +3951,7 @@@ function fix_private_photos($s, $uid, $
                                        // Check to see if we should replace this photo link with an embedded image
                                        // 1. No need to do so if the photo is public
                                        // 2. If there's a contact-id provided, see if they're in the access list
 -                                      //    for the photo. If so, embed it. 
 +                                      //    for the photo. If so, embed it.
                                        // 3. Otherwise, if we have an item, see if the item permissions match the photo
                                        //    permissions, regardless of order but first check to see if they're an exact
                                        //    match to save some processing overhead.
                                                if($cid) {
                                                        $recips = enumerate_permissions($r[0]);
                                                        if(in_array($cid, $recips)) {
 -                                                              $replace = true;        
 +                                                              $replace = true;
                                                        }
                                                }
                                                elseif($item) {
                                        }
                                }
                        }
 -              }       
 +              }
  
                $new_body = $new_body . substr($orig_body, 0, $img_start + $img_st_close) . $image . '[/img]';
                $orig_body = substr($orig_body, $img_start + $img_st_close + $img_len + strlen('[/img]'));
@@@ -4018,7 -4001,7 +4018,7 @@@ function has_permissions($obj) 
  }
  
  function compare_permissions($obj1,$obj2) {
 -      // first part is easy. Check that these are exactly the same. 
 +      // first part is easy. Check that these are exactly the same.
        if(($obj1['allow_cid'] == $obj2['allow_cid'])
                && ($obj1['allow_gid'] == $obj2['allow_gid'])
                && ($obj1['deny_cid'] == $obj2['deny_cid'])
@@@ -4060,14 -4043,14 +4060,14 @@@ function item_getfeedtags($item) 
                                $ret[] = array('#',$matches[1][$x], $matches[2][$x]);
                }
        }
 -      $matches = false; 
 +      $matches = false;
        $cnt = preg_match_all('|\@\[url\=(.*?)\](.*?)\[\/url\]|',$item['tag'],$matches);
        if($cnt) {
                for($x = 0; $x < $cnt; $x ++) {
                        if($matches[1][$x])
                                $ret[] = array('@',$matches[1][$x], $matches[2][$x]);
                }
 -      } 
 +      }
        return $ret;
  }
  
@@@ -4104,10 -4087,10 +4104,10 @@@ function item_expire($uid,$days) 
        $expire_network_only = get_pconfig($uid,'expire','network_only');
        $sql_extra = ((intval($expire_network_only)) ? " AND wall = 0 " : "");
  
 -      $r = q("SELECT * FROM `item` 
 -              WHERE `uid` = %d 
 -              AND `created` < UTC_TIMESTAMP() - INTERVAL %d DAY 
 -              AND `id` = `parent` 
 +      $r = q("SELECT * FROM `item`
 +              WHERE `uid` = %d
 +              AND `created` < UTC_TIMESTAMP() - INTERVAL %d DAY
 +              AND `id` = `parent`
                $sql_extra
                AND `deleted` = 0",
                intval($uid),
        }
  
        proc_run('php',"include/notifier.php","expire","$uid");
 -      
 +
  }
  
  
@@@ -4274,10 -4257,10 +4274,10 @@@ function drop_item($id,$interactive = t
                        }
                }
  
 -              // If item is a link to a photo resource, nuke all the associated photos 
 +              // If item is a link to a photo resource, nuke all the associated photos
                // (visitors will not have photo resources)
                // This only applies to photos uploaded from the photos page. Photos inserted into a post do not
 -              // generate a resource-id and therefore aren't intimately linked to the item. 
 +              // generate a resource-id and therefore aren't intimately linked to the item.
  
                if(strlen($item['resource-id'])) {
                        q("DELETE FROM `photo` WHERE `resource-id` = '%s' AND `uid` = %d ",
@@@ -4417,7 -4400,7 +4417,7 @@@ function posted_dates($uid,$wall) 
        if(! $dthen)
                return array();
  
 -      // If it's near the end of a long month, backup to the 28th so that in 
 +      // If it's near the end of a long month, backup to the 28th so that in
        // consecutive loops we'll always get a whole month difference.
  
        if(intval(substr($dnow,8)) > 28)