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

Changing the alternative of a parameter value associated with a parameter list with non-numeric values causes an error #2330

Closed
DillonJ opened this issue Sep 25, 2023 · 4 comments
Labels
bug Something isn't working
Milestone

Comments

@DillonJ
Copy link
Contributor

DillonJ commented Sep 25, 2023

See attached minimal example. To reproduce in the attached minimal example, change the alterative of the testp2 parameter value.

From any other DB:

  • Create a new parameter list with non-numeric values (e.g. "abc" and "def").
  • Now associate that list with an object parameter definition
  • Create a parameter value for an object selecting one of the items from the newly created list (e.g. "abc")
  • Create a new alternative called "dummy"
  • Change the alternative of the above non-numerical parameter value to "dummy".

You now get an error along the lines of

From test.sqlite:
Invalid value '3.0' - it should be one from the parameter value list: '"abc"', '"def"'.

test.zip

Tested in 0.7, haven't tried in 0.8

@DillonJ DillonJ added the bug Something isn't working label Sep 25, 2023
@jkiviluo
Copy link
Member

Great to have this reproduced. I've seen the error message many times, but haven't figured out the pattern. I don't think anything goes wrong other than the message, right? It's confusing though.

Let's see if it's in 0.8, fix there first and roll it to 0.7 too.

@jkiviluo jkiviluo modified the milestones: v0.8.0, V0.7 Sep 30, 2023
@PiispaH
Copy link
Collaborator

PiispaH commented Oct 11, 2023

This doesn't seem to happen in 0.8 anymore. Probably not worth it to fix in 0.7 @soininen @jkiviluo ?

@jkiviluo
Copy link
Member

I'm ok if it's not fixed in 0.7, since it doesn't corrupt data (although users do see it from time to time). How about @DillonJ?

@PiispaH
Copy link
Collaborator

PiispaH commented Apr 15, 2024

Doesn't replicate in 0.8.

@PiispaH PiispaH closed this as not planned Won't fix, can't repro, duplicate, stale Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants