Author Topic: Ping patch  (Read 4783 times)

0 Members and 2 Guests are viewing this topic.

Offline tomiks

  • Newbie
  • *
  • Posts: 2
  • Country: pl
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

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 tomiks

  • Newbie
  • *
  • Posts: 2
  • Country: pl
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 »