X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=classes%2FUser_group.php;h=3dc5fd4b2d8e0e1e799fad943b59301621a05ee3;hb=2f86cd860231f92213950d439e9a109b7a5f11c4;hp=1ec2b09ae86ce92bbf615a2ccf0adbd11dfdd768;hpb=f433f7ce774d977466891a9e432dc722a2dde536;p=quix0rs-gnu-social.git diff --git a/classes/User_group.php b/classes/User_group.php index 1ec2b09ae8..3dc5fd4b2d 100644 --- a/classes/User_group.php +++ b/classes/User_group.php @@ -15,31 +15,21 @@ 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) - { - return parent::multiGet('User_group', $keyCol, $keyVals, $skipNulls); - } - /* the code above is auto generated do not remove the tag below */ ###END_AUTOCODE @@ -48,37 +38,67 @@ class User_group extends Managed_DataObject 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', @@ -93,7 +113,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)); } @@ -108,7 +128,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)); } @@ -121,8 +141,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; @@ -135,13 +157,6 @@ class User_group extends Managed_DataObject return $stream->getNotices($offset, $limit, $since_id, $max_id); } - - function allowedNickname($nickname) - { - static $blacklist = array('new'); - return !in_array($nickname, $blacklist); - } - function getMembers($offset=0, $limit=null) { $ids = null; if (is_null($limit) || $offset + $limit > User_group::CACHE_WINDOW) { @@ -220,7 +235,16 @@ class User_group extends Managed_DataObject return $members; } - function getMemberCount() + public function getAdminCount() + { + $block = new Group_member(); + $block->group_id = $this->id; + $block->is_admin = 1; + + return $block->count(); + } + + public function getMemberCount() { $key = sprintf("group:member_count:%d", $this->id); @@ -262,50 +286,27 @@ 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; } @@ -392,7 +393,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__); @@ -403,14 +407,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) { + if ($profile instanceof Profile) { $group = $profile->getGroups(0, null); - while ($group->fetch()) { + while ($group instanceof User_group && $group->fetch()) { if ($group->nickname == $nickname) { // @fixme is this the best way? return clone($group); @@ -419,14 +423,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; } @@ -531,19 +534,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. @@ -570,7 +560,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); @@ -579,6 +569,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 @@ -598,8 +590,6 @@ class User_group extends Managed_DataObject $group = new User_group(); - $group->query('BEGIN'); - if (empty($uri)) { // fill in later... $uri = null; @@ -608,14 +598,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']); @@ -633,7 +642,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.')); @@ -690,11 +699,11 @@ class User_group extends Managed_DataObject } } - $group->query('COMMIT'); - Event::handle('EndGroupSave', array($group)); } + $profile->query('COMMIT'); + return $group; } @@ -706,53 +715,96 @@ 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; + } + if ($profile->update($origpro) === false) { + throw new ServerException(_('Unable to update profile')); } - parent::delete(); + + return parent::update($dataObject); } function isPrivate() @@ -760,4 +812,43 @@ class User_group extends Managed_DataObject return ($this->join_policy == self::JOIN_POLICY_MODERATE && $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); + } }