Exclude broadcaster if username in uppercase

belayanejnost · November 29, 2020, 2:51am

Expected behavior

Should be excluded.

Actual behavior

I setter broadcaster username in uppercase, and it’s not excluded in top system.

Like i’m using !top time and broadcaster in list.

Steps to reproduce

Set channel name to uppercased.

Additional informations

master https://github.com/sogehige/sogeBot/commit/f4b1ae511c3a4143e48a8aca71ca84e011c1de77

soge · November 30, 2020, 1:49pm

Can you give an example? We are already excluding lower cased broadcaster

github.com

sogehige/sogeBot/blob/master/src/bot/systems/top.ts#L122


const type = opts.parameters;
// count ignored users
const _total = 10 + getIgnoreList().length;
const connection = await getConnection();
switch (type) {
  case TYPE.TIME:
    sorted
      = (await getRepository(User).createQueryBuilder('user')
        .where('user.username != :botusername', { botusername: oauth.botUsername.toLowerCase() })
        .andWhere('user.username != :broadcasterusername', { broadcasterusername: oauth.broadcasterUsername.toLowerCase() })
        .orderBy('user.watchedTime', 'DESC')
        .limit(_total)
        .getMany())
        .filter(o => !isIgnored({ username: o.username, userId: o.userId }))
        .map(o => {
          return { username: o.username, value: o.watchedTime };
        });
    message = translate('systems.top.time').replace(/\$amount/g, 10);
    break;
  case TYPE.TIPS:

belayanejnost · November 30, 2020, 8:17pm

Yea, i know, but for some reason it’s not working for me.

Example is simple, just write your channel in uppercase in oauth settings.

soge · December 1, 2020, 12:21pm

You mean channel or broadcaster? We are excluding only broadcaster, is user connected with broadcaster auth?