refactor: Move me and add fetchMe() (#9029)

This commit is contained in:
Jiralite
2023-01-10 09:49:43 +00:00
committed by GitHub
parent f0d42644df
commit b01c81dd72
14 changed files with 78 additions and 32 deletions

View File

@@ -156,7 +156,7 @@ class GuildEmojiManager extends BaseGuildEmojiManager {
throw new Error('EMOJI_MANAGED'); throw new Error('EMOJI_MANAGED');
} }
const { me } = this.guild; const { me } = this.guild.members;
if (!me) throw new Error('GUILD_UNCACHED_ME'); if (!me) throw new Error('GUILD_UNCACHED_ME');
if (!me.permissions.has(Permissions.FLAGS.MANAGE_EMOJIS_AND_STICKERS)) { if (!me.permissions.has(Permissions.FLAGS.MANAGE_EMOJIS_AND_STICKERS)) {
throw new Error('MISSING_MANAGE_EMOJIS_AND_STICKERS_PERMISSION', this.guild); throw new Error('MISSING_MANAGE_EMOJIS_AND_STICKERS_PERMISSION', this.guild);

View File

@@ -9,6 +9,7 @@ const BaseGuildVoiceChannel = require('../structures/BaseGuildVoiceChannel');
const { GuildMember } = require('../structures/GuildMember'); const { GuildMember } = require('../structures/GuildMember');
const { Role } = require('../structures/Role'); const { Role } = require('../structures/Role');
const { Events, Opcodes } = require('../util/Constants'); const { Events, Opcodes } = require('../util/Constants');
const { PartialTypes } = require('../util/Constants');
const SnowflakeUtil = require('../util/SnowflakeUtil'); const SnowflakeUtil = require('../util/SnowflakeUtil');
/** /**
@@ -118,6 +119,20 @@ class GuildMemberManager extends CachedManager {
return data instanceof Buffer ? (options.fetchWhenExisting === false ? null : this.fetch(userId)) : this._add(data); return data instanceof Buffer ? (options.fetchWhenExisting === false ? null : this.fetch(userId)) : this._add(data);
} }
/**
* The client user as a GuildMember of this guild
* @type {?GuildMember}
* @readonly
*/
get me() {
return (
this.resolve(this.client.user.id) ??
(this.client.options.partials.includes(PartialTypes.GUILD_MEMBER)
? this._add({ user: { id: this.client.user.id } }, true)
: null)
);
}
/** /**
* Options used to fetch a single member from a guild. * Options used to fetch a single member from a guild.
* @typedef {BaseFetchOptions} FetchMemberOptions * @typedef {BaseFetchOptions} FetchMemberOptions
@@ -189,6 +204,15 @@ class GuildMemberManager extends CachedManager {
return this._fetchMany(options); return this._fetchMany(options);
} }
/**
* Fetches the client user as a GuildMember of the guild.
* @param {BaseFetchOptions} [options] The options for fetching the member
* @returns {Promise<GuildMember>}
*/
fetchMe(options) {
return this.fetch({ ...options, user: this.client.user.id });
}
/** /**
* Options used for searching guild members. * Options used for searching guild members.
* @typedef {Object} GuildSearchMembersOptions * @typedef {Object} GuildSearchMembersOptions

View File

@@ -36,6 +36,24 @@ class ThreadMemberManager extends CachedManager {
return member; return member;
} }
/**
* Fetches the client user as a ThreadMember of the thread.
* @param {BaseFetchOptions} [options] The options for fetching the member
* @returns {Promise<ThreadMember>}
*/
fetchMe(options) {
return this.fetch(this.client.user.id, options);
}
/**
* The client user as a ThreadMember of this ThreadChannel
* @type {?ThreadMember}
* @readonly
*/
get me() {
return this.resolve(this.client.user.id);
}
/** /**
* Data that resolves to give a ThreadMember object. This can be: * Data that resolves to give a ThreadMember object. This can be:
* * A ThreadMember object * * A ThreadMember object

View File

@@ -75,7 +75,7 @@ class BaseGuildVoiceChannel extends GuildChannel {
if (permissions.has(Permissions.FLAGS.ADMINISTRATOR, false)) return true; if (permissions.has(Permissions.FLAGS.ADMINISTRATOR, false)) return true;
return ( return (
this.guild.me.communicationDisabledUntilTimestamp < Date.now() && this.guild.members.me.communicationDisabledUntilTimestamp < Date.now() &&
permissions.has(Permissions.FLAGS.CONNECT, false) permissions.has(Permissions.FLAGS.CONNECT, false)
); );
} }

View File

@@ -25,7 +25,6 @@ const VoiceStateManager = require('../managers/VoiceStateManager');
const { const {
ChannelTypes, ChannelTypes,
DefaultMessageNotificationLevels, DefaultMessageNotificationLevels,
PartialTypes,
VerificationLevels, VerificationLevels,
ExplicitContentFilterLevels, ExplicitContentFilterLevels,
Status, Status,
@@ -39,6 +38,7 @@ const Util = require('../util/Util');
let deprecationEmittedForSetChannelPositions = false; let deprecationEmittedForSetChannelPositions = false;
let deprecationEmittedForSetRolePositions = false; let deprecationEmittedForSetRolePositions = false;
let deprecationEmittedForDeleted = false; let deprecationEmittedForDeleted = false;
let deprecationEmittedForMe = false;
/** /**
* @type {WeakSet<Guild>} * @type {WeakSet<Guild>}
@@ -590,15 +590,16 @@ class Guild extends AnonymousGuild {
/** /**
* The client user as a GuildMember of this guild * The client user as a GuildMember of this guild
* @type {?GuildMember} * @type {?GuildMember}
* @deprecated Use {@link GuildMemberManager#me} instead.
* @readonly * @readonly
*/ */
get me() { get me() {
return ( if (!deprecationEmittedForMe) {
this.members.resolve(this.client.user.id) ?? process.emitWarning('Guild#me is deprecated. Use Guild#members#me instead.', 'DeprecationWarning');
(this.client.options.partials.includes(PartialTypes.GUILD_MEMBER) deprecationEmittedForMe = true;
? this.members._add({ user: { id: this.client.user.id } }, true) }
: null)
); return this.members.me;
} }
/** /**

View File

@@ -415,7 +415,7 @@ class GuildChannel extends Channel {
// This flag allows managing even if timed out // This flag allows managing even if timed out
if (permissions.has(Permissions.FLAGS.ADMINISTRATOR, false)) return true; if (permissions.has(Permissions.FLAGS.ADMINISTRATOR, false)) return true;
if (this.guild.me.communicationDisabledUntilTimestamp > Date.now()) return false; if (this.guild.members.me.communicationDisabledUntilTimestamp > Date.now()) return false;
const bitfield = VoiceBasedChannelTypes.includes(this.type) const bitfield = VoiceBasedChannelTypes.includes(this.type)
? Permissions.FLAGS.MANAGE_CHANNELS | Permissions.FLAGS.CONNECT ? Permissions.FLAGS.MANAGE_CHANNELS | Permissions.FLAGS.CONNECT

View File

@@ -55,8 +55,8 @@ class GuildEmoji extends BaseGuildEmoji {
* @readonly * @readonly
*/ */
get deletable() { get deletable() {
if (!this.guild.me) throw new Error('GUILD_UNCACHED_ME'); if (!this.guild.members.me) throw new Error('GUILD_UNCACHED_ME');
return !this.managed && this.guild.me.permissions.has(Permissions.FLAGS.MANAGE_EMOJIS_AND_STICKERS); return !this.managed && this.guild.members.me.permissions.has(Permissions.FLAGS.MANAGE_EMOJIS_AND_STICKERS);
} }
/** /**

View File

@@ -272,8 +272,8 @@ class GuildMember extends Base {
if (this.user.id === this.guild.ownerId) return false; if (this.user.id === this.guild.ownerId) return false;
if (this.user.id === this.client.user.id) return false; if (this.user.id === this.client.user.id) return false;
if (this.client.user.id === this.guild.ownerId) return true; if (this.client.user.id === this.guild.ownerId) return true;
if (!this.guild.me) throw new Error('GUILD_UNCACHED_ME'); if (!this.guild.members.me) throw new Error('GUILD_UNCACHED_ME');
return this.guild.me.roles.highest.comparePositionTo(this.roles.highest) > 0; return this.guild.members.me.roles.highest.comparePositionTo(this.roles.highest) > 0;
} }
/** /**
@@ -282,7 +282,7 @@ class GuildMember extends Base {
* @readonly * @readonly
*/ */
get kickable() { get kickable() {
return this.manageable && this.guild.me.permissions.has(Permissions.FLAGS.KICK_MEMBERS); return this.manageable && this.guild.members.me.permissions.has(Permissions.FLAGS.KICK_MEMBERS);
} }
/** /**
@@ -291,7 +291,7 @@ class GuildMember extends Base {
* @readonly * @readonly
*/ */
get bannable() { get bannable() {
return this.manageable && this.guild.me.permissions.has(Permissions.FLAGS.BAN_MEMBERS); return this.manageable && this.guild.members.me.permissions.has(Permissions.FLAGS.BAN_MEMBERS);
} }
/** /**
@@ -303,7 +303,7 @@ class GuildMember extends Base {
return ( return (
!this.permissions.has(Permissions.FLAGS.ADMINISTRATOR) && !this.permissions.has(Permissions.FLAGS.ADMINISTRATOR) &&
this.manageable && this.manageable &&
(this.guild.me?.permissions.has(Permissions.FLAGS.MODERATE_MEMBERS) ?? false) (this.guild.members.me?.permissions.has(Permissions.FLAGS.MODERATE_MEMBERS) ?? false)
); );
} }

View File

@@ -241,10 +241,10 @@ class Invite extends Base {
get deletable() { get deletable() {
const guild = this.guild; const guild = this.guild;
if (!guild || !this.client.guilds.cache.has(guild.id)) return false; if (!guild || !this.client.guilds.cache.has(guild.id)) return false;
if (!guild.me) throw new Error('GUILD_UNCACHED_ME'); if (!guild.members.me) throw new Error('GUILD_UNCACHED_ME');
return ( return (
this.channel.permissionsFor(this.client.user).has(Permissions.FLAGS.MANAGE_CHANNELS, false) || this.channel.permissionsFor(this.client.user).has(Permissions.FLAGS.MANAGE_CHANNELS, false) ||
guild.me.permissions.has(Permissions.FLAGS.MANAGE_GUILD) guild.members.me.permissions.has(Permissions.FLAGS.MANAGE_GUILD)
); );
} }

View File

@@ -632,7 +632,7 @@ class Message extends Base {
return Boolean( return Boolean(
this.author.id === this.client.user.id || this.author.id === this.client.user.id ||
(permissions.has(Permissions.FLAGS.MANAGE_MESSAGES, false) && (permissions.has(Permissions.FLAGS.MANAGE_MESSAGES, false) &&
this.guild.me.communicationDisabledUntilTimestamp < Date.now()), this.guild.members.me.communicationDisabledUntilTimestamp < Date.now()),
); );
} }

View File

@@ -521,7 +521,7 @@ class ThreadChannel extends Channel {
if (permissions.has(Permissions.FLAGS.ADMINISTRATOR, false)) return true; if (permissions.has(Permissions.FLAGS.ADMINISTRATOR, false)) return true;
return ( return (
this.guild.me.communicationDisabledUntilTimestamp < Date.now() && this.guild.members.me.communicationDisabledUntilTimestamp < Date.now() &&
permissions.has(Permissions.FLAGS.MANAGE_THREADS, false) permissions.has(Permissions.FLAGS.MANAGE_THREADS, false)
); );
} }
@@ -553,7 +553,7 @@ class ThreadChannel extends Channel {
!(this.archived && this.locked && !this.manageable) && !(this.archived && this.locked && !this.manageable) &&
(this.type !== 'GUILD_PRIVATE_THREAD' || this.joined || this.manageable) && (this.type !== 'GUILD_PRIVATE_THREAD' || this.joined || this.manageable) &&
permissions.has(Permissions.FLAGS.SEND_MESSAGES_IN_THREADS, false) && permissions.has(Permissions.FLAGS.SEND_MESSAGES_IN_THREADS, false) &&
this.guild.me.communicationDisabledUntilTimestamp < Date.now() this.guild.members.me.communicationDisabledUntilTimestamp < Date.now()
); );
} }

View File

@@ -113,7 +113,8 @@ class VoiceChannel extends BaseGuildVoiceChannel {
if (permissions.has(Permissions.FLAGS.ADMINISTRATOR, false)) return true; if (permissions.has(Permissions.FLAGS.ADMINISTRATOR, false)) return true;
return ( return (
this.guild.me.communicationDisabledUntilTimestamp < Date.now() && permissions.has(Permissions.FLAGS.SPEAK, false) this.guild.members.me.communicationDisabledUntilTimestamp < Date.now() &&
permissions.has(Permissions.FLAGS.SPEAK, false)
); );
} }

View File

@@ -208,10 +208,10 @@ class VoiceState extends Base {
* @param {boolean} [request=true] Whether or not the client is requesting to become a speaker. * @param {boolean} [request=true] Whether or not the client is requesting to become a speaker.
* @example * @example
* // Making the client request to speak in a stage channel (raise its hand) * // Making the client request to speak in a stage channel (raise its hand)
* guild.me.voice.setRequestToSpeak(true); * guild.members.me.voice.setRequestToSpeak(true);
* @example * @example
* // Making the client cancel a request to speak * // Making the client cancel a request to speak
* guild.me.voice.setRequestToSpeak(false); * guild.members.me.voice.setRequestToSpeak(false);
* @returns {Promise<void>} * @returns {Promise<void>}
*/ */
async setRequestToSpeak(request = true) { async setRequestToSpeak(request = true) {
@@ -232,10 +232,10 @@ class VoiceState extends Base {
* @param {boolean} [suppressed=true] Whether or not the user should be suppressed. * @param {boolean} [suppressed=true] Whether or not the user should be suppressed.
* @example * @example
* // Making the client a speaker * // Making the client a speaker
* guild.me.voice.setSuppressed(false); * guild.members.me.voice.setSuppressed(false);
* @example * @example
* // Making the client an audience member * // Making the client an audience member
* guild.me.voice.setSuppressed(true); * guild.members.me.voice.setSuppressed(true);
* @example * @example
* // Inviting another user to speak * // Inviting another user to speak
* voiceState.setSuppressed(false); * voiceState.setSuppressed(false);

12
typings/index.d.ts vendored
View File

@@ -705,9 +705,7 @@ export interface CollectorEventTypes<K, V, F extends unknown[] = []> {
end: [collected: Collection<K, V>, reason: string]; end: [collected: Collection<K, V>, reason: string];
} }
export type ChannelFlagsString = export type ChannelFlagsString = 'PINNED' | 'REQUIRE_TAG';
| 'PINNED'
| 'REQUIRE_TAG';
export class ChannelFlags extends BitField<ChannelFlagsString> { export class ChannelFlags extends BitField<ChannelFlagsString> {
public static FLAGS: Record<ChannelFlagsString, number>; public static FLAGS: Record<ChannelFlagsString, number>;
public static resolve(bit?: BitFieldResolvable<ChannelFlagsString, number>): number; public static resolve(bit?: BitFieldResolvable<ChannelFlagsString, number>): number;
@@ -966,6 +964,7 @@ export class Guild extends AnonymousGuild {
public large: boolean; public large: boolean;
public maximumMembers: number | null; public maximumMembers: number | null;
public maximumPresences: number | null; public maximumPresences: number | null;
/** @deprecated Use {@link GuildMemberManager.me} instead. */
public readonly me: GuildMember | null; public readonly me: GuildMember | null;
public memberCount: number; public memberCount: number;
public members: GuildMemberManager; public members: GuildMemberManager;
@@ -3371,6 +3370,7 @@ export class GuildManager extends CachedManager<Snowflake, Guild, GuildResolvabl
export class GuildMemberManager extends CachedManager<Snowflake, GuildMember, GuildMemberResolvable> { export class GuildMemberManager extends CachedManager<Snowflake, GuildMember, GuildMemberResolvable> {
private constructor(guild: Guild, iterable?: Iterable<RawGuildMemberData>); private constructor(guild: Guild, iterable?: Iterable<RawGuildMemberData>);
public guild: Guild; public guild: Guild;
public get me(): GuildMember | null;
public add( public add(
user: UserResolvable, user: UserResolvable,
options: AddGuildMemberOptions & { fetchWhenExisting: false }, options: AddGuildMemberOptions & { fetchWhenExisting: false },
@@ -3382,6 +3382,7 @@ export class GuildMemberManager extends CachedManager<Snowflake, GuildMember, Gu
options: UserResolvable | FetchMemberOptions | (FetchMembersOptions & { user: UserResolvable }), options: UserResolvable | FetchMemberOptions | (FetchMembersOptions & { user: UserResolvable }),
): Promise<GuildMember>; ): Promise<GuildMember>;
public fetch(options?: FetchMembersOptions): Promise<Collection<Snowflake, GuildMember>>; public fetch(options?: FetchMembersOptions): Promise<Collection<Snowflake, GuildMember>>;
public fetchMe(options?: BaseFetchOptions): Promise<GuildMember>;
public kick(user: UserResolvable, reason?: string): Promise<GuildMember | User | Snowflake>; public kick(user: UserResolvable, reason?: string): Promise<GuildMember | User | Snowflake>;
public list(options?: GuildListMembersOptions): Promise<Collection<Snowflake, GuildMember>>; public list(options?: GuildListMembersOptions): Promise<Collection<Snowflake, GuildMember>>;
public prune(options: GuildPruneMembersOptions & { dry?: false; count: false }): Promise<null>; public prune(options: GuildPruneMembersOptions & { dry?: false; count: false }): Promise<null>;
@@ -3579,10 +3580,12 @@ export class GuildForumThreadManager extends ThreadManager {
export class ThreadMemberManager extends CachedManager<Snowflake, ThreadMember, ThreadMemberResolvable> { export class ThreadMemberManager extends CachedManager<Snowflake, ThreadMember, ThreadMemberResolvable> {
private constructor(thread: ThreadChannel, iterable?: Iterable<RawThreadMemberData>); private constructor(thread: ThreadChannel, iterable?: Iterable<RawThreadMemberData>);
public thread: ThreadChannel; public thread: ThreadChannel;
public get me(): ThreadMember | null;
public add(member: UserResolvable | '@me', reason?: string): Promise<Snowflake>; public add(member: UserResolvable | '@me', reason?: string): Promise<Snowflake>;
public fetch(member?: UserResolvable, options?: BaseFetchOptions): Promise<ThreadMember>; public fetch(member?: UserResolvable, options?: BaseFetchOptions): Promise<ThreadMember>;
/** @deprecated Use `fetch(member, options)` instead. */ /** @deprecated Use `fetch(member, options)` instead. */
public fetch(cache?: boolean): Promise<Collection<Snowflake, ThreadMember>>; public fetch(cache?: boolean): Promise<Collection<Snowflake, ThreadMember>>;
public fetchMe(options?: BaseFetchOptions): Promise<ThreadMember>;
public remove(id: Snowflake | '@me', reason?: string): Promise<Snowflake>; public remove(id: Snowflake | '@me', reason?: string): Promise<Snowflake>;
} }
@@ -5476,8 +5479,7 @@ export type MessageComponentType = keyof typeof MessageComponentTypes;
export type MessageComponentTypeResolvable = MessageComponentType | MessageComponentTypes; export type MessageComponentTypeResolvable = MessageComponentType | MessageComponentTypes;
export type GuildForumThreadMessageCreateOptions = MessageOptions & export type GuildForumThreadMessageCreateOptions = MessageOptions & Pick<MessageOptions, 'flags' | 'stickers'>;
Pick<MessageOptions, 'flags' | 'stickers'>;
export interface MessageEditOptions { export interface MessageEditOptions {
attachments?: MessageAttachment[]; attachments?: MessageAttachment[];