Closed Bug 954966 Opened 10 years ago Closed 10 years ago

Allow using a custom username for IRC.

Categories

(Chat Core :: IRC, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: clokep, Assigned: clokep)

Details

Attachments

(1 file, 1 obsolete file)

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

*** Due to BzAPI limitations, the initial description is in comment 1 ***
Attached patch Patch v1 (obsolete) — Splinter Review
*** 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.
Attachment #8353413 - Flags: review?(bugzilla)
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 :)
Attachment #8353413 - Flags: review?(bugzilla) → review+
*** 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? ;)
Attached patch Patch v2Splinter Review
*** 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.
Attachment #8353414 - Flags: review?(bugzilla)
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 ***
Attachment #8353413 - Attachment is obsolete: true
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 :)
Attachment #8353414 - Flags: review?(bugzilla) → review+
Whiteboard: [checkin-needed]
*** Original post on bio 1534 at 2012-06-24 01:30:26 UTC ***

Committed as http://hg.instantbird.org/instantbird/rev/9134c46bae1b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [checkin-needed]
Target Milestone: --- → 1.2
You need to log in before you can comment on or make changes to this bug.