From 3e701b90ac1b2341833f6847c97c68a7f770afaa Mon Sep 17 00:00:00 2001 From: Roland Haeder Date: Sun, 1 May 2016 14:24:44 +0200 Subject: [PATCH] Continued rewriting: - use dba::is_result() everywhere (where I found the old, bad way) - converted some spaces -> tabs for code - converted some CRLF -> LF as mixures of both is not good Signed-off-by: Roland Haeder --- include/Contact.php | 2 +- include/Photo.php | 2 +- include/acl_selectors.php | 10 +- include/api.php | 4 +- include/auth.php | 4 +- include/cache.php | 4 +- include/config.php | 4 +- include/contact_selectors.php | 2 +- include/contact_widgets.php | 6 +- include/datetime.php | 2 +- include/dba.php | 2 +- include/delivery.php | 4 +- include/dfrn.php | 16 +-- include/enotify.php | 4 +- include/event.php | 8 +- include/expire.php | 2 +- include/fcontact.php | 4 +- include/follow.php | 8 +- include/gprobe.php | 2 +- include/group.php | 18 +-- include/identity.php | 10 +- include/items.php | 24 ++-- include/like.php | 6 +- include/lock.php | 2 +- include/message.php | 8 +- include/notifier.php | 10 +- include/oauth.php | 14 +- include/onepoll.php | 8 +- include/plugin.php | 6 +- include/queue_fn.php | 6 +- include/security.php | 12 +- include/session.php | 2 +- include/socgraph.php | 14 +- include/text.php | 12 +- include/threads.php | 2 +- include/user.php | 6 +- mod/admin.php | 2 +- mod/common.php | 4 +- mod/community.php | 2 +- mod/contactgroup.php | 4 +- mod/contacts.php | 8 +- mod/content.php | 6 +- mod/delegate.php | 12 +- mod/dfrn_confirm.php | 14 +- mod/dfrn_poll.php | 10 +- mod/dfrn_request.php | 20 +-- mod/directory.php | 4 +- mod/display.php | 22 ++-- mod/editpost.php | 2 +- mod/events.php | 6 +- mod/friendica.php | 4 +- mod/fsuggest.php | 4 +- mod/group.php | 6 +- mod/item.php | 16 +-- mod/lockview.php | 8 +- mod/manage.php | 2 +- mod/message.php | 6 +- mod/mood.php | 2 +- mod/msearch.php | 10 +- mod/network.php | 8 +- mod/nogroup.php | 4 +- mod/noscrape.php | 4 +- mod/notes.php | 6 +- mod/notice.php | 2 +- mod/notifications.php | 10 +- mod/photo.php | 8 +- mod/photos.php | 34 ++--- mod/poco.php | 4 +- mod/poke.php | 4 +- mod/profile.php | 18 +-- mod/profile_photo.php | 4 +- mod/profiles.php | 6 +- mod/profperm.php | 8 +- mod/proxy.php | 2 +- mod/pubsubhubbub.php | 2 +- mod/qsearch.php | 4 +- mod/regmod.php | 2 +- mod/salmon.php | 6 +- mod/search.php | 2 +- mod/settings.php | 20 +-- mod/subthread.php | 4 +- mod/tagger.php | 6 +- mod/uexport.php | 6 +- mod/videos.php | 12 +- mod/viewcontacts.php | 2 +- mod/viewsrc.php | 2 +- mod/wall_attach.php | 2 +- mod/wall_upload.php | 2 +- object/Item.php | 2 +- tests/get_tags_test.php | 240 +++++++++++++++++----------------- update.php | 26 ++-- view/theme/diabook/theme.php | 6 +- view/theme/vier/theme.php | 9 +- 93 files changed, 451 insertions(+), 450 deletions(-) diff --git a/include/Contact.php b/include/Contact.php index 79a14ab581..a5930e1cfc 100644 --- a/include/Contact.php +++ b/include/Contact.php @@ -368,7 +368,7 @@ function random_profile() { ORDER BY rand() LIMIT 1", dbesc(NETWORK_DFRN)); - if(count($r)) + if(dba::is_result($r)) return dirname($r[0]['url']); return ''; } diff --git a/include/Photo.php b/include/Photo.php index 91fce55a86..23148f69cc 100644 --- a/include/Photo.php +++ b/include/Photo.php @@ -589,7 +589,7 @@ class Photo { $r = q("select `guid` from photo where `resource-id` = '%s' and `guid` != '' limit 1", dbesc($rid) ); - if(count($r)) + if(dba::is_result($r)) $guid = $r[0]['guid']; else $guid = get_guid(); diff --git a/include/acl_selectors.php b/include/acl_selectors.php index fc3df52327..ea0a7aeb08 100644 --- a/include/acl_selectors.php +++ b/include/acl_selectors.php @@ -33,7 +33,7 @@ function group_select($selname,$selclass,$preselected = false,$size = 4) { call_hooks($a->module . '_pre_' . $selname, $arr); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if((is_array($preselected)) && in_array($rr['id'], $preselected)) $selected = " selected=\"selected\" "; @@ -144,7 +144,7 @@ function contact_selector($selname, $selclass, $preselected = false, $options) { call_hooks($a->module . '_pre_' . $selname, $arr); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if((is_array($preselected)) && in_array($rr['id'], $preselected)) $selected = " selected=\"selected\" "; @@ -220,7 +220,7 @@ function contact_select($selname, $selclass, $preselected = false, $size = 4, $p $receiverlist = array(); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if((is_array($preselected)) && in_array($rr['id'], $preselected)) $selected = " selected=\"selected\" "; @@ -314,7 +314,7 @@ function populate_acl($user = null, $show_jotnets = false) { $r = q("SELECT `pubmail` FROM `mailacct` WHERE `uid` = %d AND `server` != '' LIMIT 1", intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $mail_enabled = true; if(intval($r[0]['pubmail'])) $pubmail_enabled = true; @@ -577,7 +577,7 @@ function acl_lookup(&$a, $out_type = 'json') { $r = array(); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $g){ $contacts[] = array( "type" => "c", diff --git a/include/api.php b/include/api.php index 488e858445..3342655ac9 100644 --- a/include/api.php +++ b/include/api.php @@ -207,7 +207,7 @@ dbesc(trim($user)), dbesc($encrypted) ); - if(count($r)) + if(dba::is_result($r)) $record = $r[0]; } @@ -1249,7 +1249,7 @@ if (!count($r)) $r = q("SELECT `id` FROM `gcontact` WHERE `nick`='%s'", dbesc($_GET["q"])); - if (count($r)) { + if (dba::is_result($r)) { foreach ($r AS $user) { $user_info = api_get_user($a, $user["id"]); //echo print_r($user_info, true)."\n"; diff --git a/include/auth.php b/include/auth.php index d1917b8b30..f3193ee904 100644 --- a/include/auth.php +++ b/include/auth.php @@ -50,7 +50,7 @@ if (isset($_SESSION) && x($_SESSION,'authenticated') && (!x($_POST,'auth-params' $r = q("SELECT * FROM `contact` WHERE `id` = %d LIMIT 1", intval($_SESSION['visitor_id']) ); - if (count($r)) { + if (dba::is_result($r)) { $a->contact = $r[0]; } } @@ -169,7 +169,7 @@ if (isset($_SESSION) && x($_SESSION,'authenticated') && (!x($_POST,'auth-params' dbesc(trim($_POST['username'])), dbesc($encrypted) ); - if (count($r)) + if (dba::is_result($r)) $record = $r[0]; } diff --git a/include/cache.php b/include/cache.php index d0b0dfafda..7640f347b1 100644 --- a/include/cache.php +++ b/include/cache.php @@ -10,7 +10,7 @@ dbesc($key) ); - if (count($r)) + if (dba::is_result($r)) return $r[0]['v']; return null; @@ -38,7 +38,7 @@ * $r = q("SELECT * FROM `cache` WHERE `k`='%s' limit 1", * dbesc($key) * ); - * if(count($r)) { + * if(dba::is_result($r)) { * q("UPDATE `cache` SET `v` = '%s', `updated = '%s' WHERE `k` = '%s'", * dbesc($value), * dbesc(datetime_convert()), diff --git a/include/config.php b/include/config.php index b27274e9a9..513578ef99 100644 --- a/include/config.php +++ b/include/config.php @@ -20,7 +20,7 @@ function load_config($family) { global $a; $r = q("SELECT `v`, `k` FROM `config` WHERE `cat` = '%s'", dbesc($family)); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $k = $rr['k']; if ($family === 'config') { @@ -174,7 +174,7 @@ function load_pconfig($uid,$family) { dbesc($family), intval($uid) ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $k = $rr['k']; $a->config[$uid][$family][$k] = $rr['v']; diff --git a/include/contact_selectors.php b/include/contact_selectors.php index 3bf68f764e..77d38590b5 100644 --- a/include/contact_selectors.php +++ b/include/contact_selectors.php @@ -12,7 +12,7 @@ function contact_profile_assign($current,$foreign_net) { $r = q("SELECT `id`, `profile-name` FROM `profile` WHERE `uid` = %d", intval($_SESSION['uid'])); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $selected = (($rr['id'] == $current) ? " selected=\"selected\" " : ""); $o .= "\r\n"; diff --git a/include/contact_widgets.php b/include/contact_widgets.php index bbbd941b56..8fabc6ff69 100644 --- a/include/contact_widgets.php +++ b/include/contact_widgets.php @@ -93,7 +93,7 @@ function networks_widget($baseurl,$selected = '') { ); $nets = array(); - if(count($r)) { + if(dba::is_result($r)) { require_once('include/contact_selectors.php'); foreach($r as $rr) { if($rr['network']) @@ -204,13 +204,13 @@ function common_friends_visitor_widget($profile_uid) { dbesc(normalise_link(get_my_url())), intval($profile_uid) ); - if(count($r)) + if(dba::is_result($r)) $cid = $r[0]['id']; else { $r = q("select id from gcontact where nurl = '%s' limit 1", dbesc(normalise_link(get_my_url())) ); - if(count($r)) + if(dba::is_result($r)) $zcid = $r[0]['id']; } } diff --git a/include/datetime.php b/include/datetime.php index 89305a2406..da9e84d90f 100644 --- a/include/datetime.php +++ b/include/datetime.php @@ -544,7 +544,7 @@ function update_contact_birthdays() { // In-network birthdays are handled within local_delivery $r = q("SELECT * FROM contact WHERE `bd` != '' AND `bd` != '0000-00-00' AND SUBSTRING(`bd`,1,4) != `bdyear` "); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { logger('update_contact_birthday: ' . $rr['bd']); diff --git a/include/dba.php b/include/dba.php index 3a16c187c7..f11df6f17e 100644 --- a/include/dba.php +++ b/include/dba.php @@ -236,7 +236,7 @@ class dba { * @param $array mixed A filled array with at least one entry * @return Whether $array is a filled array */ - public function is_result ($array) { + public static function is_result ($array) { return (is_array($array) && count($array) > 0); } diff --git a/include/delivery.php b/include/delivery.php index fe33774382..7375beb232 100644 --- a/include/delivery.php +++ b/include/delivery.php @@ -254,7 +254,7 @@ function delivery_run(&$argv, &$argc){ intval($contact_id) ); - if (count($r)) + if (dba::is_result($r)) $contact = $r[0]; if ($contact['self']) @@ -416,7 +416,7 @@ function delivery_run(&$argv, &$argc){ intval($argv[2]), intval($uid) ); - if (count($r)) + if (dba::is_result($r)) $it = $r[0]; } if (!$it) diff --git a/include/dfrn.php b/include/dfrn.php index f6f4366029..4d116153bf 100644 --- a/include/dfrn.php +++ b/include/dfrn.php @@ -1430,7 +1430,7 @@ class dfrn { dbesc(normalise_link($suggest["url"])), intval($suggest["uid"]) ); - if(count($r)) + if(dba::is_result($r)) return false; // Do we already have an fcontact record for this person? @@ -1441,7 +1441,7 @@ class dfrn { dbesc($suggest["name"]), dbesc($suggest["request"]) ); - if(count($r)) { + if(dba::is_result($r)) { $fid = $r[0]["id"]; // OK, we do. Do we already have an introduction for this person ? @@ -1449,7 +1449,7 @@ class dfrn { intval($suggest["uid"]), intval($fid) ); - if(count($r)) + if(dba::is_result($r)) return false; } if(!$fid) @@ -1464,7 +1464,7 @@ class dfrn { dbesc($suggest["name"]), dbesc($suggest["request"]) ); - if(count($r)) + if(dba::is_result($r)) $fid = $r[0]["id"]; else // database record did not get created. Quietly give up. @@ -2120,7 +2120,7 @@ class dfrn { dbesc($item["uri"]), intval($importer["uid"]) ); - if(count($r)) + if(dba::is_result($r)) $ev["id"] = $r[0]["id"]; $event_id = event_store($ev); @@ -2141,7 +2141,7 @@ class dfrn { } // Update content if 'updated' changes - if(count($r)) { + if(dba::is_result($r)) { if (self::update_content($r[0], $item, $importer, $entrytype)) logger("Item ".$item["uri"]." was updated.", LOGGER_DEBUG); else @@ -2163,7 +2163,7 @@ class dfrn { intval($posted_id), intval($importer["importer_uid"]) ); - if(count($r)) { + if(dba::is_result($r)) { $parent = $r[0]["parent"]; $parent_uri = $r[0]["parent-uri"]; } @@ -2345,7 +2345,7 @@ class dfrn { dbesc($item["parent-uri"]), intval($importer["uid"]) ); - if(count($r)) { + if(dba::is_result($r)) { q("UPDATE `item` SET `last-child` = 1 WHERE `id` = %d", intval($r[0]["id"]) ); diff --git a/include/enotify.php b/include/enotify.php index 54b59857c2..55fec1f318 100644 --- a/include/enotify.php +++ b/include/enotify.php @@ -401,7 +401,7 @@ function notification($params) { $hash = random_string(); $r = q("SELECT `id` FROM `notify` WHERE `hash` = '%s' LIMIT 1", dbesc($hash)); - if(count($r)) + if(dba::is_result($r)) $dups = true; } while($dups == true); @@ -718,7 +718,7 @@ function check_item_notification($itemid, $uid, $defaulttype = "") { foreach ($tags AS $tag) { $r = q("SELECT `id` FROM `contact` WHERE `nurl` = '%s' AND `uid` = %d AND `notify_new_posts`", normalise_link($tag["url"]), intval($uid)); - if (count($r)) + if (dba::is_result($r)) $send_notification = true; } } diff --git a/include/event.php b/include/event.php index a9f054fc2e..4d5f3b8a1b 100644 --- a/include/event.php +++ b/include/event.php @@ -276,7 +276,7 @@ function event_store($arr) { intval($arr['id']), intval($arr['uid']) ); - return((count($r)) ? $r[0]['id'] : 0); + return((dba::is_result($r)) ? $r[0]['id'] : 0); } // The event changed. Update it. @@ -317,7 +317,7 @@ function event_store($arr) { intval($arr['id']), intval($arr['uid']) ); - if(count($r)) { + if(dba::is_result($r)) { $object = '' . xmlify(ACTIVITY_OBJ_EVENT) . '' . xmlify($arr['uri']) . ''; $object .= '' . xmlify(format_event_bbcode($arr)) . ''; $object .= '' . "\n"; @@ -376,7 +376,7 @@ function event_store($arr) { dbesc($arr['uri']), intval($arr['uid']) ); - if(count($r)) + if(dba::is_result($r)) $event = $r[0]; $item_arr = array(); @@ -418,7 +418,7 @@ function event_store($arr) { $r = q("SELECT * FROM `user` WHERE `uid` = %d LIMIT 1", intval($arr['uid']) ); - //if(count($r)) + //if(dba::is_result($r)) // $plink = $a->get_baseurl() . '/display/' . $r[0]['nickname'] . '/' . $item_id; diff --git a/include/expire.php b/include/expire.php index 873c594e84..5cc22d6830 100644 --- a/include/expire.php +++ b/include/expire.php @@ -39,7 +39,7 @@ function expire_run(&$argv, &$argc){ logger('expire: start'); $r = q("SELECT `uid`,`username`,`expire` FROM `user` WHERE `expire` != 0"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { logger('Expire: ' . $rr['username'] . ' interval: ' . $rr['expire'], LOGGER_DEBUG); item_expire($rr['uid'],$rr['expire']); diff --git a/include/fcontact.php b/include/fcontact.php index 8821a985f2..b268f2a037 100644 --- a/include/fcontact.php +++ b/include/fcontact.php @@ -10,7 +10,7 @@ function fcontact_store($url,$name,$photo) { dbesc($nurl) ); - if(count($r)) + if(dba::is_result($r)) return $r[0]['id']; $r = q("INSERT INTO `fcontact` ( `url`, `name`, `photo` ) VALUES ( '%s', '%s', '%s' ) ", @@ -23,7 +23,7 @@ function fcontact_store($url,$name,$photo) { $r = q("SELECT `id` FROM `fcontact` WHERE `url` = '%s' LIMIT 1", dbesc($nurl) ); - if(count($r)) + if(dba::is_result($r)) return $r[0]['id']; } diff --git a/include/follow.php b/include/follow.php index 2461bf0356..d5870e05de 100644 --- a/include/follow.php +++ b/include/follow.php @@ -178,7 +178,7 @@ function new_contact($uid,$url,$interactive = false) { intval($uid), dbesc(normalise_link($url)), dbesc($ret['network']) ); - if(count($r)) { + if(dba::is_result($r)) { // update contact if($r[0]['rel'] == CONTACT_IS_FOLLOWER || ($network === NETWORK_DIASPORA && $r[0]['rel'] == CONTACT_IS_SHARING)) { q("UPDATE `contact` SET `rel` = %d , `subhub` = %d, `readonly` = 0 WHERE `id` = %d AND `uid` = %d", @@ -196,7 +196,7 @@ function new_contact($uid,$url,$interactive = false) { $r = q("select count(*) as total from contact where uid = %d and pending = 0 and self = 0", intval($uid) ); - if(count($r)) + if(dba::is_result($r)) $total_contacts = $r[0]['total']; if(! service_class_allows($uid,'total_contacts',$total_contacts)) { @@ -208,7 +208,7 @@ function new_contact($uid,$url,$interactive = false) { intval($uid), dbesc($network) ); - if(count($r)) + if(dba::is_result($r)) $total_network = $r[0]['total']; if(! service_class_allows($uid,'total_contacts_' . $network,$total_network)) { @@ -295,7 +295,7 @@ function new_contact($uid,$url,$interactive = false) { intval($uid) ); - if(count($r)) { + if(dba::is_result($r)) { if(($contact['network'] == NETWORK_OSTATUS) && (strlen($contact['notify']))) { require_once('include/salmon.php'); slapper($r[0],$contact['notify'],$slap); diff --git a/include/gprobe.php b/include/gprobe.php index 91b9b16e33..70c45547eb 100644 --- a/include/gprobe.php +++ b/include/gprobe.php @@ -65,7 +65,7 @@ function gprobe_run(&$argv, &$argc){ dbesc(normalise_link($url)) ); } - if(count($r)) { + if(dba::is_result($r)) { // Check for accessibility and do a poco discovery if (poco_last_updated($r[0]['url'], true) AND ($r[0]["network"] == NETWORK_DFRN)) poco_load(0,0,$r[0]['id'], str_replace('/profile/','/poco/',$r[0]['url'])); diff --git a/include/group.php b/include/group.php index 00b66ad586..1c1fa631a1 100644 --- a/include/group.php +++ b/include/group.php @@ -44,7 +44,7 @@ function group_rmv($uid,$name) { intval($uid), dbesc($name) ); - if(count($r)) + if(dba::is_result($r)) $group_id = $r[0]['id']; if(! $group_id) return false; @@ -106,7 +106,7 @@ function group_byname($uid,$name) { intval($uid), dbesc($name) ); - if(count($r)) + if(dba::is_result($r)) return $r[0]['id']; return false; } @@ -139,7 +139,7 @@ function group_add_member($uid,$name,$member,$gid = 0) { intval($gid), intval($member) ); - if(count($r)) + if(dba::is_result($r)) return true; // You might question this, but // we indicate success because the group member was in fact created // -- It was just created at another time @@ -164,7 +164,7 @@ function group_get_members($gid) { intval($gid), intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $ret = $r; } return $ret; @@ -181,7 +181,7 @@ function group_public_members($gid) { intval(local_user()), dbesc(NETWORK_OSTATUS) ); - if(count($r)) + if(dba::is_result($r)) $ret = count($r); } return $ret; @@ -197,7 +197,7 @@ function mini_group_select($uid,$gid = 0, $label = "") { intval($uid) ); $grps[] = array('name' => '', 'id' => '0', 'selected' => ''); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $grps[] = array('name' => $rr['name'], 'id' => $rr['id'], 'selected' => (($gid == $rr['id']) ? 'true' : '')); } @@ -255,7 +255,7 @@ function group_side($every="contacts",$each="group",$editmode = "standard", $gro $member_of = groups_containing(local_user(),$cid); } - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $selected = (($group_id == $rr['id']) ? ' group-selected' : ''); @@ -316,7 +316,7 @@ function expand_groups($a,$check_dead = false, $use_gcontact = false) { $ret = array(); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $ret[] = $rr['contact-id']; if($check_dead AND !$use_gcontact) { @@ -345,7 +345,7 @@ function groups_containing($uid,$c) { ); $ret = array(); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) $ret[] = $rr['gid']; } diff --git a/include/identity.php b/include/identity.php index 888a09ee6f..18154a0fb8 100644 --- a/include/identity.php +++ b/include/identity.php @@ -138,7 +138,7 @@ function get_profiledata_by_nick($nickname, $uid = 0, $profile = 0) { $r = q("SELECT `profile-id` FROM `contact` WHERE `id` = %d LIMIT 1", intval($visitor['cid']) ); - if(count($r)) + if(dba::is_result($r)) $profile = $r[0]['profile-id']; break; } @@ -230,7 +230,7 @@ function profile_sidebar($profile, $block = 0) { $r = q("SELECT * FROM `contact` WHERE NOT `pending` AND `uid` = %d AND `nurl` = '%s'", local_user(), $profile_url); - if (count($r)) + if (dba::is_result($r)) $connect = false; } @@ -263,7 +263,7 @@ function profile_sidebar($profile, $block = 0) { 'entries' => array(), ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $profile['menu']['entries'][] = array( @@ -344,7 +344,7 @@ function profile_sidebar($profile, $block = 0) { if(is_array($a->profile) AND !$a->profile['hide-friends']) { $r = q("SELECT `gcontact`.`updated` FROM `contact` INNER JOIN `gcontact` WHERE `gcontact`.`nurl` = `contact`.`nurl` AND `self` AND `uid` = %d LIMIT 1", intval($a->profile['uid'])); - if(count($r)) + if(dba::is_result($r)) $updated = date("c", strtotime($r[0]['updated'])); $r = q("SELECT COUNT(*) AS `total` FROM `contact` WHERE `uid` = %d AND `self` = 0 AND `blocked` = 0 AND `pending` = 0 AND `hidden` = 0 AND `archive` = 0 @@ -354,7 +354,7 @@ function profile_sidebar($profile, $block = 0) { dbesc(NETWORK_DIASPORA), dbesc(NETWORK_OSTATUS) ); - if(count($r)) + if(dba::is_result($r)) $contacts = intval($r[0]['total']); } } diff --git a/include/items.php b/include/items.php index 5c0bb047ae..cc416d8d98 100644 --- a/include/items.php +++ b/include/items.php @@ -413,7 +413,7 @@ function item_store($arr,$force_parent = false, $notify = false, $dontcache = fa /* check for create date and expire time */ $uid = intval($arr['uid']); $r = q("SELECT expire FROM user WHERE uid = %d", intval($uid)); - if(count($r)) { + if(dba::is_result($r)) { $expire_interval = $r[0]['expire']; if ($expire_interval>0) { $expire_date = new DateTime( '- '.$expire_interval.' days', new DateTimeZone('UTC')); @@ -535,7 +535,7 @@ function item_store($arr,$force_parent = false, $notify = false, $dontcache = fa intval($arr['uid']) ); - if(count($r)) + if(dba::is_result($r)) $arr['network'] = $r[0]["network"]; // Fallback to friendica (why is it empty in some cases?) @@ -583,7 +583,7 @@ function item_store($arr,$force_parent = false, $notify = false, $dontcache = fa $r = q("SELECT `guid` FROM `item` WHERE `guid` = '%s' AND `network` = '%s' AND `uid` = '%d' LIMIT 1", dbesc($arr['guid']), dbesc($arr['network']), intval($arr['uid'])); - if(count($r)) { + if(dba::is_result($r)) { logger('found item with guid '.$arr['guid'].' for user '.$arr['uid'].' on network '.$arr['network'], LOGGER_DEBUG); return 0; } @@ -612,7 +612,7 @@ function item_store($arr,$force_parent = false, $notify = false, $dontcache = fa intval($arr['uid']) ); - if(count($r)) { + if(dba::is_result($r)) { // is the new message multi-level threaded? // even though we don't support it now, preserve the info @@ -768,7 +768,7 @@ function item_store($arr,$force_parent = false, $notify = false, $dontcache = fa intval($r[0]["id"]) ); return 0; - } elseif(count($r)) { + } elseif(dba::is_result($r)) { $current_post = $r[0]['id']; logger('item_store: created item ' . $current_post); @@ -979,7 +979,7 @@ function item_body_set_hashtags(&$item) { function get_item_guid($id) { $r = q("SELECT `guid` FROM `item` WHERE `id` = %d LIMIT 1", intval($id)); - if (count($r)) + if (dba::is_result($r)) return($r[0]["guid"]); else return(""); @@ -998,7 +998,7 @@ function get_item_id($guid, $uid = 0) { $r = q("SELECT `item`.`id`, `user`.`nickname` FROM `item` INNER JOIN `user` ON `user`.`uid` = `item`.`uid` WHERE `item`.`visible` = 1 AND `item`.`deleted` = 0 and `item`.`moderated` = 0 AND `item`.`guid` = '%s' AND `item`.`uid` = %d", dbesc($guid), intval($uid)); - if (count($r)) { + if (dba::is_result($r)) { $id = $r[0]["id"]; $nick = $r[0]["nickname"]; } @@ -1012,7 +1012,7 @@ function get_item_id($guid, $uid = 0) { AND `item`.`deny_cid` = '' AND `item`.`deny_gid` = '' AND `item`.`private` = 0 AND `item`.`wall` = 1 AND `item`.`guid` = '%s'", dbesc($guid)); - if (count($r)) { + if (dba::is_result($r)) { $id = $r[0]["id"]; $nick = $r[0]["nickname"]; } @@ -1313,7 +1313,7 @@ function item_is_remote_self($contact, &$datarray) { if ($contact['remote_self'] == 2) { $r = q("SELECT `id`,`url`,`name`,`thumb` FROM `contact` WHERE `uid` = %d AND `self`", intval($contact['uid'])); - if (count($r)) { + if (dba::is_result($r)) { $datarray['contact-id'] = $r[0]["id"]; $datarray['owner-name'] = $r[0]["name"]; @@ -1390,7 +1390,7 @@ function new_follower($importer,$contact,$datarray,$item,$sharing = false) { intval($importer['uid']), dbesc($url) ); - if(count($r)) { + if(dba::is_result($r)) { $contact_record = $r[0]; $photos = import_profile_photo($photo,$importer["uid"],$contact_record["id"]); @@ -1982,7 +1982,7 @@ function drop_item($id,$interactive = true) { dbesc($item['parent-uri']), intval($item['uid']) ); - if(count($r)) { + if(dba::is_result($r)) { q("UPDATE `item` SET `last-child` = 1 WHERE `id` = %d", intval($r[0]['id']) ); @@ -2019,7 +2019,7 @@ function first_post_date($uid,$wall = false) { intval($uid), intval($wall ? 1 : 0) ); - if(count($r)) { + if(dba::is_result($r)) { // logger('first_post_date: ' . $r[0]['id'] . ' ' . $r[0]['created'], LOGGER_DATA); return substr(datetime_convert('',date_default_timezone_get(),$r[0]['created']),0,10); } diff --git a/include/like.php b/include/like.php index 15633fc767..3ca005f28b 100644 --- a/include/like.php +++ b/include/like.php @@ -90,7 +90,7 @@ function do_like($item_id, $verb) { WHERE `contact`.`self` = 1 AND `contact`.`uid` = %d LIMIT 1", intval($owner_uid) ); - if(count($r)) + if(dba::is_result($r)) $owner = $r[0]; if(! $owner) { @@ -112,7 +112,7 @@ function do_like($item_id, $verb) { intval($_SESSION['visitor_id']), intval($owner_uid) ); - if(count($r)) + if(dba::is_result($r)) $contact = $r[0]; } if(! $contact) { @@ -135,7 +135,7 @@ function do_like($item_id, $verb) { dbesc($item_id), dbesc($item_id), dbesc($item['uri']) ); - if(count($r)) { + if(dba::is_result($r)) { $like_item = $r[0]; // Already voted, undo it diff --git a/include/lock.php b/include/lock.php index 70cf4b787b..35d2b02aa8 100644 --- a/include/lock.php +++ b/include/lock.php @@ -16,7 +16,7 @@ function lock_function($fn_name, $block = true, $wait_sec = 2, $timeout = 30) { dbesc($fn_name) ); - if((count($r)) AND (!$r[0]['locked'] OR (strtotime($r[0]['created']) < time() - 3600))) { + if((dba::is_result($r)) AND (!$r[0]['locked'] OR (strtotime($r[0]['created']) < time() - 3600))) { q("UPDATE `locks` SET `locked` = 1, `created` = '%s' WHERE `name` = '%s'", dbesc(datetime_convert()), dbesc($fn_name) diff --git a/include/message.php b/include/message.php index 0f4b53c626..fcbd53e4e7 100644 --- a/include/message.php +++ b/include/message.php @@ -41,7 +41,7 @@ function send_message($recipient=0, $body='', $subject='', $replyto=''){ dbesc($replyto), dbesc($replyto) ); - if(count($r)) + if(dba::is_result($r)) $convid = $r[0]['convid']; } @@ -74,7 +74,7 @@ function send_message($recipient=0, $body='', $subject='', $replyto=''){ dbesc($conv_guid), intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $convid = $r[0]['id']; } @@ -113,7 +113,7 @@ function send_message($recipient=0, $body='', $subject='', $replyto=''){ dbesc($uri), intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $post_id = $r[0]['id']; /** @@ -210,7 +210,7 @@ function send_wallmessage($recipient='', $body='', $subject='', $replyto=''){ dbesc($conv_guid), intval($recipient['uid']) ); - if(count($r)) + if(dba::is_result($r)) $convid = $r[0]['id']; if(! $convid) { diff --git a/include/notifier.php b/include/notifier.php index ffbb22e7bf..0e5f162843 100644 --- a/include/notifier.php +++ b/include/notifier.php @@ -314,7 +314,7 @@ function notifier_run(&$argv, &$argc){ intval($uid), dbesc(NETWORK_DFRN) ); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $recipients_followup[] = $rr['id']; } @@ -425,7 +425,7 @@ function notifier_run(&$argv, &$argc){ $r = q("SELECT * FROM `contact` WHERE `id` IN ($conversant_str) AND NOT `blocked` AND NOT `pending` AND NOT `archive`".$sql_extra); - if(count($r)) + if(dba::is_result($r)) $contacts = $r; } else @@ -443,7 +443,7 @@ function notifier_run(&$argv, &$argc){ intval($uid), dbesc(NETWORK_MAIL) ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) $recipients[] = $rr['id']; } @@ -471,7 +471,7 @@ function notifier_run(&$argv, &$argc){ // delivery loop - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $contact) { if(!$contact['self']) { @@ -572,7 +572,7 @@ function notifier_run(&$argv, &$argc){ $r = array_merge($r2,$r1,$r0); - if(count($r)) { + if(dba::is_result($r)) { logger('pubdeliver '.$target_item["guid"].': '.print_r($r,true), LOGGER_DEBUG); // throw everything into the queue in case we get killed diff --git a/include/oauth.php b/include/oauth.php index 6a1d72d178..bcfd79469b 100644 --- a/include/oauth.php +++ b/include/oauth.php @@ -23,7 +23,7 @@ class FKOAuthDataStore extends OAuthDataStore { $r = q("SELECT client_id, pw, redirect_uri FROM clients WHERE client_id='%s'", dbesc($consumer_key) ); - if (count($r)) + if (dba::is_result($r)) return new OAuthConsumer($r[0]['client_id'],$r[0]['pw'],$r[0]['redirect_uri']); return null; } @@ -35,7 +35,7 @@ class FKOAuthDataStore extends OAuthDataStore { dbesc($token_type), dbesc($token) ); - if (count($r)){ + if (dba::is_result($r)){ $ot=new OAuthToken($r[0]['id'],$r[0]['secret']); $ot->scope=$r[0]['scope']; $ot->expires = $r[0]['expires']; @@ -52,7 +52,7 @@ class FKOAuthDataStore extends OAuthDataStore { dbesc($nonce), intval($timestamp) ); - if (count($r)) + if (dba::is_result($r)) return new OAuthToken($r[0]['id'],$r[0]['secret']); return null; } @@ -136,7 +136,7 @@ class FKOAuth1 extends OAuthServer { $r = q("SELECT * FROM `user` WHERE uid=%d AND `blocked` = 0 AND `account_expired` = 0 AND `account_removed` = 0 AND `verified` = 1 LIMIT 1", intval($uid) ); - if(count($r)){ + if(dba::is_result($r)){ $record = $r[0]; } else { logger('FKOAuth1::loginUser failure: ' . print_r($_SERVER,true), LOGGER_DEBUG); @@ -162,7 +162,7 @@ class FKOAuth1 extends OAuthServer { $r = q("SELECT * FROM `contact` WHERE `uid` = %s AND `self` = 1 LIMIT 1", intval($_SESSION['uid'])); - if(count($r)) { + if(dba::is_result($r)) { $a->contact = $r[0]; $a->cid = $r[0]['id']; $_SESSION['cid'] = $a->cid; @@ -219,7 +219,7 @@ class FKOAuth2 extends OAuth2 { $r = q("SELECT client_id, expires, scope FROM tokens WHERE id = '%s'", dbesc($oauth_token)); - if (count($r)) + if (dba::is_result($r)) return $r[0]; return null; } @@ -247,7 +247,7 @@ class FKOAuth2 extends OAuth2 { $r = q("SELECT id, client_id, redirect_uri, expires, scope FROM auth_codes WHERE id = '%s'", dbesc($code)); - if (count($r)) + if (dba::is_result($r)) return $r[0]; return null; } diff --git a/include/onepoll.php b/include/onepoll.php index eb1045de14..19fa58ce02 100644 --- a/include/onepoll.php +++ b/include/onepoll.php @@ -94,7 +94,7 @@ function onepoll_run(&$argv, &$argc){ where `cid` = %d and updated > UTC_TIMESTAMP() - INTERVAL 1 DAY", intval($contact['id']) ); - if (count($r)) + if (dba::is_result($r)) if (!$r[0]['total']) poco_load($contact['id'],$importer_uid,0,$contact['poco']); } @@ -394,7 +394,7 @@ function onepoll_run(&$argv, &$argc){ dbesc($datarray['uri']) ); - if(count($r)) { + if(dba::is_result($r)) { logger("Mail: Seen before ".$msg_uid." for ".$mailconf[0]['user']." UID: ".$importer_uid." URI: ".$datarray['uri'],LOGGER_DEBUG); // Only delete when mails aren't automatically moved or deleted @@ -447,7 +447,7 @@ function onepoll_run(&$argv, &$argc){ $r = q("SELECT `uri` , `parent-uri` FROM `item` WHERE `uri` IN ( $qstr ) AND `uid` = %d LIMIT 1", intval($importer_uid) ); - if(count($r)) + if(dba::is_result($r)) $datarray['parent-uri'] = $r[0]['parent-uri']; // Set the parent as the top-level item // $datarray['parent-uri'] = $r[0]['uri']; } @@ -479,7 +479,7 @@ function onepoll_run(&$argv, &$argc){ $r = q("SELECT `uri` , `parent-uri` FROM `item` WHERE `title` = \"%s\" AND `uid` = %d ORDER BY `created` DESC LIMIT 1", dbesc(protect_sprintf($datarray['title'])), intval($importer_uid)); - if(count($r)) + if(dba::is_result($r)) $datarray['parent-uri'] = $r[0]['parent-uri']; } diff --git a/include/plugin.php b/include/plugin.php index 2cb94b3504..912591a5d0 100644 --- a/include/plugin.php +++ b/include/plugin.php @@ -79,7 +79,7 @@ function reload_plugins() { if(strlen($plugins)) { $r = q("SELECT * FROM `addon` WHERE `installed` = 1"); - if(count($r)) + if(dba::is_result($r)) $installed = $r; else $installed = array(); @@ -150,7 +150,7 @@ function register_hook($hook,$file,$function,$priority=0) { dbesc($file), dbesc($function) ); - if(count($r)) + if(dba::is_result($r)) return true; $r = q("INSERT INTO `hook` (`hook`, `file`, `function`, `priority`) VALUES ( '%s', '%s', '%s', '%s' ) ", @@ -187,7 +187,7 @@ function load_hooks() { $a = get_app(); $a->hooks = array(); $r = q("SELECT * FROM `hook` WHERE 1 ORDER BY `priority` DESC, `file`"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if(! array_key_exists($rr['hook'],$a->hooks)) $a->hooks[$rr['hook']] = array(); diff --git a/include/queue_fn.php b/include/queue_fn.php index 5214131b2b..4c02417ea3 100644 --- a/include/queue_fn.php +++ b/include/queue_fn.php @@ -21,16 +21,14 @@ function was_recently_delayed($cid) { and last > UTC_TIMESTAMP() - interval 15 minute limit 1", intval($cid) ); - if(count($r)) + if(dba::is_result($r)) return true; $r = q("select `term-date` from contact where id = %d and `term-date` != '' and `term-date` != '0000-00-00 00:00:00' limit 1", intval($cid) ); - if(count($r)) - return true; - return false; + return (dba::is_result($r)); } diff --git a/include/security.php b/include/security.php index 2d6db95f4f..3d64ae6677 100644 --- a/include/security.php +++ b/include/security.php @@ -42,7 +42,7 @@ function authenticate_success($user_record, $login_initial = false, $interactive $r = q("select * from user where uid = %d limit 1", intval($_SESSION['submanage']) ); - if(count($r)) + if(dba::is_result($r)) $master_record = $r[0]; } @@ -70,7 +70,7 @@ function authenticate_success($user_record, $login_initial = false, $interactive $r = q("SELECT * FROM `contact` WHERE `uid` = %d AND `self` = 1 LIMIT 1", intval($_SESSION['uid'])); - if(count($r)) { + if(dba::is_result($r)) { $a->contact = $r[0]; $a->cid = $r[0]['id']; $_SESSION['cid'] = $a->cid; @@ -158,7 +158,7 @@ function can_write_wall(&$a,$owner) { intval(PAGE_COMMUNITY) ); - if(count($r)) { + if(dba::is_result($r)) { $verified = 2; return true; } @@ -212,7 +212,7 @@ function permissions_sql($owner_id,$remote_verified = false,$groups = null) { intval($remote_user), intval($owner_id) ); - if(count($r)) { + if(dba::is_result($r)) { $remote_verified = true; $groups = init_groups_visitor($remote_user); } @@ -294,7 +294,7 @@ function item_permissions_sql($owner_id,$remote_verified = false,$groups = null) intval($remote_user), intval($owner_id) ); - if(count($r)) { + if(dba::is_result($r)) { $remote_verified = true; $groups = init_groups_visitor($remote_user); } @@ -405,7 +405,7 @@ function init_groups_visitor($contact_id) { WHERE `contact-id` = %d ", intval($contact_id) ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) $groups[] = $rr['gid']; } diff --git a/include/session.php b/include/session.php index 8f9d64606c..8c76eba41a 100644 --- a/include/session.php +++ b/include/session.php @@ -17,7 +17,7 @@ function ref_session_read ($id) { if(x($id)) $r = q("SELECT `data` FROM `session` WHERE `sid`= '%s'", dbesc($id)); - if(count($r)) { + if(dba::is_result($r)) { $session_exists = true; return $r[0]['data']; } else { diff --git a/include/socgraph.php b/include/socgraph.php index 0acd2793aa..00f8fb11f3 100644 --- a/include/socgraph.php +++ b/include/socgraph.php @@ -39,7 +39,7 @@ function poco_load($cid,$uid = 0,$zcid = 0,$url = null) { $r = q("select `poco`, `uid` from `contact` where `id` = %d limit 1", intval($cid) ); - if(count($r)) { + if(dba::is_result($r)) { $url = $r[0]['poco']; $uid = $r[0]['uid']; } @@ -205,14 +205,14 @@ function poco_check($profile_url, $name, $network, $profile_photo, $about, $loca $r = q("SELECT `network` FROM `contact` WHERE `nurl` = '%s' AND `network` != '' AND `network` != '%s' LIMIT 1", dbesc(normalise_link($profile_url)), dbesc(NETWORK_STATUSNET) ); - if(count($r)) + if(dba::is_result($r)) $network = $r[0]["network"]; if (($network == "") OR ($network == NETWORK_OSTATUS)) { $r = q("SELECT `network`, `url` FROM `contact` WHERE `alias` IN ('%s', '%s') AND `network` != '' AND `network` != '%s' LIMIT 1", dbesc($profile_url), dbesc(normalise_link($profile_url)), dbesc(NETWORK_STATUSNET) ); - if(count($r)) { + if(dba::is_result($r)) { $network = $r[0]["network"]; //$profile_url = $r[0]["url"]; } @@ -948,7 +948,7 @@ function count_common_friends($uid,$cid) { ); // logger("count_common_friends: $uid $cid {$r[0]['total']}"); - if(count($r)) + if(dba::is_result($r)) return $r[0]['total']; return 0; @@ -994,7 +994,7 @@ function count_common_friends_zcid($uid,$zcid) { intval($uid) ); - if(count($r)) + if(dba::is_result($r)) return $r[0]['total']; return 0; @@ -1033,7 +1033,7 @@ function count_all_friends($uid,$cid) { intval($uid) ); - if(count($r)) + if(dba::is_result($r)) return $r[0]['total']; return 0; @@ -1162,7 +1162,7 @@ function update_suggestions() { dbesc(NETWORK_DFRN), dbesc(NETWORK_DIASPORA) ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $base = substr($rr['poco'],0,strrpos($rr['poco'],'/')); if(! in_array($base,$done)) diff --git a/include/text.php b/include/text.php index e2d2e05d37..ed905d3fff 100644 --- a/include/text.php +++ b/include/text.php @@ -491,7 +491,7 @@ function item_new_uri($hostname,$uid, $guid = "") { $r = q("SELECT `id` FROM `item` WHERE `uri` = '%s' LIMIT 1", dbesc($uri)); - if(count($r)) + if(dba::is_result($r)) $dups = true; } while($dups == true); return $uri; @@ -515,7 +515,7 @@ function photo_new_resource() { $r = q("SELECT `id` FROM `photo` WHERE `resource-id` = '%s' LIMIT 1", dbesc($resource) ); - if(count($r)) + if(dba::is_result($r)) $found = true; } while($found == true); return $resource; @@ -859,7 +859,7 @@ function contact_block() { dbesc(NETWORK_OSTATUS), dbesc(NETWORK_DIASPORA) ); - if(count($r)) { + if(dba::is_result($r)) { $total = intval($r[0]['total']); } if(! $total) { @@ -877,7 +877,7 @@ function contact_block() { dbesc(NETWORK_DIASPORA), intval($shown) ); - if(count($r)) { + if(dba::is_result($r)) { $contacts = sprintf( tt('%d Contact','%d Contacts', $total),$total); $micropro = Array(); foreach($r as $rr) { @@ -2050,7 +2050,7 @@ function file_tag_update_pconfig($uid,$file_old,$file_new,$type = 'file') { // intval($uid) //); - if(count($r)) { + if(dba::is_result($r)) { unset($deleted_tags[$key]); } else { @@ -2080,7 +2080,7 @@ function file_tag_save_file($uid,$item,$file) { intval($item), intval($uid) ); - if(count($r)) { + if(dba::is_result($r)) { if(! stristr($r[0]['file'],'[' . file_tag_encode($file) . ']')) q("UPDATE `item` SET `file` = '%s' WHERE `id` = %d AND `uid` = %d", dbesc($r[0]['file'] . '[' . file_tag_encode($file) . ']'), diff --git a/include/threads.php b/include/threads.php index 0320eaa018..33d6d3fc82 100644 --- a/include/threads.php +++ b/include/threads.php @@ -88,7 +88,7 @@ function add_shadow_entry($item) { // Is there already a shadow entry? $r = q("SELECT `id` FROM `item` WHERE `uri` = '%s' AND `uid` = 0 LIMIT 1", dbesc($item['uri'])); - if (count($r)) + if (dba::is_result($r)) return; // Preparing public shadow (removing user specific data) diff --git a/include/user.php b/include/user.php index 3997a8a61f..a61da7b303 100644 --- a/include/user.php +++ b/include/user.php @@ -130,7 +130,7 @@ function create_user($arr) { $r = q("SELECT * FROM `user` WHERE `email` = '%s' LIMIT 1", dbesc($email) ); - if(count($r)) + if(dba::is_result($r)) $result['message'] .= t('Cannot use that email.') . EOL; } @@ -143,7 +143,7 @@ function create_user($arr) { WHERE `nickname` = '%s' LIMIT 1", dbesc($nickname) ); - if(count($r)) + if(dba::is_result($r)) $result['message'] .= t('Nickname is already registered. Please choose another.') . EOL; // Check deleted accounts that had this nickname. Doesn't matter to us, @@ -153,7 +153,7 @@ function create_user($arr) { WHERE `username` = '%s' LIMIT 1", dbesc($nickname) ); - if(count($r)) + if(dba::is_result($r)) $result['message'] .= t('Nickname was once registered here and may not be re-used. Please choose another.') . EOL; if(strlen($result['message'])) { diff --git a/mod/admin.php b/mod/admin.php index 2fc9c48a78..3e19935cb5 100644 --- a/mod/admin.php +++ b/mod/admin.php @@ -1078,7 +1078,7 @@ function admin_page_dbsync(&$a) { $failed = array(); $r = q("select k, v from config where `cat` = 'database' "); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $upd = intval(substr($rr['k'],7)); if($upd < 1139 || $rr['v'] === 'success') diff --git a/mod/common.php b/mod/common.php index 62a5185fed..bc65336249 100644 --- a/mod/common.php +++ b/mod/common.php @@ -57,13 +57,13 @@ function common_content(&$a) { dbesc(normalise_link(get_my_url())), intval($profile_uid) ); - if(count($r)) + if(dba::is_result($r)) $cid = $r[0]['id']; else { $r = q("SELECT `id` FROM `gcontact` WHERE `nurl` = '%s' LIMIT 1", dbesc(normalise_link(get_my_url())) ); - if(count($r)) + if(dba::is_result($r)) $zcid = $r[0]['id']; } } diff --git a/mod/community.php b/mod/community.php index b6d72a3555..fb50ad4bd2 100644 --- a/mod/community.php +++ b/mod/community.php @@ -59,7 +59,7 @@ function community_content(&$a, $update = 0) { AND `item`.`private` = 0 AND `item`.`wall` = 1" ); - if(count($r)) + if(dba::is_result($r)) $a->set_pager_total($r[0]['total']); if(! $r[0]['total']) { diff --git a/mod/contactgroup.php b/mod/contactgroup.php index bf81afe079..be69832512 100644 --- a/mod/contactgroup.php +++ b/mod/contactgroup.php @@ -14,7 +14,7 @@ function contactgroup_content(&$a) { intval($a->argv[2]), intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $change = intval($a->argv[2]); } @@ -47,4 +47,4 @@ function contactgroup_content(&$a) { } killme(); -} \ No newline at end of file +} diff --git a/mod/contacts.php b/mod/contacts.php index 4897663a05..19cbf4e6fb 100644 --- a/mod/contacts.php +++ b/mod/contacts.php @@ -758,22 +758,22 @@ function contacts_content(&$a) { $r = q("SELECT COUNT(*) AS `total` FROM `contact` WHERE `uid` = %d AND `self` = 0 AND `pending` = 0 $sql_extra $sql_extra2 ", intval($_SESSION['uid'])); - if(count($r)) { + if(dba::is_result($r)) { $a->set_pager_total($r[0]['total']); $total = $r[0]['total']; } $sql_extra3 = unavailable_networks(); + $contacts = array(); + $r = q("SELECT * FROM `contact` WHERE `uid` = %d AND `self` = 0 AND `pending` = 0 $sql_extra $sql_extra2 $sql_extra3 ORDER BY `name` ASC LIMIT %d , %d ", intval($_SESSION['uid']), intval($a->pager['start']), intval($a->pager['itemspage']) ); - $contacts = array(); - - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $contacts[] = _contact_detail_for_template($rr); } diff --git a/mod/content.php b/mod/content.php index 49cff74d2d..718dc77d29 100644 --- a/mod/content.php +++ b/mod/content.php @@ -93,7 +93,7 @@ function content_content(&$a, $update = 0) { ); $str = ''; - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $str .= '<' . $rr['id'] . '>'; if(strlen($str)) @@ -141,7 +141,7 @@ function content_content(&$a, $update = 0) { AND `blocked` = 0 AND `pending` = 0 LIMIT 1", intval($cid) ); - if(count($r)) { + if(dba::is_result($r)) { $sql_extra = " AND `item`.`parent` IN ( SELECT DISTINCT(`parent`) FROM `item` WHERE 1 $sql_options AND `contact-id` = " . intval($cid) . " and deleted = 0 ) "; } @@ -259,7 +259,7 @@ function content_content(&$a, $update = 0) { $parents_arr = array(); $parents_str = ''; - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) if(! in_array($rr['item_id'],$parents_arr)) $parents_arr[] = $rr['item_id']; diff --git a/mod/delegate.php b/mod/delegate.php index 20d2e605e0..55951cd51b 100644 --- a/mod/delegate.php +++ b/mod/delegate.php @@ -26,12 +26,12 @@ function delegate_content(&$a) { $r = q("select `nickname` from user where uid = %d limit 1", intval($id) ); - if(count($r)) { + if(dba::is_result($r)) { $r = q("select id from contact where uid = %d and nurl = '%s' limit 1", intval(local_user()), dbesc(normalise_link($a->get_baseurl() . '/profile/' . $r[0]['nickname'])) ); - if(count($r)) { + if(dba::is_result($r)) { q("insert into manage ( uid, mid ) values ( %d , %d ) ", intval($a->argv[2]), intval(local_user()) @@ -64,7 +64,7 @@ function delegate_content(&$a) { dbesc($a->user['email']), dbesc($a->user['password']) ); - if(count($r)) + if(dba::is_result($r)) $full_managers = $r; $delegates = array(); @@ -75,7 +75,7 @@ function delegate_content(&$a) { intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $delegates = $r; $uids = array(); @@ -104,7 +104,7 @@ function delegate_content(&$a) { $nicknames = array(); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $nicknames[] = "'" . dbesc(basename($rr['nurl'])) . "'"; } @@ -118,7 +118,7 @@ function delegate_content(&$a) { $r = q("select `uid`, `username`, `nickname` from user where nickname in ( $nicks )"); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) if(! in_array($rr['uid'],$uids)) $potentials[] = $rr; diff --git a/mod/dfrn_confirm.php b/mod/dfrn_confirm.php index 9c70ba21ab..29d3630aca 100644 --- a/mod/dfrn_confirm.php +++ b/mod/dfrn_confirm.php @@ -418,7 +418,7 @@ function dfrn_confirm_post(&$a,$handsfree = null) { $r = q("SELECT * FROM `contact` WHERE `id` = %d LIMIT 1", intval($contact_id) ); - if(count($r)) + if(dba::is_result($r)) $contact = $r[0]; else $contact = null; @@ -438,7 +438,7 @@ function dfrn_confirm_post(&$a,$handsfree = null) { intval($uid) ); - if((count($r)) && ($r[0]['hide-friends'] == 0) && ($activity) && (! $hidden)) { + if((dba::is_result($r)) && ($r[0]['hide-friends'] == 0) && ($activity) && (! $hidden)) { require_once('include/items.php'); @@ -624,7 +624,7 @@ function dfrn_confirm_post(&$a,$handsfree = null) { $r = q("SELECT * FROM `contact` WHERE `dfrn-id` = '%s' LIMIT 1", dbesc($decrypted_dfrn_id) ); - if(count($r)) { + if(dba::is_result($r)) { $message = t('The ID provided by your system is a duplicate on our system. It should work if you try again.'); xml_status(1,$message); // Birthday paradox - duplicate dfrn-id // NOTREACHED @@ -656,7 +656,7 @@ function dfrn_confirm_post(&$a,$handsfree = null) { $r = q("SELECT `photo` FROM `contact` WHERE `id` = %d LIMIT 1", intval($dfrn_record)); - if(count($r)) + if(dba::is_result($r)) $photo = $r[0]['photo']; else $photo = $a->get_baseurl() . '/images/person-175.jpg'; @@ -709,10 +709,10 @@ function dfrn_confirm_post(&$a,$handsfree = null) { intval($dfrn_record) ); - if(count($r)) + if(dba::is_result($r)) $combined = $r[0]; - if((count($r)) && ($r[0]['notify-flags'] & NOTIFY_CONFIRM)) { + if((dba::is_result($r)) && ($r[0]['notify-flags'] & NOTIFY_CONFIRM)) { $mutual = ($new_relation == CONTACT_IS_FRIEND); notification(array( 'type' => NOTIFY_CONFIRM, @@ -737,7 +737,7 @@ function dfrn_confirm_post(&$a,$handsfree = null) { intval($local_uid) ); - if((count($r)) && ($r[0]['hide-friends'] == 0)) { + if((dba::is_result($r)) && ($r[0]['hide-friends'] == 0)) { require_once('include/items.php'); diff --git a/mod/dfrn_poll.php b/mod/dfrn_poll.php index ab6637607e..a75fbc40b8 100644 --- a/mod/dfrn_poll.php +++ b/mod/dfrn_poll.php @@ -76,7 +76,7 @@ function dfrn_poll_init(&$a) { dbesc($a->argv[1]) ); - if(count($r)) { + if(dba::is_result($r)) { $s = fetch_url($r[0]['poll'] . '?dfrn_id=' . $my_id . '&type=profile-check'); @@ -187,7 +187,7 @@ function dfrn_poll_init(&$a) { q("DELETE FROM `profile_check` WHERE `expire` < " . intval(time())); $r = q("SELECT * FROM `profile_check` WHERE `dfrn_id` = '%s' ORDER BY `expire` DESC", dbesc($dfrn_id)); - if(count($r)) { + if(dba::is_result($r)) { xml_status(1); return; // NOTREACHED } @@ -332,7 +332,7 @@ function dfrn_poll_post(&$a) { $reputation = 0; $text = ''; - if(count($r)) { + if(dba::is_result($r)) { $reputation = $r[0]['rating']; $text = $r[0]['reason']; @@ -445,7 +445,7 @@ function dfrn_poll_content(&$a) { dbesc($nickname) ); - if(count($r)) { + if(dba::is_result($r)) { $challenge = ''; $encrypted_id = ''; @@ -492,7 +492,7 @@ function dfrn_poll_content(&$a) { )); } - $profile = ((count($r) && $r[0]['nickname']) ? $r[0]['nickname'] : $nickname); + $profile = ((dba::is_result($r) && $r[0]['nickname']) ? $r[0]['nickname'] : $nickname); switch($destination_url) { case 'profile': diff --git a/mod/dfrn_request.php b/mod/dfrn_request.php index 32e835b715..5932b9b669 100644 --- a/mod/dfrn_request.php +++ b/mod/dfrn_request.php @@ -89,7 +89,7 @@ function dfrn_request_post(&$a) { dbesc(normalise_link($dfrn_url)) ); - if(count($r)) { + if(dba::is_result($r)) { if(strlen($r[0]['dfrn-id'])) { /** @@ -180,7 +180,7 @@ function dfrn_request_post(&$a) { dbesc($dfrn_url), $parms['key'] // this was already escaped ); - if(count($r)) { + if(dba::is_result($r)) { $def_gid = get_default_group(local_user(), $r[0]["network"]); if(intval($def_gid)) group_add_member(local_user(), '', $r[0]['id'], $def_gid); @@ -285,7 +285,7 @@ function dfrn_request_post(&$a) { AND `intro`.`datetime` < UTC_TIMESTAMP() - INTERVAL 30 MINUTE ", dbesc(NETWORK_MAIL2) ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if(! $rr['rel']) { q("DELETE FROM `contact` WHERE `id` = %d", @@ -310,7 +310,7 @@ function dfrn_request_post(&$a) { AND `intro`.`datetime` < UTC_TIMESTAMP() - INTERVAL 3 DAY ", dbesc(NETWORK_MAIL2) ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if(! $rr['rel']) { q("DELETE FROM `contact` WHERE `id` = %d", @@ -390,7 +390,7 @@ function dfrn_request_post(&$a) { dbesc($poll), intval($uid) ); - if(count($r)) { + if(dba::is_result($r)) { $contact_id = $r[0]['id']; $def_gid = get_default_group($uid, $r[0]["network"]); @@ -562,7 +562,7 @@ function dfrn_request_post(&$a) { $parms['url'], $parms['issued-id'] ); - if(count($r)) + if(dba::is_result($r)) $contact_record = $r[0]; } @@ -711,16 +711,16 @@ function dfrn_request_content(&$a) { dbesc($_GET['confirm_key']) ); - if(count($intro)) { + if(dba::is_result($intro)) { + + $auto_confirm = false; $r = q("SELECT `contact`.*, `user`.* FROM `contact` LEFT JOIN `user` ON `contact`.`uid` = `user`.`uid` WHERE `contact`.`id` = %d LIMIT 1", intval($intro[0]['contact-id']) ); - $auto_confirm = false; - - if(count($r)) { + if(dba::is_result($r)) { if(($r[0]['page-flags'] != PAGE_NORMAL) && ($r[0]['page-flags'] != PAGE_PRVGROUP)) $auto_confirm = true; diff --git a/mod/directory.php b/mod/directory.php index 625f6c95ac..531283936c 100644 --- a/mod/directory.php +++ b/mod/directory.php @@ -78,7 +78,7 @@ function directory_content(&$a) { $r = $db->q("SELECT COUNT(*) AS `total` FROM `profile` LEFT JOIN `user` ON `user`.`uid` = `profile`.`uid` WHERE `is-default` = 1 $publish AND `user`.`blocked` = 0 $sql_extra "); - if(count($r)) + if(dba::is_result($r)) $a->set_pager_total($r[0]['total']); $order = " ORDER BY `name` ASC "; @@ -90,7 +90,7 @@ function directory_content(&$a) { LEFT JOIN `user` ON `user`.`uid` = `profile`.`uid` LEFT JOIN `contact` ON `contact`.`uid` = `user`.`uid` WHERE `is-default` = 1 $publish AND `user`.`blocked` = 0 AND `contact`.`self` $sql_extra $order LIMIT ".$limit); - if(count($r)) { + if(dba::is_result($r)) { if(in_array('small', $a->argv)) $photo = 'thumb'; diff --git a/mod/display.php b/mod/display.php index fbbd263172..849fef3f24 100644 --- a/mod/display.php +++ b/mod/display.php @@ -19,7 +19,7 @@ function display_init(&$a) { $r = q("SELECT `id`, `parent`, `author-name`, `author-link`, `author-avatar`, `network`, `body`, `uid` FROM `item` WHERE `item`.`visible` AND NOT `item`.`deleted` AND NOT `item`.`moderated` AND `guid` = '%s' AND `uid` = %d", dbesc($a->argv[1]), local_user()); - if (count($r)) { + if (dba::is_result($r)) { $nick = $a->user["nickname"]; $itemuid = local_user(); } @@ -36,7 +36,7 @@ function display_init(&$a) { AND NOT `item`.`private` AND NOT `user`.`hidewall` AND `item`.`guid` = '%s'", dbesc($a->argv[1])); // AND NOT `item`.`private` AND `item`.`wall` - if (count($r)) { + if (dba::is_result($r)) { $nick = $r[0]["nickname"]; $itemuid = $r[0]["uid"]; } @@ -53,7 +53,7 @@ function display_init(&$a) { AND `item`.`guid` = '%s'", dbesc($a->argv[1])); // AND NOT `item`.`private` AND `item`.`wall` } - if (count($r)) { + if (dba::is_result($r)) { if ($r[0]["id"] != $r[0]["parent"]) $r = q("SELECT `id`, `author-name`, `author-link`, `author-avatar`, `network`, `body`, `uid` FROM `item` WHERE `item`.`visible` AND NOT `item`.`deleted` AND NOT `item`.`moderated` @@ -70,7 +70,7 @@ function display_init(&$a) { WHERE `user`.`nickname` = '%s' AND `profile`.`is-default` AND `contact`.`self` LIMIT 1", dbesc($nickname) ); - if (count($r)) + if (dba::is_result($r)) $profiledata = $r[0]; $profiledata["network"] = NETWORK_DFRN; @@ -168,7 +168,7 @@ function display_fetchauthor($a, $item) { dbesc(normalise_link($profiledata["url"])), intval(local_user()), intval(CONTACT_IS_SHARING), intval(CONTACT_IS_FRIEND)); - if (count($r)) { + if (dba::is_result($r)) { $profiledata["name"] = $r[0]["name"]; $profiledata["photo"] = $r[0]["photo"]; $profiledata["nickname"] = $r[0]["nick"]; @@ -187,7 +187,7 @@ function display_fetchauthor($a, $item) { // Fetching profile data from global contacts if ($profiledata["network"] != NETWORK_FEED) { $r = q("SELECT `name`, `photo`, `nick`, `addr`, `location`, `about`, `gender`, `keywords`, `network` FROM `gcontact` WHERE `nurl` = '%s'", dbesc(normalise_link($profiledata["url"]))); - if (count($r)) { + if (dba::is_result($r)) { $profiledata["name"] = $r[0]["name"]; $profiledata["photo"] = $r[0]["photo"]; $profiledata["nickname"] = $r[0]["nick"]; @@ -259,7 +259,7 @@ function display_content(&$a, $update = 0) { $r = q("SELECT `id` FROM `item` WHERE `item`.`visible` AND NOT `item`.`deleted` AND NOT `item`.`moderated` AND `guid` = '%s' AND `uid` = %d", dbesc($a->argv[1]), local_user()); - if (count($r)) { + if (dba::is_result($r)) { $item_id = $r[0]["id"]; $nick = $a->user["nickname"]; } @@ -273,7 +273,7 @@ function display_content(&$a, $update = 0) { AND NOT `item`.`private` AND NOT `user`.`hidewall` AND `item`.`guid` = '%s'", dbesc($a->argv[1])); // AND NOT `item`.`private` AND `item`.`wall` - if (count($r)) { + if (dba::is_result($r)) { $item_id = $r[0]["id"]; $nick = $r[0]["nickname"]; } @@ -286,7 +286,7 @@ function display_content(&$a, $update = 0) { AND NOT `item`.`private` AND `item`.`uid` = 0 AND `item`.`guid` = '%s'", dbesc($a->argv[1])); // AND NOT `item`.`private` AND `item`.`wall` - if (count($r)) { + if (dba::is_result($r)) { $item_id = $r[0]["id"]; } } @@ -331,7 +331,7 @@ function display_content(&$a, $update = 0) { intval($contact_id), intval($a->profile['uid']) ); - if(count($r)) { + if(dba::is_result($r)) { $contact = $r[0]; $remote_contact = true; } @@ -347,7 +347,7 @@ function display_content(&$a, $update = 0) { $r = q("SELECT * FROM `contact` WHERE `uid` = %d AND `self` LIMIT 1", intval($a->profile['uid']) ); - if(count($r)) + if(dba::is_result($r)) $a->page_contact = $r[0]; $is_owner = ((local_user()) && (local_user() == $a->profile['profile_uid']) ? true : false); diff --git a/mod/editpost.php b/mod/editpost.php index 9a80d0b2f4..bbd21a064d 100644 --- a/mod/editpost.php +++ b/mod/editpost.php @@ -78,7 +78,7 @@ function editpost_content(&$a) { $r = q("SELECT * FROM `mailacct` WHERE `uid` = %d AND `server` != '' LIMIT 1", intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $mail_enabled = true; if(intval($r[0]['pubmail'])) $pubmail_enabled = true; diff --git a/mod/events.php b/mod/events.php index 653ae489b8..1b2d8f9cef 100644 --- a/mod/events.php +++ b/mod/events.php @@ -361,7 +361,7 @@ function events_content(&$a) { $links = array(); - if(count($r)) { + if(dba::is_result($r)) { $r = sort_by_date($r); foreach($r as $rr) { $j = (($rr['adjust']) ? datetime_convert('UTC',date_default_timezone_get(),$rr['start'], 'j') : datetime_convert('UTC','UTC',$rr['start'],'j')); @@ -376,7 +376,7 @@ function events_content(&$a) { $last_date = ''; $fmt = t('l, F j'); - if(count($r)) { + if(dba::is_result($r)) { $r = sort_by_date($r); foreach($r as $rr) { @@ -475,7 +475,7 @@ function events_content(&$a) { intval($event_id), intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $orig_event = $r[0]; } diff --git a/mod/friendica.php b/mod/friendica.php index aad5964baf..2ec8ab95a3 100644 --- a/mod/friendica.php +++ b/mod/friendica.php @@ -24,7 +24,7 @@ function friendica_init(&$a) { $visible_plugins = array(); if(is_array($a->plugins) && count($a->plugins)) { $r = q("select * from addon where hidden = 0"); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $visible_plugins[] = $rr['name']; } @@ -80,7 +80,7 @@ function friendica_content(&$a) { $visible_plugins = array(); if(is_array($a->plugins) && count($a->plugins)) { $r = q("select * from addon where hidden = 0"); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $visible_plugins[] = $rr['name']; } diff --git a/mod/fsuggest.php b/mod/fsuggest.php index 6b1cbd7533..986f92e34b 100644 --- a/mod/fsuggest.php +++ b/mod/fsuggest.php @@ -33,7 +33,7 @@ function fsuggest_post(&$a) { intval($new_contact), intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $x = q("INSERT INTO `fsuggest` ( `uid`,`cid`,`name`,`url`,`request`,`photo`,`note`,`created`) VALUES ( %d, %d, '%s','%s','%s','%s','%s','%s')", @@ -50,7 +50,7 @@ function fsuggest_post(&$a) { dbesc($hash), intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $fsuggest_id = $r[0]['id']; q("UPDATE `fsuggest` SET `note` = '%s' WHERE `id` = %d AND `uid` = %d", dbesc($note), diff --git a/mod/group.php b/mod/group.php index 5b28784f56..94947c1890 100644 --- a/mod/group.php +++ b/mod/group.php @@ -107,7 +107,7 @@ function group_content(&$a) { intval($a->argv[2]), intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $result = group_rmv(local_user(),$r[0]['name']); if($result) info( t('Group removed.') . EOL); @@ -125,7 +125,7 @@ function group_content(&$a) { intval($a->argv[2]), intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $change = intval($a->argv[2]); } @@ -209,7 +209,7 @@ function group_content(&$a) { intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $textmode = (($switchtotext && (count($r) > $switchtotext)) ? true : false); foreach($r as $member) { if(! in_array($member['id'],$preselected)) { diff --git a/mod/item.php b/mod/item.php index d0e1ffed50..935d66e6d8 100644 --- a/mod/item.php +++ b/mod/item.php @@ -129,7 +129,7 @@ function item_post(&$a) { intval($parent_item['contact-id']), intval($uid) ); - if(count($r)) + if(dba::is_result($r)) $parent_contact = $r[0]; // If the contact id doesn't fit with the contact, then set the contact to null @@ -140,7 +140,7 @@ function item_post(&$a) { $r = q("SELECT * FROM `gcontact` WHERE `nurl` = '%s' LIMIT 1", dbesc(normalise_link($thrparent[0]["author-link"]))); - if (count($r)) { + if (dba::is_result($r)) { $parent_contact = $r[0]; $parent_contact["thumb"] = $parent_contact["photo"]; $parent_contact["micro"] = $parent_contact["photo"]; @@ -219,7 +219,7 @@ function item_post(&$a) { $r = q("SELECT * FROM `user` WHERE `uid` = %d LIMIT 1", intval($profile_uid) ); - if(count($r)) + if(dba::is_result($r)) $user = $r[0]; if($orig_post) { @@ -322,7 +322,7 @@ function item_post(&$a) { $r = q("SELECT * FROM `mailacct` WHERE `uid` = %d AND `server` != '' LIMIT 1", intval(local_user()) ); - if(count($r) && intval($r[0]['pubmail'])) + if(dba::is_result($r) && intval($r[0]['pubmail'])) $pubmail_enabled = true; } } @@ -391,7 +391,7 @@ function item_post(&$a) { } } - if(count($r)) { + if(dba::is_result($r)) { $author = $r[0]; $contact_id = $author['id']; } @@ -405,7 +405,7 @@ function item_post(&$a) { $r = q("SELECT * FROM `contact` WHERE `uid` = %d AND `self` = 1 LIMIT 1", intval($profile_uid) ); - if(count($r)) + if(dba::is_result($r)) $contact_record = $r[0]; } @@ -490,7 +490,7 @@ function item_post(&$a) { intval($profile_uid), intval($attach) ); - if(count($r)) { + if(dba::is_result($r)) { $r = q("UPDATE `attach` SET `allow_cid` = '%s', `allow_gid` = '%s', `deny_cid` = '%s', `deny_gid` = '%s' WHERE `uid` = %d AND `id` = %d", dbesc($str_contact_allow), @@ -631,7 +631,7 @@ function item_post(&$a) { intval($profile_uid), intval($mtch) ); - if(count($r)) { + if(dba::is_result($r)) { if(strlen($attachments)) $attachments .= ','; $attachments .= '[attach]href="' . $a->get_baseurl() . '/attach/' . $r[0]['id'] . '" length="' . $r[0]['filesize'] . '" type="' . $r[0]['filetype'] . '" title="' . (($r[0]['filename']) ? $r[0]['filename'] : '') . '"[/attach]'; diff --git a/mod/lockview.php b/mod/lockview.php index 0ae54c8c12..df11a7f6fb 100644 --- a/mod/lockview.php +++ b/mod/lockview.php @@ -52,7 +52,7 @@ function lockview_content(&$a) { $r = q("SELECT `name` FROM `group` WHERE `id` IN ( %s )", dbesc(implode(', ', $allowed_groups)) ); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $l[] = '' . $rr['name'] . ''; } @@ -60,7 +60,7 @@ function lockview_content(&$a) { $r = q("SELECT `name` FROM `contact` WHERE `id` IN ( %s )", dbesc(implode(', ',$allowed_users)) ); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $l[] = $rr['name']; @@ -70,7 +70,7 @@ function lockview_content(&$a) { $r = q("SELECT `name` FROM `group` WHERE `id` IN ( %s )", dbesc(implode(', ', $deny_groups)) ); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $l[] = '' . $rr['name'] . ''; } @@ -78,7 +78,7 @@ function lockview_content(&$a) { $r = q("SELECT `name` FROM `contact` WHERE `id` IN ( %s )", dbesc(implode(', ',$deny_users)) ); - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $l[] = '' . $rr['name'] . ''; diff --git a/mod/manage.php b/mod/manage.php index adcc3d787a..c56e4d8bc9 100644 --- a/mod/manage.php +++ b/mod/manage.php @@ -15,7 +15,7 @@ function manage_post(&$a) { $r = q("select * from user where uid = %d limit 1", intval($_SESSION['submanage']) ); - if(count($r)) { + if(dba::is_result($r)) { $uid = intval($r[0]['uid']); $orig_record = $r[0]; } diff --git a/mod/message.php b/mod/message.php index 433da08a88..2a1152b8c0 100644 --- a/mod/message.php +++ b/mod/message.php @@ -242,7 +242,7 @@ function message_content(&$a) { intval($a->argv[2]), intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $parent = $r[0]['parent-uri']; $convid = $r[0]['convid']; @@ -309,7 +309,7 @@ function message_content(&$a) { intval(local_user()), intval($a->argv[2]) ); - if(count($r)) { + if(dba::is_result($r)) { $prename = $r[0]['name']; $preurl = $r[0]['url']; $preid = $r[0]['id']; @@ -393,7 +393,7 @@ function message_content(&$a) { intval(local_user()), intval($a->argv[1]) ); - if(count($r)) { + if(dba::is_result($r)) { $contact_id = $r[0]['contact-id']; $convid = $r[0]['convid']; diff --git a/mod/mood.php b/mod/mood.php index 5e6ca0fcfc..9be60de1d8 100644 --- a/mod/mood.php +++ b/mod/mood.php @@ -36,7 +36,7 @@ function mood_init(&$a) { intval($parent), intval($uid) ); - if(count($r)) { + if(dba::is_result($r)) { $parent_uri = $r[0]['uri']; $private = $r[0]['private']; $allow_cid = $r[0]['allow_cid']; diff --git a/mod/msearch.php b/mod/msearch.php index 89de5b7057..8393a97ae8 100644 --- a/mod/msearch.php +++ b/mod/msearch.php @@ -13,17 +13,19 @@ function msearch_post(&$a) { $r = q("SELECT COUNT(*) AS `total` FROM `profile` LEFT JOIN `user` ON `user`.`uid` = `profile`.`uid` WHERE `is-default` = 1 AND `user`.`hidewall` = 0 AND MATCH `pub_keywords` AGAINST ('%s') ", dbesc($search) ); - if(count($r)) + + if(dba::is_result($r)) $total = $r[0]['total']; + $results = array(); + $r = q("SELECT `pub_keywords`, `username`, `nickname`, `user`.`uid` FROM `user` LEFT JOIN `profile` ON `user`.`uid` = `profile`.`uid` WHERE `is-default` = 1 AND `user`.`hidewall` = 0 AND MATCH `pub_keywords` AGAINST ('%s') LIMIT %d , %d ", dbesc($search), intval($startrec), intval($perpage) ); - $results = array(); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) $results[] = array( 'name' => $rr['name'], @@ -39,4 +41,4 @@ function msearch_post(&$a) { killme(); -} \ No newline at end of file +} diff --git a/mod/network.php b/mod/network.php index c93e2ad929..28ce8fdbfe 100644 --- a/mod/network.php +++ b/mod/network.php @@ -182,7 +182,7 @@ function saved_searches($search) { $saved = array(); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $saved[] = array( 'id' => $rr['id'], @@ -465,7 +465,7 @@ function network_content(&$a, $update = 0) { ); $str = ''; - if(count($r)) + if(dba::is_result($r)) foreach($r as $rr) $str .= '<' . $rr['id'] . '>'; if(strlen($str)) @@ -587,7 +587,7 @@ function network_content(&$a, $update = 0) { AND `blocked` = 0 AND `pending` = 0 LIMIT 1", intval($cid) ); - if(count($r)) { + if(dba::is_result($r)) { $sql_extra = " AND ".$sql_table.".`contact-id` = ".intval($cid); $entries[0] = array( @@ -689,7 +689,7 @@ function network_content(&$a, $update = 0) { intval($_SESSION['uid']) ); - if(count($r)) { + if(dba::is_result($r)) { $a->set_pager_total($r[0]['total']); } } diff --git a/mod/nogroup.php b/mod/nogroup.php index 9f6e978433..a29cb9c86e 100644 --- a/mod/nogroup.php +++ b/mod/nogroup.php @@ -28,11 +28,11 @@ function nogroup_content(&$a) { require_once('include/Contact.php'); $r = contacts_not_grouped(local_user()); - if(count($r)) { + if(dba::is_result($r)) { $a->set_pager_total($r[0]['total']); } $r = contacts_not_grouped(local_user(),$a->pager['start'],$a->pager['itemspage']); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $contact_details = get_contact_details_by_url($rr['url'], local_user()); diff --git a/mod/noscrape.php b/mod/noscrape.php index 4be1a740cd..7b759d15d8 100644 --- a/mod/noscrape.php +++ b/mod/noscrape.php @@ -39,7 +39,7 @@ function noscrape_init(&$a) { if(is_array($a->profile) AND !$a->profile['hide-friends']) { $r = q("SELECT `gcontact`.`updated` FROM `contact` INNER JOIN `gcontact` WHERE `gcontact`.`nurl` = `contact`.`nurl` AND `self` AND `uid` = %d LIMIT 1", intval($a->profile['uid'])); - if(count($r)) + if(dba::is_result($r)) $json_info["updated"] = date("c", strtotime($r[0]['updated'])); $r = q("SELECT COUNT(*) AS `total` FROM `contact` WHERE `uid` = %d AND `self` = 0 AND `blocked` = 0 and `pending` = 0 AND `hidden` = 0 AND `archive` = 0 @@ -49,7 +49,7 @@ function noscrape_init(&$a) { dbesc(NETWORK_DIASPORA), dbesc(NETWORK_OSTATUS) ); - if(count($r)) + if(dba::is_result($r)) $json_info["contacts"] = intval($r[0]['total']); } diff --git a/mod/notes.php b/mod/notes.php index 73c1507e3e..ba5ef1b6d4 100644 --- a/mod/notes.php +++ b/mod/notes.php @@ -83,7 +83,7 @@ function notes_content(&$a,$update = false) { ); - if(count($r)) { + if(dba::is_result($r)) { $a->set_pager_total($r[0]['total']); $a->set_pager_itemspage(40); } @@ -105,7 +105,7 @@ function notes_content(&$a,$update = false) { $parents_arr = array(); $parents_str = ''; - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) $parents_arr[] = $rr['item_id']; $parents_str = implode(', ', $parents_arr); @@ -124,7 +124,7 @@ function notes_content(&$a,$update = false) { dbesc($parents_str) ); - if(count($r)) { + if(dba::is_result($r)) { $items = conv_sort($r,"`commented`"); $o .= conversation($a,$items,'notes',$update); diff --git a/mod/notice.php b/mod/notice.php index 19cf53189a..161273b0ac 100644 --- a/mod/notice.php +++ b/mod/notice.php @@ -6,7 +6,7 @@ $r = q("SELECT user.nickname FROM user LEFT JOIN item ON item.uid=user.uid WHERE item.id=%d", intval($id) ); - if (count($r)){ + if (dba::is_result($r)){ $nick = $r[0]['nickname']; $url = $a->get_baseurl()."/display/$nick/$id"; goaway($url); diff --git a/mod/notifications.php b/mod/notifications.php index f6c4e8f51f..6d8c68eb35 100644 --- a/mod/notifications.php +++ b/mod/notifications.php @@ -21,7 +21,7 @@ function notifications_post(&$a) { intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $intro_id = $r[0]['id']; $contact_id = $r[0]['contact-id']; } @@ -149,7 +149,7 @@ function notifications_content(&$a) { WHERE `intro`.`uid` = %d $sql_extra AND `intro`.`blocked` = 0 ", intval($_SESSION['uid'])); - if(($r !== false) && (count($r))) { + if(dba::is_result($r)) { $sugg = get_markup_template('suggestions.tpl'); $tpl = get_markup_template("intros.tpl"); @@ -372,7 +372,7 @@ function notifications_content(&$a) { intval(local_user()) ); - if (count($r) > 0) { + if (dba::is_result($r)) { foreach ($r as $it) { $notif_content .= replace_macros($not_tpl,array( '$item_link' => $a->get_baseurl(true).'/notify/view/'. $it['id'], @@ -424,7 +424,7 @@ function notifications_content(&$a) { $notif_content = ''; - if (count($r) > 0) { + if (dba::is_result($r)) { foreach ($r as $it) { switch($it['verb']){ @@ -515,7 +515,7 @@ function notifications_content(&$a) { $notif_content = ''; - if (count($r) > 0) { + if (dba::is_result($r)) { foreach ($r as $it) { switch($it['verb']){ diff --git a/mod/photo.php b/mod/photo.php index 4166b4d539..33a0242352 100644 --- a/mod/photo.php +++ b/mod/photo.php @@ -76,7 +76,7 @@ function photo_init(&$a) { intval($resolution), intval($uid) ); - if(count($r)) { + if(dba::is_result($r)) { $data = $r[0]['data']; $mimetype = $r[0]['type']; } @@ -106,7 +106,7 @@ function photo_init(&$a) { dbesc($photo), intval($resolution) ); - if(count($r)) { + if(dba::is_result($r)) { $sql_extra = permissions_sql($r[0]['uid']); @@ -117,9 +117,9 @@ function photo_init(&$a) { intval($resolution) ); - $public = ($r[0]['allow_cid'] == '') AND ($r[0]['allow_gid'] == '') AND ($r[0]['deny_cid'] == '') AND ($r[0]['deny_gid'] == ''); + $public = (dba::is_result($r)) && ($r[0]['allow_cid'] == '') AND ($r[0]['allow_gid'] == '') AND ($r[0]['deny_cid'] == '') AND ($r[0]['deny_gid'] == ''); - if(count($r)) { + if(dba::is_result($r)) { $resolution = $r[0]['scale']; $data = $r[0]['data']; $mimetype = $r[0]['type']; diff --git a/mod/photos.php b/mod/photos.php index 4761b627d8..b05bc12b1a 100644 --- a/mod/photos.php +++ b/mod/photos.php @@ -159,7 +159,7 @@ function photos_post(&$a) { intval($cid), intval($page_owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $can_post = true; $visitor = $cid; } @@ -267,7 +267,7 @@ function photos_post(&$a) { dbesc($album) ); } - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $res[] = "'" . dbesc($rr['rid']) . "'" ; } @@ -290,7 +290,7 @@ function photos_post(&$a) { $r = q("SELECT `parent-uri` FROM `item` WHERE `resource-id` IN ( $str_res ) AND `uid` = %d", intval($page_owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { q("UPDATE `item` SET `deleted` = 1, `changed` = '%s' WHERE `parent-uri` = '%s' AND `uid` = %d", dbesc(datetime_convert()), @@ -352,7 +352,7 @@ function photos_post(&$a) { dbesc($a->argv[2]) ); } - if(count($r)) { + if(dba::is_result($r)) { q("DELETE FROM `photo` WHERE `uid` = %d AND `resource-id` = '%s'", intval($page_owner_uid), dbesc($r[0]['resource-id']) @@ -408,7 +408,7 @@ function photos_post(&$a) { dbesc($resource_id), intval($page_owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $ph = new Photo($r[0]['data'], $r[0]['type']); if($ph->is_valid()) { $rotate_deg = ( (intval($_POST['rotate']) == 1) ? 270 : 90 ); @@ -525,7 +525,7 @@ function photos_post(&$a) { intval($page_owner_uid) ); } - if(count($r)) { + if(dba::is_result($r)) { $old_tag = $r[0]['tag']; $old_inform = $r[0]['inform']; } @@ -612,7 +612,7 @@ function photos_post(&$a) { intval($page_owner_uid) ); }*/ - if(count($r)) { + if(dba::is_result($r)) { $newname = $r[0]['name']; $profile = $r[0]['url']; $notify = 'cid:' . $r[0]['id']; @@ -1028,7 +1028,7 @@ function photos_content(&$a) { intval($contact_id), intval($owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $can_post = true; $contact = $r[0]; $remote_contact = true; @@ -1056,7 +1056,7 @@ function photos_content(&$a) { intval($contact_id), intval($owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $contact = $r[0]; $remote_contact = true; } @@ -1209,8 +1209,8 @@ function photos_content(&$a) { intval($owner_uid), dbesc($album) ); - if(count($r)) { - $a->set_pager_total(count($r)); + if(dba::is_result($r)) { + $a->set_pager_total(dba::is_result($r)); $a->set_pager_itemspage(20); } @@ -1266,7 +1266,7 @@ function photos_content(&$a) { $photos = array(); - if(count($r)) + if(dba::is_result($r)) $twist = 'rotright'; foreach($r as $rr) { if($twist == 'rotright') @@ -1461,7 +1461,7 @@ function photos_content(&$a) { ); - if(count($r)) + if(dba::is_result($r)) $a->set_pager_total($r[0]['total']); @@ -1638,7 +1638,7 @@ function photos_content(&$a) { // display comments - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $item) { builtin_activity_puller($item, $conv_responses); @@ -1824,8 +1824,8 @@ function photos_content(&$a) { dbesc('Contact Photos'), dbesc( t('Contact Photos')) ); - if(count($r)) { - $a->set_pager_total(count($r)); + if(dba::is_result($r)) { + $a->set_pager_total(dba::is_result($r)); $a->set_pager_itemspage(20); } @@ -1842,7 +1842,7 @@ function photos_content(&$a) { $photos = array(); - if(count($r)) { + if(dba::is_result($r)) { $twist = 'rotright'; foreach($r as $rr) { //hide profile photos to others diff --git a/mod/poco.php b/mod/poco.php index 0a1b392169..372f5b45b6 100644 --- a/mod/poco.php +++ b/mod/poco.php @@ -81,7 +81,7 @@ function poco_init(&$a) { dbesc(NETWORK_STATUSNET) ); } - if(count($r)) + if(dba::is_result($r)) $totalResults = intval($r[0]['total']); else $totalResults = 0; @@ -168,7 +168,7 @@ function poco_init(&$a) { } if(is_array($r)) { - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if (!isset($rr['generation'])) { if ($global) diff --git a/mod/poke.php b/mod/poke.php index 4a643435be..f59ec18697 100644 --- a/mod/poke.php +++ b/mod/poke.php @@ -66,7 +66,7 @@ function poke_init(&$a) { intval($parent), intval($uid) ); - if(count($r)) { + if(dba::is_result($r)) { $parent_uri = $r[0]['uri']; $private = $r[0]['private']; $allow_cid = $r[0]['allow_cid']; @@ -159,7 +159,7 @@ function poke_content(&$a) { intval($_GET['c']), intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $name = $r[0]['name']; $id = $r[0]['id']; } diff --git a/mod/profile.php b/mod/profile.php index c3f58ab528..66d7ce3999 100644 --- a/mod/profile.php +++ b/mod/profile.php @@ -13,7 +13,7 @@ function profile_init(&$a) { $which = $a->argv[1]; else { $r = q("select nickname from user where blocked = 0 and account_expired = 0 and account_removed = 0 and verified = 1 order by rand() limit 1"); - if(count($r)) { + if(dba::is_result($r)) { goaway($a->get_baseurl() . '/profile/' . $r[0]['nickname']); } else { @@ -136,7 +136,7 @@ function profile_content(&$a, $update = 0) { intval($contact_id), intval($a->profile['profile_uid']) ); - if(count($r)) { + if(dba::is_result($r)) { $contact = $r[0]; $remote_contact = true; } @@ -257,21 +257,21 @@ function profile_content(&$a, $update = 0) { AND `thread`.`wall` = 1 $sql_extra $sql_extra2 ", intval($a->profile['profile_uid']) - ); + ); - if(count($r)) { - $a->set_pager_total($r[0]['total']); + if(dba::is_result($r)) { + $a->set_pager_total($r[0]['total']); } } // check if we serve a mobile device and get the user settings // accordingly if ($a->is_mobile) { - $itemspage_network = get_pconfig(local_user(),'system','itemspage_mobile_network'); - $itemspage_network = ((intval($itemspage_network)) ? $itemspage_network : 10); + $itemspage_network = get_pconfig(local_user(),'system','itemspage_mobile_network'); + $itemspage_network = ((intval($itemspage_network)) ? $itemspage_network : 10); } else { - $itemspage_network = get_pconfig(local_user(),'system','itemspage_network'); - $itemspage_network = ((intval($itemspage_network)) ? $itemspage_network : 20); + $itemspage_network = get_pconfig(local_user(),'system','itemspage_network'); + $itemspage_network = ((intval($itemspage_network)) ? $itemspage_network : 20); } // now that we have the user settings, see if the theme forces // a maximum item number which is lower then the user choice diff --git a/mod/profile_photo.php b/mod/profile_photo.php index 4e8d279a97..75b364d2b9 100644 --- a/mod/profile_photo.php +++ b/mod/profile_photo.php @@ -32,7 +32,7 @@ function profile_photo_post(&$a) { intval($_REQUEST['profile']), intval(local_user()) ); - if(count($r) && (! intval($r[0]['is-default']))) + if(dba::is_result($r) && (! intval($r[0]['is-default']))) $is_default_profile = 0; } @@ -63,7 +63,7 @@ function profile_photo_post(&$a) { dbesc(local_user()), intval($scale)); - if(count($r)) { + if(dba::is_result($r)) { $base_image = $r[0]; diff --git a/mod/profiles.php b/mod/profiles.php index 39382fbdd5..dfab29894e 100644 --- a/mod/profiles.php +++ b/mod/profiles.php @@ -285,7 +285,7 @@ function profiles_post(&$a) { intval(local_user()) ); } - if(count($r)) { + if(dba::is_result($r)) { $prf = $r[0]['url']; $newname = $r[0]['name']; } @@ -778,7 +778,7 @@ function profiles_content(&$a) { "SELECT * FROM `profile` WHERE `uid` = %d AND `is-default`=1", local_user() ); - if(count($r)){ + if(dba::is_result($r)){ //Go to the default profile. goaway('profiles/'.$r[0]['id']); } @@ -786,7 +786,7 @@ function profiles_content(&$a) { $r = q("SELECT * FROM `profile` WHERE `uid` = %d", local_user()); - if(count($r)) { + if(dba::is_result($r)) { $tpl_header = get_markup_template('profile_listing_header.tpl'); $o .= replace_macros($tpl_header,array( diff --git a/mod/profperm.php b/mod/profperm.php index 077f695bea..91b84e2b94 100644 --- a/mod/profperm.php +++ b/mod/profperm.php @@ -42,7 +42,7 @@ function profperm_content(&$a) { intval($a->argv[2]), intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $change = intval($a->argv[2]); } @@ -64,7 +64,7 @@ function profperm_content(&$a) { ); $ingroup = array(); - if(count($r)) + if(dba::is_result($r)) foreach($r as $member) $ingroup[] = $member['id']; @@ -94,7 +94,7 @@ function profperm_content(&$a) { $members = $r; $ingroup = array(); - if(count($r)) + if(dba::is_result($r)) foreach($r as $member) $ingroup[] = $member['id']; } @@ -138,7 +138,7 @@ function profperm_content(&$a) { dbesc(NETWORK_DFRN) ); - if(count($r)) { + if(dba::is_result($r)) { $textmode = (($switchtotext && (count($r) > $switchtotext)) ? true : false); foreach($r as $member) { if(! in_array($member['id'],$ingroup)) { diff --git a/mod/proxy.php b/mod/proxy.php index abcaf49127..02def84ca1 100644 --- a/mod/proxy.php +++ b/mod/proxy.php @@ -136,7 +136,7 @@ function proxy_init() { if (!$direct_cache AND ($cachefile == "")) { $r = q("SELECT * FROM `photo` WHERE `resource-id` = '%s' LIMIT 1", $urlhash); - if (count($r)) { + if (dba::is_result($r)) { $img_str = $r[0]['data']; $mime = $r[0]["desc"]; if ($mime == "") $mime = "image/jpeg"; diff --git a/mod/pubsubhubbub.php b/mod/pubsubhubbub.php index 5d7621cc74..a50efc9229 100644 --- a/mod/pubsubhubbub.php +++ b/mod/pubsubhubbub.php @@ -132,7 +132,7 @@ function pubsubhubbub_init(&$a) { // if we are just updating an old subscription, keep the // old values for push and last_update - if (count($r)) { + if (dba::is_result($r)) { $last_update = $r[0]['last_update']; $push_flag = $r[0]['push']; } diff --git a/mod/qsearch.php b/mod/qsearch.php index c35e253b67..0eabf6b7c6 100644 --- a/mod/qsearch.php +++ b/mod/qsearch.php @@ -23,7 +23,7 @@ function qsearch_init(&$a) { intval($limit) ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) $results[] = array( 0, (int) $rr['id'], $rr['name'], '', ''); @@ -38,7 +38,7 @@ function qsearch_init(&$a) { ); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) $results[] = array( (int) $rr['id'], 0, $rr['name'],$rr['url'],$rr['photo']); diff --git a/mod/regmod.php b/mod/regmod.php index 5a90db1f90..47258e0914 100644 --- a/mod/regmod.php +++ b/mod/regmod.php @@ -34,7 +34,7 @@ function user_allow($hash) { $r = q("SELECT * FROM `profile` WHERE `uid` = %d AND `is-default` = 1", intval($user[0]['uid']) ); - if(count($r) && $r[0]['net-publish']) { + if(dba::is_result($r) && $r[0]['net-publish']) { $url = $a->get_baseurl() . '/profile/' . $user[0]['nickname']; if($url && strlen(get_config('system','directory'))) proc_run('php',"include/directory.php","$url"); diff --git a/mod/salmon.php b/mod/salmon.php index 37230a5573..ec0693b941 100644 --- a/mod/salmon.php +++ b/mod/salmon.php @@ -169,8 +169,8 @@ function salmon_post(&$a) { // Have we ignored the person? // If so we can not accept this post. - //if((count($r)) && (($r[0]['readonly']) || ($r[0]['rel'] == CONTACT_IS_FOLLOWER) || ($r[0]['blocked']))) { - if(count($r) && $r[0]['blocked']) { + //if((dba::is_result($r)) && (($r[0]['readonly']) || ($r[0]['rel'] == CONTACT_IS_FOLLOWER) || ($r[0]['blocked']))) { + if(dba::is_result($r) && $r[0]['blocked']) { logger('mod-salmon: Ignoring this author.'); http_status_exit(202); // NOTREACHED @@ -179,7 +179,7 @@ function salmon_post(&$a) { // Placeholder for hub discovery. $hub = ''; - $contact_rec = ((count($r)) ? $r[0] : null); + $contact_rec = ((dba::is_result($r)) ? $r[0] : null); ostatus::import($data,$importer,$contact_rec, $hub); diff --git a/mod/search.php b/mod/search.php index 790f577ba6..9ad1fe5077 100644 --- a/mod/search.php +++ b/mod/search.php @@ -15,7 +15,7 @@ function search_saved_searches() { intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $saved = array(); foreach($r as $rr) { $saved[] = array( diff --git a/mod/settings.php b/mod/settings.php index c7659212bf..8417423639 100644 --- a/mod/settings.php +++ b/mod/settings.php @@ -254,7 +254,7 @@ function settings_post(&$a) { $r = q("SELECT * FROM `mailacct` WHERE `uid` = %d LIMIT 1", intval(local_user()) ); - if(count($r)) { + if(dba::is_result($r)) { $eacct = $r[0]; require_once('include/email.php'); $mb = construct_mailbox_name($eacct); @@ -842,15 +842,15 @@ function settings_content(&$a) { $r = null; } - $mail_server = ((count($r)) ? $r[0]['server'] : ''); - $mail_port = ((count($r) && intval($r[0]['port'])) ? intval($r[0]['port']) : ''); - $mail_ssl = ((count($r)) ? $r[0]['ssltype'] : ''); - $mail_user = ((count($r)) ? $r[0]['user'] : ''); - $mail_replyto = ((count($r)) ? $r[0]['reply_to'] : ''); - $mail_pubmail = ((count($r)) ? $r[0]['pubmail'] : 0); - $mail_action = ((count($r)) ? $r[0]['action'] : 0); - $mail_movetofolder = ((count($r)) ? $r[0]['movetofolder'] : ''); - $mail_chk = ((count($r)) ? $r[0]['last_check'] : '0000-00-00 00:00:00'); + $mail_server = ((dba::is_result($r)) ? $r[0]['server'] : ''); + $mail_port = ((dba::is_result($r) && intval($r[0]['port'])) ? intval($r[0]['port']) : ''); + $mail_ssl = ((dba::is_result($r)) ? $r[0]['ssltype'] : ''); + $mail_user = ((dba::is_result($r)) ? $r[0]['user'] : ''); + $mail_replyto = ((dba::is_result($r)) ? $r[0]['reply_to'] : ''); + $mail_pubmail = ((dba::is_result($r)) ? $r[0]['pubmail'] : 0); + $mail_action = ((dba::is_result($r)) ? $r[0]['action'] : 0); + $mail_movetofolder = ((dba::is_result($r)) ? $r[0]['movetofolder'] : ''); + $mail_chk = ((dba::is_result($r)) ? $r[0]['last_check'] : '0000-00-00 00:00:00'); $tpl = get_markup_template("settings_connectors.tpl"); diff --git a/mod/subthread.php b/mod/subthread.php index 33cf7489c1..4b417b75b9 100644 --- a/mod/subthread.php +++ b/mod/subthread.php @@ -53,7 +53,7 @@ function subthread_content(&$a) { WHERE `contact`.`self` = 1 AND `contact`.`uid` = %d LIMIT 1", intval($owner_uid) ); - if(count($r)) + if(dba::is_result($r)) $owner = $r[0]; if(! $owner) { @@ -75,7 +75,7 @@ function subthread_content(&$a) { intval($_SESSION['visitor_id']), intval($owner_uid) ); - if(count($r)) + if(dba::is_result($r)) $contact = $r[0]; } if(! $contact) { diff --git a/mod/tagger.php b/mod/tagger.php index 26166a3cc0..78bf11cc4c 100644 --- a/mod/tagger.php +++ b/mod/tagger.php @@ -39,7 +39,7 @@ function tagger_content(&$a) { $r = q("select `nickname`,`blocktags` from user where uid = %d limit 1", intval($owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $owner_nick = $r[0]['nickname']; $blocktags = $r[0]['blocktags']; } @@ -50,7 +50,7 @@ function tagger_content(&$a) { $r = q("select * from contact where self = 1 and uid = %d limit 1", intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $contact = $r[0]; else { logger('tagger: no contact_id'); @@ -178,7 +178,7 @@ EOT; $r = q("select `tag`,`id`,`uid` from item where `origin` = 1 AND `uri` = '%s' LIMIT 1", dbesc($item['uri']) ); - if(count($r)) { + if(dba::is_result($r)) { $x = q("SELECT `blocktags` FROM `user` WHERE `uid` = %d limit 1", intval($r[0]['uid']) ); diff --git a/mod/uexport.php b/mod/uexport.php index 3114add7e4..dde5fe090d 100644 --- a/mod/uexport.php +++ b/mod/uexport.php @@ -44,7 +44,7 @@ function uexport_content(&$a){ function _uexport_multirow($query) { $result = array(); $r = q($query); -// if(count($r)) { +// if(dba::is_result($r)) { if ($r){ foreach($r as $rr){ $p = array(); @@ -130,7 +130,7 @@ function uexport_all(&$a) { $r = q("SELECT count(*) as `total` FROM `item` WHERE `uid` = %d ", intval(local_user()) ); - if(count($r)) + if(dba::is_result($r)) $total = $r[0]['total']; // chunk the output to avoid exhausting memory @@ -142,7 +142,7 @@ function uexport_all(&$a) { intval($x), intval(500) ); - /*if(count($r)) { + /*if(dba::is_result($r)) { foreach($r as $rr) foreach($rr as $k => $v) $item[][$k] = $v; diff --git a/mod/videos.php b/mod/videos.php index bf8d696b60..2cd013b008 100644 --- a/mod/videos.php +++ b/mod/videos.php @@ -143,7 +143,7 @@ function videos_post(&$a) { dbesc($video_id) ); - if(count($r)) { + if(dba::is_result($r)) { q("DELETE FROM `attach` WHERE `uid` = %d AND `id` = '%s'", intval(local_user()), dbesc($video_id) @@ -262,7 +262,7 @@ function videos_content(&$a) { intval($contact_id), intval($owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $can_post = true; $contact = $r[0]; $remote_contact = true; @@ -290,7 +290,7 @@ function videos_content(&$a) { intval($contact_id), intval($owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $contact = $r[0]; $remote_contact = true; } @@ -350,8 +350,8 @@ function videos_content(&$a) { $sql_extra GROUP BY hash", intval($a->data['user']['uid']) ); - if(count($r)) { - $a->set_pager_total(count($r)); + if(dba::is_result($r)) { + $a->set_pager_total(dba::is_result($r)); $a->set_pager_itemspage(20); } @@ -366,7 +366,7 @@ function videos_content(&$a) { $videos = array(); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if($a->theme['template_engine'] === 'internal') { $alt_e = template_escape($rr['filename']); diff --git a/mod/viewcontacts.php b/mod/viewcontacts.php index 04520e0d93..43e3151308 100644 --- a/mod/viewcontacts.php +++ b/mod/viewcontacts.php @@ -54,7 +54,7 @@ function viewcontacts_content(&$a) { dbesc(NETWORK_DIASPORA), dbesc(NETWORK_OSTATUS) ); - if(count($r)) + if(dba::is_result($r)) $a->set_pager_total($r[0]['total']); $r = q("SELECT * FROM `contact` diff --git a/mod/viewsrc.php b/mod/viewsrc.php index 3fa4eaed53..0c10cbf7c8 100644 --- a/mod/viewsrc.php +++ b/mod/viewsrc.php @@ -24,7 +24,7 @@ function viewsrc_content(&$a) { dbesc($item_id) ); - if(count($r)) + if(dba::is_result($r)) if(is_ajax()) { echo str_replace("\n",'
',$r[0]['body']); killme(); diff --git a/mod/wall_attach.php b/mod/wall_attach.php index 68752a0e1f..9a041d9448 100644 --- a/mod/wall_attach.php +++ b/mod/wall_attach.php @@ -55,7 +55,7 @@ function wall_attach_post(&$a) { intval($cid), intval($page_owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $can_post = true; $visitor = $cid; } diff --git a/mod/wall_upload.php b/mod/wall_upload.php index b815348c70..ac7dd77649 100644 --- a/mod/wall_upload.php +++ b/mod/wall_upload.php @@ -63,7 +63,7 @@ function wall_upload_post(&$a, $desktopmode = true) { intval($cid), intval($page_owner_uid) ); - if(count($r)) { + if(dba::is_result($r)) { $can_post = true; $visitor = $cid; } diff --git a/object/Item.php b/object/Item.php index 59659cdaff..e1a46532a1 100644 --- a/object/Item.php +++ b/object/Item.php @@ -225,7 +225,7 @@ class Item extends BaseObject { intval($item['uid']), intval($item['id']) ); - if (count($r)) { + if (dba::is_result($r)) { $ignore = array( 'do' => t("ignore thread"), 'undo' => t("unignore thread"), diff --git a/tests/get_tags_test.php b/tests/get_tags_test.php index 40f016747f..79dcb36a7f 100644 --- a/tests/get_tags_test.php +++ b/tests/get_tags_test.php @@ -55,13 +55,13 @@ function q($sql) { return $result; } //second call in handle_body, name - if($result[0]['name']===$args[1]) { - return $result; + if($result[0]['name']===$args[1]) { + return $result; } } //third call in handle_body, nick or attag - if($result[0]['nick']===$args[2] || $result[0]['attag']===$args[1]) { - return $result; + if($result[0]['nick']===$args[2] || $result[0]['attag']===$args[1]) { + return $result; } } @@ -88,23 +88,23 @@ function dbesc($str) { */ class GetTagsTest extends PHPUnit_Framework_TestCase { /** the mock to use as app */ - private $a; + private $a; /** * initialize the test. That's a phpUnit function, * don't change its name. - */ - public function setUp() { + */ + public function setUp() { $this->a=new MockApp(); } - /** - * test with one Person tag - */ - public function testGetTagsShortPerson() { - $text="hi @Mike"; - - $tags=get_tags($text); + /** + * test with one Person tag + */ + public function testGetTagsShortPerson() { + $text="hi @Mike"; + + $tags=get_tags($text); $inform=''; $str_tags=''; @@ -113,32 +113,32 @@ class GetTagsTest extends PHPUnit_Framework_TestCase { } //correct tags found? - $this->assertEquals(1, count($tags)); + $this->assertEquals(1, count($tags)); $this->assertTrue(in_array("@Mike", $tags)); //correct output from handle_tag? $this->assertEquals("cid:15", $inform); $this->assertEquals("@[url=http://justatest.de]Mike Lastname[/url]", $str_tags); - $this->assertEquals("hi @[url=http://justatest.de]Mike Lastname[/url]", $text); + $this->assertEquals("hi @[url=http://justatest.de]Mike Lastname[/url]", $text); } - /** + /** * test with one Person tag. - * There's a minor spelling mistake... - */ - public function testGetTagsShortPersonSpelling() { - $text="hi @Mike.because"; - - $tags=get_tags($text); + * There's a minor spelling mistake... + */ + public function testGetTagsShortPersonSpelling() { + $text="hi @Mike.because"; - //correct tags found? - $this->assertEquals(1, count($tags)); + $tags=get_tags($text); + + //correct tags found? + $this->assertEquals(1, count($tags)); $this->assertTrue(in_array("@Mike.because", $tags)); - - $inform=''; - $str_tags=''; - handle_tag($this->a, $text, $inform, $str_tags, 11, $tags[0]); - + + $inform=''; + $str_tags=''; + handle_tag($this->a, $text, $inform, $str_tags, 11, $tags[0]); + // (mike) - This is a tricky case. // we support mentions as in @mike@example.com - which contains a period. // This shouldn't match anything unless you have a contact named "Mike.because". @@ -147,91 +147,91 @@ class GetTagsTest extends PHPUnit_Framework_TestCase { // $this->assertEquals("cid:15", $inform); // $this->assertEquals("@[url=http://justatest.de]Mike Lastname[/url]", $str_tags); -// $this->assertEquals("hi @[url=http://justatest.de]Mike Lastname[/url].because", $text); +// $this->assertEquals("hi @[url=http://justatest.de]Mike Lastname[/url].because", $text); $this->assertEquals("", $inform); $this->assertEquals("", $str_tags); } - /** + /** * test with two Person tags. - * There's a minor spelling mistake... - */ + * There's a minor spelling mistake... + */ - public function testGetTagsPerson2Spelling() { - $text="hi @Mike@campino@friendica.eu"; - - $tags=get_tags($text); + public function testGetTagsPerson2Spelling() { + $text="hi @Mike@campino@friendica.eu"; + + $tags=get_tags($text); // This construct is not supported. Results are indeterminate -// $this->assertEquals(2, count($tags)); +// $this->assertEquals(2, count($tags)); // $this->assertTrue(in_array("@Mike", $tags)); -// $this->assertTrue(in_array("@campino@friendica.eu", $tags)); - } +// $this->assertTrue(in_array("@campino@friendica.eu", $tags)); + } /** * Test with one hash tag. - */ - public function testGetTagsShortTag() { - $text="This is a #test_case"; - - $tags=get_tags($text); + */ + public function testGetTagsShortTag() { + $text="This is a #test_case"; - $this->assertEquals(1, count($tags)); - $this->assertTrue(in_array("#test_case", $tags)); - } + $tags=get_tags($text); + + $this->assertEquals(1, count($tags)); + $this->assertTrue(in_array("#test_case", $tags)); + } /** * test with a person and a hash tag - */ - public function testGetTagsShortTagAndPerson() { - $text="hi @Mike This is a #test_case"; - - $tags=get_tags($text); + */ + public function testGetTagsShortTagAndPerson() { + $text="hi @Mike This is a #test_case"; + + $tags=get_tags($text); $this->assertEquals(3, count($tags)); $this->assertTrue(in_array("@Mike", $tags)); - $this->assertTrue(in_array("@Mike This", $tags)); - $this->assertTrue(in_array("#test_case", $tags)); + $this->assertTrue(in_array("@Mike This", $tags)); + $this->assertTrue(in_array("#test_case", $tags)); $inform=''; $str_tags=''; foreach($tags as $tag) { handle_tag($this->a, $text, $inform, $str_tags, 11, $tag); } - + $this->assertEquals("cid:15", $inform); $this->assertEquals("@[url=http://justatest.de]Mike Lastname[/url],#[url=baseurl/search?tag=test%20case]test case[/url]", $str_tags); $this->assertEquals("hi @[url=http://justatest.de]Mike Lastname[/url] This is a #[url=baseurl/search?tag=test%20case]test case[/url]", $text); - - } + + } /** * test with a person, a hash tag and some special chars. - */ - public function testGetTagsShortTagAndPersonSpecialChars() { - $text="hi @Mike, This is a #test_case."; - - $tags=get_tags($text); - + */ + public function testGetTagsShortTagAndPersonSpecialChars() { + $text="hi @Mike, This is a #test_case."; + + $tags=get_tags($text); + $this->assertEquals(2, count($tags)); $this->assertTrue(in_array("@Mike", $tags)); - $this->assertTrue(in_array("#test_case", $tags)); - } + $this->assertTrue(in_array("#test_case", $tags)); + } /** * Test with a person tag and text behind it. - */ - public function testGetTagsPersonOnly() { - $text="@Test I saw the Theme Dev group was created."; - - $tags=get_tags($text); + */ + public function testGetTagsPersonOnly() { + $text="@Test I saw the Theme Dev group was created."; - $this->assertEquals(2, count($tags)); + $tags=get_tags($text); + + $this->assertEquals(2, count($tags)); $this->assertTrue(in_array("@Test I", $tags)); - $this->assertTrue(in_array("@Test", $tags)); - } + $this->assertTrue(in_array("@Test", $tags)); + } /** * this test demonstrates strange behaviour by intval. @@ -255,72 +255,72 @@ class GetTagsTest extends PHPUnit_Framework_TestCase { //happens right now, but it shouldn't be necessary $this->assertTrue(in_array("@mike+15 id", $tags)); - $inform=''; + $inform=''; $str_tags=''; foreach($tags as $tag) { handle_tag($this->a, $text, $inform, $str_tags, 11, $tag); } - $this->assertEquals("Test with @[url=http://justatest.de]Mike Lastname[/url] id tag", $text); - $this->assertEquals("@[url=http://justatest.de]Mike Lastname[/url]", $str_tags); + $this->assertEquals("Test with @[url=http://justatest.de]Mike Lastname[/url] id tag", $text); + $this->assertEquals("@[url=http://justatest.de]Mike Lastname[/url]", $str_tags); // this test may produce two cid:15 entries - which is OK because duplicates are pruned before delivery $this->assertContains("cid:15",$inform); } /** * test with two persons and one special tag. - */ - public function testGetTags2Persons1TagSpecialChars() { - $text="hi @Mike, I'm just writing #test_cases, so" - ." so @somebody@friendica.com may change #things."; - - $tags=get_tags($text); + */ + public function testGetTags2Persons1TagSpecialChars() { + $text="hi @Mike, I'm just writing #test_cases, so" + ." so @somebody@friendica.com may change #things."; - $this->assertEquals(5, count($tags)); - $this->assertTrue(in_array("@Mike", $tags)); + $tags=get_tags($text); + + $this->assertEquals(5, count($tags)); + $this->assertTrue(in_array("@Mike", $tags)); $this->assertTrue(in_array("#test_cases", $tags)); - $this->assertTrue(in_array("@somebody@friendica.com", $tags)); - $this->assertTrue(in_array("@somebody@friendica.com may", $tags)); - $this->assertTrue(in_array("#things", $tags)); - } + $this->assertTrue(in_array("@somebody@friendica.com", $tags)); + $this->assertTrue(in_array("@somebody@friendica.com may", $tags)); + $this->assertTrue(in_array("#things", $tags)); + } /** * test with a long text. - */ - public function testGetTags() { - $text="hi @Mike, I'm just writing #test_cases, " - ." so @somebody@friendica.com may change #things. Of course I " - ."look for a lot of #pitfalls, like #tags at the end of a sentence " - ."@comment. I hope noone forgets about @fullstops.because that might" - ." break #things. @Mike@campino@friendica.eu is also #nice, isn't it? " - ."Now, add a @first_last tag. "; - - $tags=get_tags($text); - - $this->assertTrue(in_array("@Mike", $tags)); - $this->assertTrue(in_array("#test_cases", $tags)); - $this->assertTrue(in_array("@somebody@friendica.com", $tags)); - $this->assertTrue(in_array("#things", $tags)); - $this->assertTrue(in_array("#pitfalls", $tags)); - $this->assertTrue(in_array("#tags", $tags)); - $this->assertTrue(in_array("@comment", $tags)); - $this->assertTrue(in_array("@fullstops.because", $tags)); - $this->assertTrue(in_array("#things", $tags)); - $this->assertTrue(in_array("@Mike", $tags)); - $this->assertTrue(in_array("#nice", $tags)); + */ + public function testGetTags() { + $text="hi @Mike, I'm just writing #test_cases, " + ." so @somebody@friendica.com may change #things. Of course I " + ."look for a lot of #pitfalls, like #tags at the end of a sentence " + ."@comment. I hope noone forgets about @fullstops.because that might" + ." break #things. @Mike@campino@friendica.eu is also #nice, isn't it? " + ."Now, add a @first_last tag. "; + + $tags=get_tags($text); + + $this->assertTrue(in_array("@Mike", $tags)); + $this->assertTrue(in_array("#test_cases", $tags)); + $this->assertTrue(in_array("@somebody@friendica.com", $tags)); + $this->assertTrue(in_array("#things", $tags)); + $this->assertTrue(in_array("#pitfalls", $tags)); + $this->assertTrue(in_array("#tags", $tags)); + $this->assertTrue(in_array("@comment", $tags)); + $this->assertTrue(in_array("@fullstops.because", $tags)); + $this->assertTrue(in_array("#things", $tags)); + $this->assertTrue(in_array("@Mike", $tags)); + $this->assertTrue(in_array("#nice", $tags)); $this->assertTrue(in_array("@first_last", $tags)); //right now, none of the is matched (unsupported) -// $this->assertFalse(in_array("@Mike@campino@friendica.eu", $tags)); +// $this->assertFalse(in_array("@Mike@campino@friendica.eu", $tags)); // $this->assertTrue(in_array("@campino@friendica.eu", $tags)); -// $this->assertTrue(in_array("@campino@friendica.eu is", $tags)); - } +// $this->assertTrue(in_array("@campino@friendica.eu is", $tags)); + } /** * test with an empty string - */ - public function testGetTagsEmpty() { - $tags=get_tags(""); - $this->assertEquals(0, count($tags)); + */ + public function testGetTagsEmpty() { + $tags=get_tags(""); + $this->assertEquals(0, count($tags)); } -} \ No newline at end of file +} diff --git a/update.php b/update.php index b74edf7bbd..5ab3e69f3b 100644 --- a/update.php +++ b/update.php @@ -85,7 +85,7 @@ function update_1006() { // create 's' keys for everybody that does not have one $r = q("SELECT * FROM `user` WHERE `spubkey` = '' "); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $sres=openssl_pkey_new(array('encrypt_key' => false )); $sprvkey = ''; @@ -122,7 +122,7 @@ function update_1010() { function update_1011() { q("ALTER TABLE `contact` ADD `nick` CHAR( 255 ) NOT NULL AFTER `name` "); $r = q("SELECT * FROM `contact` WHERE 1"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { q("UPDATE `contact` SET `nick` = '%s' WHERE `id` = %d", dbesc(basename($rr['url'])), @@ -145,7 +145,7 @@ function update_1014() { require_once('include/Photo.php'); q("ALTER TABLE `contact` ADD `micro` TEXT NOT NULL AFTER `thumb` "); $r = q("SELECT * FROM `photo` WHERE `scale` = 4"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $ph = new Photo($rr['data']); if($ph->is_valid()) { @@ -155,7 +155,7 @@ function update_1014() { } } $r = q("SELECT * FROM `contact` WHERE 1"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { if(stristr($rr['thumb'],'avatar')) q("UPDATE `contact` SET `micro` = '%s' WHERE `id` = %d", @@ -356,7 +356,7 @@ function update_1035() { function update_1036() { $r = dbq("SELECT * FROM `contact` WHERE `network` = 'dfrn' && `photo` LIKE '%include/photo%' "); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { q("UPDATE `contact` SET `photo` = '%s', `thumb` = '%s', `micro` = '%s' WHERE `id` = %d", dbesc(str_replace('include/photo','photo',$rr['photo'])), @@ -594,7 +594,7 @@ function update_1073() { function update_1074() { q("ALTER TABLE `user` ADD `hidewall` TINYINT( 1) NOT NULL DEFAULT '0' AFTER `blockwall` "); $r = q("SELECT `uid` FROM `profile` WHERE `is-default` = 1 AND `hidewall` = 1"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) q("UPDATE `user` SET `hidewall` = 1 WHERE `uid` = %d", intval($rr['uid']) @@ -606,7 +606,7 @@ function update_1074() { function update_1075() { q("ALTER TABLE `user` ADD `guid` CHAR( 16 ) NOT NULL AFTER `uid` "); $r = q("SELECT `uid` FROM `user` WHERE 1"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $found = true; do { @@ -688,7 +688,7 @@ function update_1082() { if($r && count($r)) return; $r = q("SELECT distinct(`resource-id`) FROM `photo` WHERE 1 group by `id`"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $guid = get_guid(); q("update `photo` set `guid` = '%s' where `resource-id` = '%s'", @@ -731,7 +731,7 @@ function update_1087() { q("ALTER TABLE `item` ADD `commented` datetime NOT NULL DEFAULT '0000-00-00 00:00:00' AFTER `edited` "); $r = q("SELECT `id` FROM `item` WHERE `parent` = `id` "); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $x = q("SELECT max(`created`) AS `cdate` FROM `item` WHERE `parent` = %d LIMIT 1", intval($rr['id']) @@ -854,7 +854,7 @@ function update_1100() { require_once('include/text.php'); $r = q("select id, url from contact where url != '' and nurl = '' "); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { q("update contact set nurl = '%s' where id = %d", dbesc(normalise_link($rr['url'])), @@ -1168,7 +1168,7 @@ function update_1136() { // order in reverse so that we save the newest entry $r = q("select * from config where 1 order by id desc"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $found = false; foreach($arr as $x) { @@ -1187,7 +1187,7 @@ function update_1136() { $arr = array(); $r = q("select * from pconfig where 1 order by id desc"); - if(count($r)) { + if(dba::is_result($r)) { foreach($r as $rr) { $found = false; foreach($arr as $x) { @@ -1692,7 +1692,7 @@ function update_1190() { ); // convert old forumlist addon entries in new config entries - if (count($r)) { + if (dba::is_result($r)) { foreach ($r as $rr) { $uid = $rr['uid']; $family = $rr['cat']; diff --git a/view/theme/diabook/theme.php b/view/theme/diabook/theme.php index 972ad2e7cb..a13923ce4f 100644 --- a/view/theme/diabook/theme.php +++ b/view/theme/diabook/theme.php @@ -393,7 +393,7 @@ if ($color=="dark") $color_path = "/diabook-dark/"; $r = q("select gcontact.* from gcontact left join glink on glink.gcid = gcontact.id where glink.cid = 0 and glink.uid = 0 order by rand() limit 9"); $tpl = get_markup_template('ch_directory_item.tpl'); - if(count($r)) { + if(dba::is_result($r)) { $photo = 'photo'; foreach($r as $rr) { $profile_link = $a->get_baseurl() . '/profile/' . ((strlen($rr['nickname'])) ? $rr['nickname'] : $rr['profile_uid']); @@ -422,7 +422,7 @@ if ($color=="dark") $color_path = "/diabook-dark/"; 9 ); $tpl = get_markup_template('ch_directory_item.tpl'); - if(count($r)) { + if(dba::is_result($r)) { $photo = 'thumb'; foreach($r as $rr) { $profile_link = $a->get_baseurl() . '/profile/' . ((strlen($rr['nickname'])) ? $rr['nickname'] : $rr['profile_uid']); @@ -499,7 +499,7 @@ if ($color=="dark") $color_path = "/diabook-dark/"; dbesc(t('Contact Photos')), dbesc(t('Profile Photos')) ); - if(count($r)) { + if(dba::is_result($r)) { $tpl = get_markup_template('ch_directory_item.tpl'); foreach($r as $rr) { $photo_page = $a->get_baseurl() . '/photos/' . $rr['nickname'] . '/image/' . $rr['resource-id']; diff --git a/view/theme/vier/theme.php b/view/theme/vier/theme.php index 925ac76a1f..2d00f231cd 100644 --- a/view/theme/vier/theme.php +++ b/view/theme/vier/theme.php @@ -147,7 +147,7 @@ function vier_community_info() { $r = suggestion_query(local_user(), 0, 9); $tpl = get_markup_template('ch_directory_item.tpl'); - if(count($r)) { + if(dba::is_result($r)) { $aside['$comunity_profiles_title'] = t('Community Profiles'); $aside['$comunity_profiles_items'] = array(); @@ -170,13 +170,14 @@ function vier_community_info() { $publish = (get_config('system','publish_all') ? '' : " AND `publish` = 1 "); $order = " ORDER BY `register_date` DESC "; + $tpl = get_markup_template('ch_directory_item.tpl'); + $r = q("SELECT `profile`.*, `profile`.`uid` AS `profile_uid`, `user`.`nickname` FROM `profile` LEFT JOIN `user` ON `user`.`uid` = `profile`.`uid` WHERE `is-default` = 1 $publish AND `user`.`blocked` = 0 $order LIMIT %d , %d ", 0, 9); - $tpl = get_markup_template('ch_directory_item.tpl'); - if(count($r)) { + if(dba::is_result($r)) { $aside['$lastusers_title'] = t('Last users'); $aside['$lastusers_items'] = array(); @@ -367,7 +368,7 @@ function vier_community_info() { $tpl = get_markup_template('ch_connectors.tpl'); - if(count($r)) { + if(dba::is_result($r)) { $con_services = array(); $con_services['title'] = Array("", t('Connect Services'), "", "");