955179 - Add NAMESX support (to get all user modes of someone instead of just one)

Status: RESOLVED FIXED

(Chat Core :: IRC, enhancement)

Description

[Benedikt Pfeifer]

*** Original post on bio 1748 at 2012-10-31 13:25:00 UTC ***

Currently we don't get all the user modes when we join a channel after someone. The NAMESX extension is supposed to fix this.

Thanks to clokep for already looking up documentation:
http://wiki.inspircd.org/Modules/2.1/namesx

Comment 1

[Patrick Cloke [:clokep]]

Attachment: Patch

*** Original post on bio 1748 as attmnt 2111 at 2012-11-21 15:47:00 UTC ***

I really dislike the loop introduced in the normalize method, but I couldn't come up with a cleaned way to do it...

Comment 2

[aleth [:aleth]]

Comment on attachment 8353872 [details] [diff] [review]
Patch

*** Original change on bio 1748 attmnt 2111 at 2012-11-21 21:00:38 UTC ***

Thanks for adding this!

>+    if (aPrefixes) {
>+      let i;
>+      for (i = 0; i < str.length && aPrefixes.indexOf(str[i]) != -1; ++i) { }
>+      str = str.slice(i);
>+    }
Like you said, this looks like it wants to be better, but I can't think of anything either.

Comment 3

[Florian Quèze [:florian]]

*** Original post on bio 1748 at 2012-11-21 23:51:13 UTC ***

(In reply to comment #2)
> Comment on attachment 8353872 [details] [diff] [review] (bio-attmnt 2111) [details]
> Patch
> 
> Thanks for adding this!
> 
> >+    if (aPrefixes) {
> >+      let i;
> >+      for (i = 0; i < str.length && aPrefixes.indexOf(str[i]) != -1; ++i) { }
> >+      str = str.slice(i);
> >+    }
> Like you said, this looks like it wants to be better, but I can't think of
> anything either.

I think I would prefer:
while (aPrefixes.indexOf(str[0]) != -1)
  str = str.slice(1);

Comment 4

[Florian Quèze [:florian]]

*** Original post on bio 1748 at 2012-11-24 21:37:35 UTC ***

(In reply to comment #3)
[...]
> I think I would prefer:
> while (aPrefixes.indexOf(str[0]) != -1)
>   str = str.slice(1);

So do you agree with this? If so, are you going to attach a new patch, or should I change it when doing the check-in?

Comment 5

[Patrick Cloke [:clokep]]

*** Original post on bio 1748 at 2012-11-25 01:11:26 UTC ***

(In reply to comment #4)
> (In reply to comment #3)
> [...]
> > I think I would prefer:
> > while (aPrefixes.indexOf(str[0]) != -1)
> >   str = str.slice(1);
> 
> So do you agree with this? If so, are you going to attach a new patch, or
> should I change it when doing the check-in?

I'll attach a new patch.

Comment 6

[Patrick Cloke [:clokep]]

Attachment: Patch v2

*** Original post on bio 1748 as attmnt 2129 at 2012-11-25 03:21:00 UTC ***

With Florian's nit.

Comment 7

[Patrick Cloke [:clokep]]

Comment on attachment 8353872 [details] [diff] [review]
Patch

*** Original change on bio 1748 attmnt 2111 at 2012-11-25 03:21:52 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 8

[aleth [:aleth]]

Comment on attachment 8353890 [details] [diff] [review]
Patch v2

*** Original change on bio 1748 attmnt 2129 at 2012-11-25 13:09:21 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 9

[Patrick Cloke [:clokep]]

*** Original post on bio 1748 at 2012-11-25 23:37:15 UTC ***

Committed as http://hg.instantbird.org/instantbird/rev/bac6503567d1