Opened 8 years ago

Closed 8 years ago

#7672 closed bug (fixed)

Empty NTP server added

Reported by: humdinger Owned by: leavengood
Priority: normal Milestone: R1
Component: Preferences/Time & Date Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

In the new Time prefs' network time tab you can add an emtpy entry. Also, the text field isn't cleared after adding an entry. The attached simple patch addresses the issues, but I'm not confident enough to apply it myself - noob here.

[Assigned the ticket to Ryan as he was involved recently. BTW, shouldn't the new Time prefs be added to alpha3. The network time thingy is kinda neat, and maybe more testing reveals bugs not found yet.)

Attachments (1)

time.diff (550 bytes) - added by humdinger 8 years ago.
Patch

Download all attachments as: .zip

Change History (3)

Changed 8 years ago by humdinger

Attachment: time.diff added

Patch

comment:1 Changed 8 years ago by humdinger

Has a Patch: set

comment:2 Changed 8 years ago by leavengood

Resolution: fixed
Status: newclosed

Fixed in hrev42097 with a cherry on top.

Learned a bit more about BTextControl and BTextView in the process, which is always good.

Note: See TracTickets for help on using tickets.