Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"field must be a number" when creating queue/topic entity #548

Closed
kaluznyt opened this issue May 19, 2021 · 8 comments · Fixed by #550
Closed

"field must be a number" when creating queue/topic entity #548

kaluznyt opened this issue May 19, 2021 · 8 comments · Fixed by #550
Milestone

Comments

@kaluznyt
Copy link

kaluznyt commented May 19, 2021

Hello,

I noticed in the newer versions when creating a messaging entity like queue or topic, and not filling in all of the fields it cannot create due to validation "The X value of the X field must be a number". It seems it was a good approach to default to 0 when a value is not provided. But perhaps there is some thought behind adding this validation ? It's not an issue, but just a small inconvenience :)

Tomasz

@ErikMogensen
Copy link
Collaborator

Hi,

I just tested to create a queue and a topic and both worked as long as I put anything in the Relative URI textbox. For a short while this didn't work but I don't think that was released.

Have you tried the latest version? There were some versions released where it shows the latest released version in the About box even though the actual version is not that version.

@kaluznyt
Copy link
Author

I just downloaded latest version, yet this is occurring:
image
image
image

@kaluznyt
Copy link
Author

other issue I think worked earlier is when I have a queue with "session" enabled, I'm not able to view the messages due to:
<20:49:47> The queue abc has been successfully created.
<20:49:55> Exception: It is not possible for an entity that requires sessions to create a non-sessionful message receiver. TrackingId:d18faf83-a76e-45fa-b305-fd2b61e0be1a_G1_B0, SystemTracker:abcdefg123:Queue:abc, Timestamp:2021-05-19T18:49:54
<20:49:55> InnerException: It is not possible for an entity that requires sessions to create a non-sessionful message receiver. TrackingId:d18faf83-a76e-45fa-b305-fd2b61e0be1a_G1_B0, SystemTracker:abcdefg123:Queue:abc, Timestamp:2021-05-19T18:49:54
but that's another thing, perhaps I could create another case

@kaluznyt
Copy link
Author

Ok, just took a look at the code in TimeSpanControl and it seems that it's by design that all of the particular fields are being validated, so perhaps need to just live with it and always put zeroes in the empty fields :)

@ErikMogensen
Copy link
Collaborator

Hmm, strange. Are you able to create a queue just by filling in the Relative URI field and then clicking Create?

Please create another issue for the session problem.

@kaluznyt
Copy link
Author

yeah, if there is just Relative URI then it creates it with all defaults, but if just any number is put in any of fields like I would like 1 minute lock timeout, then it fails the validation, since other "empty" fields need to be filled with zeroes

@ErikMogensen
Copy link
Collaborator

I see.

@Greybird, it would be great if you looked into how this could be fixed.

@Greybird
Copy link
Contributor

@ErikMogensen sure, will take à look !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants