Skip to content

Commit

Permalink
refactor(GuildChannelManager): Remove redundant edit code (#10370)
Browse files Browse the repository at this point in the history
refactor(GuildChannelManager): remove redundant edit code

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
  • Loading branch information
Jiralite and kodiakhq[bot] authored Jul 10, 2024
1 parent 3654efe commit 9461045
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions packages/discord.js/src/managers/GuildChannelManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -284,13 +284,13 @@ class GuildChannelManager extends CachedManager {
* .catch(console.error);
*/
async edit(channel, options) {
channel = this.resolve(channel);
if (!channel) throw new DiscordjsTypeError(ErrorCodes.InvalidType, 'channel', 'GuildChannelResolvable');
const resolvedChannel = this.resolve(channel);
if (!resolvedChannel) throw new DiscordjsTypeError(ErrorCodes.InvalidType, 'channel', 'GuildChannelResolvable');

const parent = options.parent && this.client.channels.resolveId(options.parent);

if (options.position !== undefined) {
await this.setPosition(channel, options.position, { position: options.position, reason: options.reason });
await this.setPosition(resolvedChannel, options.position, { position: options.position, reason: options.reason });
}

let permission_overwrites = options.permissionOverwrites?.map(overwrite =>
Expand All @@ -305,22 +305,22 @@ class GuildChannelManager extends CachedManager {
PermissionOverwrites.resolve(overwrite, this.guild),
);
}
} else if (channel.parent) {
permission_overwrites = channel.parent.permissionOverwrites.cache.map(overwrite =>
} else if (resolvedChannel.parent) {
permission_overwrites = resolvedChannel.parent.permissionOverwrites.cache.map(overwrite =>
PermissionOverwrites.resolve(overwrite, this.guild),
);
}
}

const newData = await this.client.rest.patch(Routes.channel(channel.id), {
const newData = await this.client.rest.patch(Routes.channel(resolvedChannel.id), {
body: {
name: (options.name ?? channel.name).trim(),
name: options.name,
type: options.type,
topic: options.topic,
nsfw: options.nsfw,
bitrate: options.bitrate ?? channel.bitrate,
user_limit: options.userLimit ?? channel.userLimit,
rtc_region: 'rtcRegion' in options ? options.rtcRegion : channel.rtcRegion,
bitrate: options.bitrate,
user_limit: options.userLimit,
rtc_region: options.rtcRegion,
video_quality_mode: options.videoQualityMode,
parent_id: parent,
lock_permissions: options.lockPermissions,
Expand Down

0 comments on commit 9461045

Please sign in to comment.