954966 - Allow using a custom username for IRC.

Status: RESOLVED FIXED

(Chat Core :: IRC, defect)

Description

[Patrick Cloke [:clokep]]

*** Original post on bio 1534 at 2012-06-21 02:17:00 UTC ***

*** Due to BzAPI limitations, the initial description is in comment 1 ***

Comment 1

[Patrick Cloke [:clokep]]

Attachment: Patch v1

*** Original post on bio 1534 as attmnt 1656 at 2012-06-21 02:17:00 UTC ***

As requested by Mook: allow using a custom username (instead of brandShortName, e.g. Instantbird or Thunderbird) by using a hidden pref on the account: "username". Note that this also matches the old libpurple pref name, so it should make migration make more sense.

I'd really rather not expose this to the user, but I have no issue supporting it in the backend.

And because you requested it Mook, you get to review it.

Comment 2

[Instantbot]

Comment on attachment 8353413 [details] [diff] [review]
Patch v1

*** Original change on bio 1534 attmnt 1656 by mook.moz+bugs.instantbird AT gmail.com at 2012-06-21 03:27:10 UTC ***

Thanks!
This is once of those places where if getString just returned undefined for missing prefs, it would be cleaner:
let username = this.getString("username") || l10nHelper("...")("brandshortName");
(which would catch the case where the user manually set it to an empty string)

... actually, just think about the empty string case and make sure it's allowable per spec, even for your existing change, please :)

Comment 3

[Florian Quèze [:florian]]

*** Original post on bio 1534 at 2012-06-21 09:14:03 UTC ***

(In reply to comment #1)

> ... actually, just think about the empty string case and make sure it's
> allowable per spec, even for your existing change, please :)

So you r+'ed, but it's actually an r-, right? ;)

Comment 4

[Patrick Cloke [:clokep]]

Attachment: Patch v2

*** Original post on bio 1534 as attmnt 1657 at 2012-06-21 10:36:00 UTC ***

Good call on the empty username. This avoids that situation.

Comment 5

[Patrick Cloke [:clokep]]

Comment on attachment 8353413 [details] [diff] [review]
Patch v1

*** Original change on bio 1534 attmnt 1656 at 2012-06-21 10:36:11 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 6

[Instantbot]

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

*** Original change on bio 1534 attmnt 1657 by mook.moz+bugs.instantbird AT gmail.com at 2012-06-21 16:11:54 UTC ***

Flo: it was a r+ but make sure condition X was considered :)

Comment 7

[Patrick Cloke [:clokep]]

*** Original post on bio 1534 at 2012-06-24 01:30:26 UTC ***

Committed as http://hg.instantbird.org/instantbird/rev/9134c46bae1b