Author Topic: Ping patch  (Read 324 times)

0 Members and 1 Guest are viewing this topic.

Offline tomiksTopic starter

  • Newbie
  • *
  • Posts: 2
  • Country: pl
  • Karma: 0
Ping patch
« on: 21 06 2016, 19:38:09 »
Hi.

Would a dev commit this patch?

- Fixed bug storing Control Account in db and select proper module for status control.

Thanks
 

Offline ghazan

  • Miranda NG founder
  • Administrator
  • *****
  • Posts: 339
  • Country: ru
  • Karma: 34
  • Jabber ID
Re: Ping patch
« Reply #1 on: 22 06 2016, 17:06:03 »
nope, not acceptable.

hint: the human readable account name might not be unique, so you still need to store the ANSI protocol name of account
the only change you need to make is to visualize the tszAccountName in the options dialog, saving its szModuleName as list item's custom info, for example
 

Offline tomiksTopic starter

  • Newbie
  • *
  • Posts: 2
  • Country: pl
  • Karma: 0
Re: Ping patch
« Reply #2 on: 23 06 2016, 07:56:26 »
Hi.

Thanks for the suggestion.
I've changed the patch. IP item is now module related, so status control behavior is by menu status control consistent.
« Last Edit: 23 06 2016, 10:22:19 by tomiks »