feat: make Instance#fetch force true by default, and fix force parame…

…ter (#6116)

Co-authored-by: Almeida <[email protected]>

src/structures/Channel.js

@@ -102,11 +102,11 @@ class Channel extends Base {
 
   /**
    * Fetches this channel.
-   * @param {boolean} [force=false] Whether to skip the cache check and request the API
+   * @param {boolean} [force=true] Whether to skip the cache check and request the API
    * @returns {Promise<Channel>}
    */
-  fetch(force = false) {
-    return this.client.channels.fetch(this.id, true, force);
+  fetch(force = true) {
+    return this.client.channels.fetch(this.id, { force });
   }
 
   /**

src/structures/DMChannel.js

@@ -62,10 +62,10 @@ class DMChannel extends Channel {
 
   /**
    * Fetch this DMChannel.
-   * @param {boolean} [force=false] Whether to skip the cache check and request the API
+   * @param {boolean} [force=true] Whether to skip the cache check and request the API
    * @returns {Promise<DMChannel>}
    */
-  fetch(force = false) {
+  fetch(force = true) {
     return this.recipient.createDM(force);
   }
 

src/structures/GuildBan.js

@@ -52,10 +52,10 @@ class GuildBan extends Base {
 
   /**
    * Fetches this GuildBan.
-   * @param {boolean} [force=false] Whether to skip the cache check and request the API
+   * @param {boolean} [force=true] Whether to skip the cache check and request the API
    * @returns {Promise<GuildBan>}
    */
-  fetch(force = false) {
+  fetch(force = true) {
     return this.guild.bans.fetch({ user: this.user, cache: true, force });
   }
 }

src/structures/GuildMember.js

@@ -301,10 +301,10 @@ class GuildMember extends Base {
 
   /**
    * Fetches this GuildMember.
-   * @param {boolean} [force=false] Whether to skip the cache check and request the API
+   * @param {boolean} [force=true] Whether to skip the cache check and request the API
    * @returns {Promise<GuildMember>}
    */
-  fetch(force = false) {
+  fetch(force = true) {
     return this.guild.members.fetch({ user: this.id, cache: true, force });
   }
 

src/structures/Message.js

@@ -721,11 +721,11 @@ class Message extends Base {
 
   /**
    * Fetch this message.
-   * @param {boolean} [force=false] Whether to skip the cache check and request the API
+   * @param {boolean} [force=true] Whether to skip the cache check and request the API
    * @returns {Promise<Message>}
    */
-  fetch(force = false) {
-    return this.channel.messages.fetch(this.id, true, force);
+  fetch(force = true) {
+    return this.channel.messages.fetch(this.id, { force });
   }
 
   /**

src/structures/User.js

@@ -230,11 +230,11 @@ class User extends Base {
 
   /**
    * Fetches this user.
-   * @param {boolean} [force=false] Whether to skip the cache check and request the API
+   * @param {boolean} [force=true] Whether to skip the cache check and request the API
    * @returns {Promise<User>}
    */
-  fetch(force = false) {
-    return this.client.users.fetch(this.id, true, force);
+  fetch(force = true) {
+    return this.client.users.fetch(this.id, { force });
   }
 
   /**