ServiceBusExplorer icon indicating copy to clipboard operation
ServiceBusExplorer copied to clipboard

Fixed issue with test topic using more than one file

Open wmmihaa opened this issue 2 years ago • 5 comments

Related to Test topic using more than one file will still only take one file #700

wmmihaa avatar Nov 11 '22 16:11 wmmihaa

However, it seems that this fix makes the number of sent messages equal to the number of selected files.

Would this not be the expected behavior?

wmmihaa avatar Nov 12 '22 23:11 wmmihaa

The expected behavior is to send the number of messages that are set in the 'Message count' property on the 'Sender' tab. Since this may be confusing there is text explaining this behavior on the tab where you select the files.

I also replied to the issue, since it does not seem to be a bug.

ErikMogensen avatar Nov 13 '22 09:11 ErikMogensen

In my humble opinion the Message Count setting makes sense when using Messages but is confusing when it comes to Files. I've made some changes automatically updating the Message Count property to the number of selected files, while setting it back to DefaultMessageCount (1) if any of the other tabs are selected.

I've made the update for both Queues and Topics

https://user-images.githubusercontent.com/3214367/201526958-89348184-efd7-4eb5-9e26-eda546737b2e.mp4

wmmihaa avatar Nov 13 '22 14:11 wmmihaa

In my humble opinion

I have never seen you being humble 😄

Nice video and thanks for the addition to the queue control. I see two problems though. One is that if you have set the Message count to a higher value than the number of files it will be lowered. That could be fixed by making sure that the Message countis never lowered. The other problem is that changing the Message count may not be expected. That could perhaps be solved by changing the text that is above the list box and mentioning this behavior.

@paolosalvatori, @SeanFeldman do you have any input?

BTW, @wmmihaa were you able to open the controls in the Visual Studio designer? I am unable to do that. Most likely due to them inheriting from an abstract base class.

ErikMogensen avatar Nov 13 '22 20:11 ErikMogensen

One is that if you have set the Message count to a higher value than the number of files it will be lowered.

Yes I was considering pointing this out, but thought less of it. But yes, if the user has selected the Message tab and sets the Message Count to any value but one, he or she might find it confusing when it's reset to "1" after selecting the Files tab and then back to Messages.

However, I think the confusion has to be put in perspective to the relation of selecting five files and hitting Send and only one message is transmitted. Or what happens if I select five files and set the Message Count to 7?

The "best" solution might be to keep a separate Message Count for each tab, but then again, perhaps a proper description is all you need as I think it's quite a rare occurance.

BTW, @wmmihaa were you able to open the controls in the Visual Studio designer? I am unable to do that. Most likely due to them inheriting from an abstract base class.

To tell you the truth I havn't open a WinForm application in 10 year so I don't know what to expect. But to answer your question, no I do not see a designer.

wmmihaa avatar Nov 13 '22 21:11 wmmihaa