X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=classes%2FUser_group.php;h=344394a682052ef9d8d3203f1e41491002881837;hb=42c653a9d0a57d8b575614827a569badd054e5a7;hp=0f1cc40b28f38e42a50b4e93323144962a40a021;hpb=8c710ad2c1b80544acccb515f7b601aadff2de16;p=quix0rs-gnu-social.git diff --git a/classes/User_group.php b/classes/User_group.php index 0f1cc40b28..344394a682 100644 --- a/classes/User_group.php +++ b/classes/User_group.php @@ -7,6 +7,7 @@ class User_group extends Managed_DataObject { const JOIN_POLICY_OPEN = 0; const JOIN_POLICY_MODERATE = 1; + const CACHE_WINDOW = 201; ###START_AUTOCODE /* the code below is auto generated do not remove the above tag */ @@ -14,70 +15,96 @@ class User_group extends Managed_DataObject public $__table = 'user_group'; // table name public $id; // int(4) primary_key not_null public $nickname; // varchar(64) - public $fullname; // varchar(255) - public $homepage; // varchar(255) + public $fullname; // varchar(191) not 255 because utf8mb4 takes more space + public $homepage; // varchar(191) not 255 because utf8mb4 takes more space public $description; // text - public $location; // varchar(255) - public $original_logo; // varchar(255) - public $homepage_logo; // varchar(255) - public $stream_logo; // varchar(255) - public $mini_logo; // varchar(255) + public $location; // varchar(191) not 255 because utf8mb4 takes more space + public $original_logo; // varchar(191) not 255 because utf8mb4 takes more space + public $homepage_logo; // varchar(191) not 255 because utf8mb4 takes more space + public $stream_logo; // varchar(191) not 255 because utf8mb4 takes more space + public $mini_logo; // varchar(191) not 255 because utf8mb4 takes more space public $created; // datetime not_null default_0000-00-00%2000%3A00%3A00 public $modified; // timestamp not_null default_CURRENT_TIMESTAMP - public $uri; // varchar(255) unique_key - public $mainpage; // varchar(255) + public $uri; // varchar(191) unique_key not 255 because utf8mb4 takes more space + public $mainpage; // varchar(191) not 255 because utf8mb4 takes more space public $join_policy; // tinyint public $force_scope; // tinyint - /* Static get */ - function staticGet($k,$v=NULL) { - return Memcached_DataObject::staticGet('User_group',$k,$v); - } - - function multiGet($keyCol, $keyVals, $skipNulls=true) + /* the code above is auto generated do not remove the tag below */ + ###END_AUTOCODE + + public function getObjectType() { - return parent::multiGet('User_group', $keyCol, $keyVals, $skipNulls); + return ActivityObject::GROUP; } - /* the code above is auto generated do not remove the tag below */ - ###END_AUTOCODE public static function schemaDef() { return array( 'fields' => array( 'id' => array('type' => 'serial', 'not null' => true, 'description' => 'unique identifier'), + 'profile_id' => array('type' => 'int', 'not null' => true, 'description' => 'foreign key to profile table'), 'nickname' => array('type' => 'varchar', 'length' => 64, 'description' => 'nickname for addressing'), - 'fullname' => array('type' => 'varchar', 'length' => 255, 'description' => 'display name'), - 'homepage' => array('type' => 'varchar', 'length' => 255, 'description' => 'URL, cached so we dont regenerate'), + 'fullname' => array('type' => 'varchar', 'length' => 191, 'description' => 'display name'), + 'homepage' => array('type' => 'varchar', 'length' => 191, 'description' => 'URL, cached so we dont regenerate'), 'description' => array('type' => 'text', 'description' => 'group description'), - 'location' => array('type' => 'varchar', 'length' => 255, 'description' => 'related physical location, if any'), + 'location' => array('type' => 'varchar', 'length' => 191, 'description' => 'related physical location, if any'), - 'original_logo' => array('type' => 'varchar', 'length' => 255, 'description' => 'original size logo'), - 'homepage_logo' => array('type' => 'varchar', 'length' => 255, 'description' => 'homepage (profile) size logo'), - 'stream_logo' => array('type' => 'varchar', 'length' => 255, 'description' => 'stream-sized logo'), - 'mini_logo' => array('type' => 'varchar', 'length' => 255, 'description' => 'mini logo'), + 'original_logo' => array('type' => 'varchar', 'length' => 191, 'description' => 'original size logo'), + 'homepage_logo' => array('type' => 'varchar', 'length' => 191, 'description' => 'homepage (profile) size logo'), + 'stream_logo' => array('type' => 'varchar', 'length' => 191, 'description' => 'stream-sized logo'), + 'mini_logo' => array('type' => 'varchar', 'length' => 191, 'description' => 'mini logo'), 'created' => array('type' => 'datetime', 'not null' => true, 'description' => 'date this record was created'), 'modified' => array('type' => 'timestamp', 'not null' => true, 'description' => 'date this record was modified'), - 'uri' => array('type' => 'varchar', 'length' => 255, 'description' => 'universal identifier'), - 'mainpage' => array('type' => 'varchar', 'length' => 255, 'description' => 'page for group info to link to'), + 'uri' => array('type' => 'varchar', 'length' => 191, 'description' => 'universal identifier'), + 'mainpage' => array('type' => 'varchar', 'length' => 191, 'description' => 'page for group info to link to'), 'join_policy' => array('type' => 'int', 'size' => 'tiny', 'description' => '0=open; 1=requires admin approval'), 'force_scope' => array('type' => 'int', 'size' => 'tiny', 'description' => '0=never,1=sometimes,-1=always'), ), 'primary key' => array('id'), 'unique keys' => array( 'user_group_uri_key' => array('uri'), +// when it's safe and everyone's run upgrade.php 'user_profile_id_key' => array('profile_id'), + ), + 'foreign keys' => array( + 'user_group_id_fkey' => array('profile', array('profile_id' => 'id')), ), 'indexes' => array( 'user_group_nickname_idx' => array('nickname'), + 'user_group_profile_id_idx' => array('profile_id'), //make this unique in future ), ); } - function defaultLogo($size) + protected $_profile = array(); + + /** + * @return Profile + * + * @throws GroupNoProfileException if user has no profile + */ + public function getProfile() + { + if (!isset($this->_profile[$this->profile_id])) { + $profile = Profile::getKV('id', $this->profile_id); + if (!$profile instanceof Profile) { + throw new GroupNoProfileException($this); + } + $this->_profile[$this->profile_id] = $profile; + } + return $this->_profile[$this->profile_id]; + } + + public function getNickname() + { + return $this->getProfile()->getNickname(); + } + + public static function defaultLogo($size) { static $sizenames = array(AVATAR_PROFILE_SIZE => 'profile', AVATAR_STREAM_SIZE => 'stream', @@ -92,7 +119,7 @@ class User_group extends Managed_DataObject // normally stored in mainpage, but older ones may be null if (!empty($this->mainpage)) { $url = $this->mainpage; - } else { + } elseif ($this->isLocal()) { $url = common_local_url('showgroup', array('nickname' => $this->nickname)); } @@ -107,7 +134,7 @@ class User_group extends Managed_DataObject if (Event::handle('StartUserGroupGetUri', array($this, &$uri))) { if (!empty($this->uri)) { $uri = $this->uri; - } else { + } elseif ($this->isLocal()) { $uri = common_local_url('groupbyid', array('id' => $this->id)); } @@ -120,8 +147,10 @@ class User_group extends Managed_DataObject { $url = null; if (Event::handle('StartUserGroupPermalink', array($this, &$url))) { - $url = common_local_url('groupbyid', - array('id' => $this->id)); + if ($this->isLocal()) { + $url = common_local_url('groupbyid', + array('id' => $this->id)); + } } Event::handle('EndUserGroupPermalink', array($this, &$url)); return $url; @@ -134,34 +163,52 @@ class User_group extends Managed_DataObject return $stream->getNotices($offset, $limit, $since_id, $max_id); } + function getMembers($offset=0, $limit=null) { + $ids = null; + if (is_null($limit) || $offset + $limit > User_group::CACHE_WINDOW) { + $ids = $this->getMemberIDs($offset, + $limit); + } else { + $key = sprintf('group:member_ids:%d', $this->id); + $window = self::cacheGet($key); + if ($window === false) { + $window = $this->getMemberIDs(0, + User_group::CACHE_WINDOW); + self::cacheSet($key, $window); + } - function allowedNickname($nickname) - { - static $blacklist = array('new'); - return !in_array($nickname, $blacklist); + $ids = array_slice($window, + $offset, + $limit); + } + + return Profile::multiGet('id', $ids); } - function getMembers($offset=0, $limit=null) + function getMemberIDs($offset=0, $limit=null) { - $qry = - 'SELECT profile.* ' . - 'FROM profile JOIN group_member '. - 'ON profile.id = group_member.profile_id ' . - 'WHERE group_member.group_id = %d ' . - 'ORDER BY group_member.created DESC '; + $gm = new Group_member(); - if ($limit != null) { - if (common_config('db','type') == 'pgsql') { - $qry .= ' LIMIT ' . $limit . ' OFFSET ' . $offset; - } else { - $qry .= ' LIMIT ' . $offset . ', ' . $limit; - } + $gm->selectAdd(); + $gm->selectAdd('profile_id'); + + $gm->group_id = $this->id; + + $gm->orderBy('created DESC'); + + if (!is_null($limit)) { + $gm->limit($offset, $limit); } - $members = new Profile(); + $ids = array(); - $members->query(sprintf($qry, $this->id)); - return $members; + if ($gm->find()) { + while ($gm->fetch()) { + $ids[] = $gm->profile_id; + } + } + + return $ids; } /** @@ -194,19 +241,35 @@ class User_group extends Managed_DataObject return $members; } - function getMemberCount() + public function getAdminCount() { - // XXX: WORM cache this + $block = new Group_member(); + $block->group_id = $this->id; + $block->is_admin = 1; + + return $block->count(); + } - $members = $this->getMembers(); - $member_count = 0; + public function getMemberCount() + { + $key = sprintf("group:member_count:%d", $this->id); - /** $member->count() doesn't work. */ - while ($members->fetch()) { - $member_count++; + $cnt = self::cacheGet($key); + + if (is_integer($cnt)) { + return (int) $cnt; } - return $member_count; + $mem = new Group_member(); + $mem->group_id = $this->id; + + // XXX: why 'distinct'? + + $cnt = (int) $mem->count('distinct profile_id'); + + self::cacheSet($key, $cnt); + + return $cnt; } function getBlockedCount() @@ -229,62 +292,47 @@ class User_group extends Managed_DataObject return $queue->count(); } - function getAdmins($offset=0, $limit=null) + function getAdmins($offset=null, $limit=null) // offset is null because DataObject wants it, 0 would mean no results { - $qry = - 'SELECT profile.* ' . - 'FROM profile JOIN group_member '. - 'ON profile.id = group_member.profile_id ' . - 'WHERE group_member.group_id = %d ' . - 'AND group_member.is_admin = 1 ' . - 'ORDER BY group_member.modified ASC '; - - if ($limit != null) { - if (common_config('db','type') == 'pgsql') { - $qry .= ' LIMIT ' . $limit . ' OFFSET ' . $offset; - } else { - $qry .= ' LIMIT ' . $offset . ', ' . $limit; - } - } - $admins = new Profile(); + $admins->joinAdd(array('id', 'group_member:profile_id')); + $admins->whereAdd(sprintf('group_member.group_id = %u AND group_member.is_admin = 1', $this->id)); + $admins->orderBy('group_member.modified ASC'); + $admins->limit($offset, $limit); + $admins->find(); - $admins->query(sprintf($qry, $this->id)); return $admins; } - function getBlocked($offset=0, $limit=null) + function getBlocked($offset=null, $limit=null) // offset is null because DataObject wants it, 0 would mean no results { - $qry = - 'SELECT profile.* ' . - 'FROM profile JOIN group_block '. - 'ON profile.id = group_block.blocked ' . - 'WHERE group_block.group_id = %d ' . - 'ORDER BY group_block.modified DESC '; - - if ($limit != null) { - if (common_config('db','type') == 'pgsql') { - $qry .= ' LIMIT ' . $limit . ' OFFSET ' . $offset; - } else { - $qry .= ' LIMIT ' . $offset . ', ' . $limit; - } - } - $blocked = new Profile(); + $blocked->joinAdd(array('id', 'group_block:blocked')); + $blocked->whereAdd(sprintf('group_block.group_id = %u', $this->id)); + $blocked->orderBy('group_block.modified DESC'); + $blocked->limit($offset, $limit); + $blocked->find(); - $blocked->query(sprintf($qry, $this->id)); return $blocked; } function setOriginal($filename) { - $imagefile = new ImageFile($this->id, Avatar::path($filename)); + // This should be handled by the Profile->setOriginal function so user and group avatars are handled the same + $imagefile = new ImageFile(null, Avatar::path($filename)); + + $sizes = array('homepage_logo' => AVATAR_PROFILE_SIZE, + 'stream_logo' => AVATAR_STREAM_SIZE, + 'mini_logo' => AVATAR_MINI_SIZE); $orig = clone($this); $this->original_logo = Avatar::url($filename); - $this->homepage_logo = Avatar::url($imagefile->resize(AVATAR_PROFILE_SIZE)); - $this->stream_logo = Avatar::url($imagefile->resize(AVATAR_STREAM_SIZE)); - $this->mini_logo = Avatar::url($imagefile->resize(AVATAR_MINI_SIZE)); + foreach ($sizes as $name=>$size) { + $filename = Avatar::filename($this->profile_id, image_type_to_extension($imagefile->preferredType()), + $size, common_timestamp()); + $imagefile->resizeTo(Avatar::path($filename), array('width'=>$size, 'height'=>$size)); + $this->$name = Avatar::url($filename); + } common_debug(common_log_objstring($this)); return $this->update($orig); } @@ -359,7 +407,10 @@ class User_group extends Managed_DataObject } foreach ($to_insert as $insalias) { - $alias->alias = $insalias; + if ($insalias === $this->nickname) { + continue; + } + $alias->alias = Nickname::normalize($insalias, true); $result = $alias->insert(); if (!$result) { common_log_db_error($alias, 'INSERT', __FILE__); @@ -370,14 +421,14 @@ class User_group extends Managed_DataObject return true; } - static function getForNickname($nickname, $profile=null) + static function getForNickname($nickname, Profile $profile=null) { - $nickname = common_canonical_nickname($nickname); + $nickname = Nickname::normalize($nickname); // Are there any matching remote groups this profile's in? - if ($profile) { - $group = $profile->getGroups(); - while ($group->fetch()) { + if ($profile instanceof Profile) { + $group = $profile->getGroups(0, null); + while ($group instanceof User_group && $group->fetch()) { if ($group->nickname == $nickname) { // @fixme is this the best way? return clone($group); @@ -386,14 +437,13 @@ class User_group extends Managed_DataObject } // If not, check local groups. - - $group = Local_group::staticGet('nickname', $nickname); - if (!empty($group)) { - return User_group::staticGet('id', $group->group_id); + $group = Local_group::getKV('nickname', $nickname); + if ($group instanceof Local_group) { + return User_group::getKV('id', $group->group_id); } - $alias = Group_alias::staticGet('alias', $nickname); - if (!empty($alias)) { - return User_group::staticGet('id', $alias->group_id); + $alias = Group_alias::getKV('alias', $nickname); + if ($alias instanceof Group_alias) { + return User_group::getKV('id', $alias->group_id); } return null; } @@ -498,19 +548,6 @@ class User_group extends Managed_DataObject return $xs->getString(); } - /** - * Returns an XML string fragment with group information as an - * Activity Streams element. - * - * Assumes that 'activity' namespace has been previously defined. - * - * @return string - */ - function asActivitySubject() - { - return $this->asActivityNoun('subject'); - } - /** * Returns an XML string fragment with group information as an * Activity Streams noun object with the given element type. @@ -537,7 +574,7 @@ class User_group extends Managed_DataObject static function register($fields) { if (!empty($fields['userid'])) { - $profile = Profile::staticGet('id', $fields['userid']); + $profile = Profile::getKV('id', $fields['userid']); if ($profile && !$profile->hasRight(Right::CREATEGROUP)) { common_log(LOG_WARNING, "Attempted group creation from banned user: " . $profile->nickname); @@ -546,6 +583,8 @@ class User_group extends Managed_DataObject } } + $fields['nickname'] = Nickname::normalize($fields['nickname']); + // MAGICALLY put fields into current scope // @fixme kill extract(); it makes debugging absurdly hard @@ -565,8 +604,6 @@ class User_group extends Managed_DataObject $group = new User_group(); - $group->query('BEGIN'); - if (empty($uri)) { // fill in later... $uri = null; @@ -575,14 +612,33 @@ class User_group extends Managed_DataObject $mainpage = common_local_url('showgroup', array('nickname' => $nickname)); } - $group->nickname = $nickname; - $group->fullname = $fullname; - $group->homepage = $homepage; - $group->description = $description; - $group->location = $location; - $group->uri = $uri; - $group->mainpage = $mainpage; - $group->created = common_sql_now(); + // We must create a new, incrementally assigned profile_id + $profile = new Profile(); + $profile->nickname = $nickname; + $profile->fullname = $fullname; + $profile->profileurl = $mainpage; + $profile->homepage = $homepage; + $profile->bio = $description; + $profile->location = $location; + $profile->created = common_sql_now(); + + $group->nickname = $profile->nickname; + $group->fullname = $profile->fullname; + $group->homepage = $profile->homepage; + $group->description = $profile->bio; + $group->location = $profile->location; + $group->mainpage = $profile->profileurl; + $group->created = $profile->created; + + $profile->query('BEGIN'); + $id = $profile->insert(); + if ($id === false) { + $profile->query('ROLLBACK'); + throw new ServerException(_('Profile insertion failed')); + } + + $group->profile_id = $id; + $group->uri = $uri; if (isset($fields['join_policy'])) { $group->join_policy = intval($fields['join_policy']); @@ -600,7 +656,7 @@ class User_group extends Managed_DataObject $result = $group->insert(); - if (!$result) { + if ($result === false) { common_log_db_error($group, 'INSERT', __FILE__); // TRANS: Server exception thrown when creating a group failed. throw new ServerException(_('Could not create group.')); @@ -657,11 +713,11 @@ class User_group extends Managed_DataObject } } - $group->query('COMMIT'); - Event::handle('EndGroupSave', array($group)); } + $profile->query('COMMIT'); + return $group; } @@ -673,58 +729,140 @@ class User_group extends Managed_DataObject * are not de-cached in the UI, including the sidebar lists on * GroupsAction */ - function delete() + function delete($useWhere=false) { - if ($this->id) { + if (empty($this->id)) { + common_log(LOG_WARNING, "Ambiguous User_group->delete(); skipping related tables."); + return parent::delete($useWhere); + } - // Safe to delete in bulk for now + try { + $profile = $this->getProfile(); + $profile->delete(); + } catch (GroupNoProfileException $unp) { + common_log(LOG_INFO, "Group {$this->nickname} has no profile; continuing deletion."); + } - $related = array('Group_inbox', - 'Group_block', - 'Group_member', - 'Related_group'); + // Safe to delete in bulk for now - Event::handle('UserGroupDeleteRelated', array($this, &$related)); + $related = array('Group_inbox', + 'Group_block', + 'Group_member', + 'Related_group'); - foreach ($related as $cls) { + Event::handle('UserGroupDeleteRelated', array($this, &$related)); - $inst = new $cls(); - $inst->group_id = $this->id; + foreach ($related as $cls) { + $inst = new $cls(); + $inst->group_id = $this->id; - if ($inst->find()) { - while ($inst->fetch()) { - $dup = clone($inst); - $dup->delete(); - } + if ($inst->find()) { + while ($inst->fetch()) { + $dup = clone($inst); + $dup->delete(); } } + } - // And related groups in the other direction... - $inst = new Related_group(); - $inst->related_group_id = $this->id; - $inst->delete(); + // And related groups in the other direction... + $inst = new Related_group(); + $inst->related_group_id = $this->id; + $inst->delete(); - // Aliases and the local_group entry need to be cleared explicitly - // or we'll miss clearing some cache keys; that can make it hard - // to create a new group with one of those names or aliases. - $this->setAliases(array()); - $local = Local_group::staticGet('group_id', $this->id); - if ($local) { - $local->delete(); - } + // Aliases and the local_group entry need to be cleared explicitly + // or we'll miss clearing some cache keys; that can make it hard + // to create a new group with one of those names or aliases. + $this->setAliases(array()); - // blow the cached ids - self::blow('user_group:notice_ids:%d', $this->id); + // $this->isLocal() but we're using the resulting object + $local = Local_group::getKV('group_id', $this->id); + if ($local instanceof Local_group) { + $local->delete(); + } - } else { - common_log(LOG_WARN, "Ambiguous user_group->delete(); skipping related tables."); + // blow the cached ids + self::blow('user_group:notice_ids:%d', $this->id); + + return parent::delete($useWhere); + } + + public function update($dataObject=false) + { + // Whenever the User_group is updated, find the Local_group + // and update its nickname too. + if ($this->nickname != $dataObject->nickname) { + $local = Local_group::getKV('group_id', $this->id); + if ($local instanceof Local_group) { + common_debug("Updating Local_group ({$this->id}) nickname from {$dataObject->nickname} to {$this->nickname}"); + $local->setNickname($this->nickname); + } + } + + // Also make sure the Profile table is up to date! + $fields = array(/*group field => profile field*/ + 'nickname' => 'nickname', + 'fullname' => 'fullname', + 'mainpage' => 'profileurl', + 'homepage' => 'homepage', + 'description' => 'bio', + 'location' => 'location', + 'created' => 'created', + 'modified' => 'modified', + ); + $profile = $this->getProfile(); + $origpro = clone($profile); + foreach ($fields as $gf=>$pf) { + $profile->$pf = $this->$gf; } - parent::delete(); + if ($profile->update($origpro) === false) { + throw new ServerException(_('Unable to update profile')); + } + + return parent::update($dataObject); } function isPrivate() { return ($this->join_policy == self::JOIN_POLICY_MODERATE && - $this->force_scope == 1); + intval($this->force_scope) === 1); + } + + public function isLocal() + { + $local = Local_group::getKV('group_id', $this->id); + return ($local instanceof Local_group); + } + + static function groupsFromText($text, Profile $profile) + { + $groups = array(); + + /* extract all !group */ + $count = preg_match_all('/(?:^|\s)!(' . Nickname::DISPLAY_FMT . ')/', + strtolower($text), + $match); + + if (!$count) { + return $groups; + } + + foreach (array_unique($match[1]) as $nickname) { + $group = self::getForNickname($nickname, $profile); + if ($group instanceof User_group && $profile->isMember($group)) { + $groups[] = clone($group); + } + } + + return $groups; + } + + static function idsFromText($text, Profile $profile) + { + $ids = array(); + $groups = self::groupsFromText($text, $profile); + foreach ($groups as $group) { + $ids[$group->id] = true; + } + return array_keys($ids); } }