refactor: make ownerID nullable

discordjs · Sep 19, 2024 · aceb5b3 · aceb5b3
1 parent c48d8f5
commit aceb5b3
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 7 deletions.
diff --git a/packages/discord.js/src/errors/Messages.js b/packages/discord.js/src/errors/Messages.js
@@ -95,7 +95,8 @@ const Messages = {
   [DjsErrorCodes.ChannelNotCached]: 'Could not find the channel where this message came from in the cache!',
   [DjsErrorCodes.StageChannelResolve]: 'Could not resolve channel to a stage channel.',
   [DjsErrorCodes.GuildScheduledEventResolve]: 'Could not resolve the guild scheduled event.',
-  [DjsErrorCodes.FetchOwnerId]: type => `Couldn't resolve the ${type} ownerId to fetch the ${type} member.`,
+  [DjsErrorCodes.FetchOwnerId]: type =>
+    `Couldn't resolve the ${type} ownerId to fetch the ${type} ${type === 'group DM' ? 'owner' : 'member'}.`,
 
   [DjsErrorCodes.InvalidType]: (name, expected, an = false) => `Supplied ${name} is not a${an ? 'n' : ''} ${expected}.`,
   [DjsErrorCodes.InvalidElement]: (type, name, elem) => `Supplied ${type} ${name} includes an invalid element: ${elem}`,

diff --git a/packages/discord.js/src/structures/PartialGroupDMChannel.js b/packages/discord.js/src/structures/PartialGroupDMChannel.js
@@ -45,11 +45,15 @@ class PartialGroupDMChannel extends BaseChannel {
      */
     this.messages = new PartialGroupDMMessageManager(this);
 
-    /**
-     * The user id of the owner of this Group DM Channel
-     * @type {Snowflake}
-     */
-    this.ownerId = data.owner_id;
+    if ('owner_id' in data) {
+      /**
+       * The user id of the owner of this Group DM Channel
+       * @type {?Snowflake}
+       */
+      this.ownerId = data.owner_id;
+    } else {
+      this.ownerId ??= null;
+    }
   }
 
   /**
@@ -67,6 +71,10 @@ class PartialGroupDMChannel extends BaseChannel {
    * @returns {Promise<User>}
    */
   async fetchOwner(options) {
+    if (!this.ownerId) {
+      throw new DiscordjsError(ErrorCodes.FetchOwnerId, 'group DM');
+    }
+
     return this.client.users.fetch(this.ownerId, options);
   }
 

diff --git a/packages/discord.js/typings/index.d.ts b/packages/discord.js/typings/index.d.ts
@@ -2558,7 +2558,7 @@ export class PartialGroupDMChannel extends BaseChannel {
   public icon: string | null;
   public recipients: PartialRecipient[];
   public messages: PartialGroupDMMessageManager;
-  public ownerId: Snowflake;
+  public ownerId: Snowflake | null;
   public iconURL(options?: ImageURLOptions): string | null;
   public fetchOwner(options?: BaseFetchOptions): Promise<User>;
   public toString(): ChannelMention;