Exception in webhook admin UI

Fout
1

Since the removal of various _.underscore methods, I’ve been noticing a few null exceptions, some may be something I need to address in my plugins, but there’s at least one in core:

  1. Navigate to /admin/api/web_hooks/new

  2. An exception is thrown.

TypeError: Cannot read property 'forEach' of undefined
at Function.findByIds (discourse/models/category:232)

See DEV: removes _.each from codebase (#6629) · discourse/discourse@9672c92 · GitHub and comments.

@j.jaffeux

ps. @pfaffman

1 like
3

Yes will fix it tomorrow and maybe write a test for this too. I didnt want to check for niull everywhere. :+1:t2:

4 likes
4

There is already an open bug for this assigned to @vinothkannans

1 like
6

This is fixed as per commit

https://github.com/discourse/discourse/commit/036790d13c728e9d30223f0079d539b4de203e7f

4 likes