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.

Sign up for GitHub

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

Jump to bottom

Creating a COG without a type causes a crashes #5293

Closed
grantfitzsimmons opened this issue Sep 28, 2024 · 2 comments · Fixed by #5257
Closed

Creating a COG without a type causes a crashes #5293

grantfitzsimmons opened this issue Sep 28, 2024 · 2 comments · Fixed by #5257
Labels
1 - Bug Incorrect behavior of the product geospecify
Milestone
7.9.8

Comments

@grantfitzsimmons
Copy link
Member

grantfitzsimmons commented Sep 28, 2024 •
edited
Loading

After creating my own COG type records, I tried to create a COG.

COGTypeID Name Type Version TimestampCreated TimestampModified CollectionID CreatedByAgentID ModifiedByAgentID
1 Discrete Discrete 0 2024-09-27 21:24:23.420 2024-09-27 21:24:23.420 4
2 Consolidated Consolidated 0 2024-09-27 21:24:23.420 2024-09-27 21:24:23.420 4
3 Drill Core Drill Core 0 2024-09-27 21:24:23.420 2024-09-27 21:24:23.420 4

It failed when I tried to create a new COG record with this type set:

image 
AssertionError at /api/specify/collectionobjectgroup/ Bad URI: Consolidated Request
image

Specify 7 Crash Report - 2024-09-28T02_26_35.341Z.txt

😢
@grantfitzsimmons grantfitzsimmons added 1 - Bug Incorrect behavior of the product geospecify labels Sep 28, 2024
@grantfitzsimmons grantfitzsimmons added this to the 7.9.8 milestone Sep 28, 2024
@sharadsw
Copy link
Contributor

sharadsw commented Sep 30, 2024 •
edited
Loading

Cannot reproduce in #5257
I think your picklist might be a type 2 picklist due to which it's not able to create the correct relation for Cog -> cogtype when saving. It should work with a type 1 picklist like this one.
image

@CarolineDenis CarolineDenis added the todo:verify Needs Verification label Oct 1, 2024
@grantfitzsimmons grantfitzsimmons closed this as not planned Won't fix, can't repro, duplicate, stale Oct 8, 2024
@grantfitzsimmons grantfitzsimmons removed the todo:verify Needs Verification label Oct 21, 2024
@grantfitzsimmons grantfitzsimmons changed the title Cannot create COG via the UI Creating a COG without a type causes a crashes Oct 21, 2024
@grantfitzsimmons
Copy link
Member Author

@sharadsw If I do not set a type, I still get a crash.

image

IntegrityError at /api/specify/collectionobjectgroup/ (1048, "Column 'COGTypeID' cannot be null")

Specify 7 Crash Report - 2024-10-21T18_39_19.808Z.txt

@sharadsw sharadsw mentioned this issue Oct 22, 2024
Merged
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1 - Bug Incorrect behavior of the product geospecify
Projects
None yet
Milestone
7.9.8
Development

Successfully merging a pull request may close this issue.

Update Schema Config to include new geo fields and COG picklists specify/specify7
3 participants
@sharadsw @grantfitzsimmons @CarolineDenis