Discord.js: Promise from `Guild.createRole` does not fail when maximum role limit is reached.

Created on 17 Aug 2017  Â·  11Comments  Â·  Source: discordjs/discord.js

Please describe the problem you are having in as much detail as possible:
image
In that image, I've placed two red marks next to some code. In most cases, only the first marked line would run. In a case where Discord's maximum role limit is reached, you'd expect that the second marked line would run after the promise fails. Instead, when the maximum role limit is reached and Guild.createRole is called, neither line runs and nothing happens. The promise doesn't fail and no error or warning is displayed in the console.

Further details:

  • discord.js version: 11.1.0
  • node.js version: 7.8.0
  • Operating system: Ubuntu 16.04.3 LTS
  • Priority this issue should have – please be realistic and elaborate if possible:
    This issue does not need high priority, as I can just tell members myself that we've hit the role limit instead of having the bot do it automatically.

  • [ ] I found this issue while running code on a __user account__
  • [ ] I have also tested the issue on latest master, commit hash:
low REST bug

All 11 comments

Same with createChannel

@ShayBox @Miroware8 please confirm the master hashes you tested this bug with

@devsnek How?

@Miroware8 check your package-lock.json

@CyberiumShadow What am I looking for?

@Miroware8 A line that looks like this
image

or like npm ls discord.js

I've confirmed that both errors are emitted on commit hash c4df2502 of master.

Channel limit: http://i.imgur.com/WIhLjtu.png

Role limit: http://i.imgur.com/dLpr9gz.png

@CyberiumShadow I don't have a line like that. All three occurrences of "discord.js" are in this screenshot.
image

@Miroware8 Gus was asking about the master branch commit you tested this on -- but seeing as you're on version 11.1, I assume you didn't test it on master.

Also see my comment above, the issue is fixed on the master branch.

^

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Lombra picture Lombra  Â·  3Comments

tiritto picture tiritto  Â·  3Comments

xCuzImPro picture xCuzImPro  Â·  3Comments

Dmitry221060 picture Dmitry221060  Â·  3Comments

Alipoodle picture Alipoodle  Â·  3Comments