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

Violates not-null constraint in PG when deleting on_hand number on the Product Edit Page #2096

Closed
MJLang opened this Issue Oct 16, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@MJLang

MJLang commented Oct 16, 2012

When deleting the number of the on_hand number in the admin interface for the Product Edit Page, and updating it throws following error:

PG::Error: ERROR: null value in column "count_on_hand" violates not-null constraint
: UPDATE "spree_products" SET "count_on_hand" = NULL, "updated_at" = '2012-10-16 22:54:43.381920' WHERE "spree_products"."id" = 1

@radar

This comment has been minimized.

Show comment
Hide comment
@radar

radar Oct 16, 2012

Member

What version of Spree is this on? Are you just setting the field to a blank?

Member

radar commented Oct 16, 2012

What version of Spree is this on? Are you just setting the field to a blank?

@MJLang

This comment has been minimized.

Show comment
Hide comment
@MJLang

MJLang Oct 16, 2012

Its the master branch, yes, i'm just setting the field blank.

MJLang commented Oct 16, 2012

Its the master branch, yes, i'm just setting the field blank.

@radar

This comment has been minimized.

Show comment
Hide comment
@radar

radar Oct 17, 2012

Member

I've been able to verify this issue exists on master.

Member

radar commented Oct 17, 2012

I've been able to verify this issue exists on master.

@radar radar closed this in 40c8cf1 Oct 17, 2012

radar added a commit that referenced this issue Oct 17, 2012

radar added a commit that referenced this issue Oct 17, 2012

@radar

This comment has been minimized.

Show comment
Hide comment
@radar

radar Oct 17, 2012

Member

This problem has been fixed on 1-1-stable, 1-2-stable and master. Thanks!

Member

radar commented Oct 17, 2012

This problem has been fixed on 1-1-stable, 1-2-stable and master. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment