patrickTingen / DataDigger

A dynamic dataviewer for your Progress / OpenEdge databases
https://datadigger.wordpress.com/
MIT License
50 stars 23 forks source link

drop the widget-id from the main window #82

Closed slegian closed 2 years ago

slegian commented 2 years ago

Hi Patrick,

We recently switched the -usewidgetid startup parameter on as we use automated test tools and getting some duplicate widget ids error messages.

A quick fix is to simply remove the widget-ids from the code. I did that in the main window, there might be more playing up, I'll fix as I come across them, but you might want to turn the auto widget-id assignment option off in your dev environment so you don't get them in the code in the first place...

Cheers, Simi

patrickTingen commented 2 years ago

Hi Simi,

Thanks for your pull request. I have no use for the WIDGET-IDs so I will remove them. Your request is on the master branch, but since I only develop on the develop-branch, I will not merge it. I will remove all WIDGET-IDs in the dev branch, though. I created a small program that removes all widget-ids from all sources (it's quite a lot to do by hand and the UIB cannot auto-remove them)

You might want to take a look at the dev branch. That's where I collect changes until there is enough to justify a new version. The dev branch is just as stable (if not more) than the master.

slegian commented 2 years ago

Hi Patrick, thanks for letting me know about the proper branch to use. I'll switch to the dev branch as well. Cheers