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

Cannot save an attribute in orderline #3207

Closed
metas-dh opened this Issue Dec 13, 2017 · 1 comment

Comments

Projects
None yet
3 participants
@metas-dh
Member

metas-dh commented Dec 13, 2017

Is this a bug or feature request?

Bug

What is the current behavior?

An attribute cannot be saved when set in orderline (sales order).
Pls note: this is only happening in Swing, it worked fine win WebUI.

Which are the steps to reproduce?

  1. Have a product from a product category, which has an attributesetinstance set (e.g. Convenience Salate)
  2. Create a sales order in Swing Client
  3. Create an orderline with that product
  4. Change the attribute in orderline, select e.g. Bio
  5. Try to save:

Ausnahme:
Ihr Test hat einen bisher unentdeckten Fehler offengelegt.
Bitte leiten Sie diese Meldung an metas weiter: Error: This method shall not be called with M_AttributeSetInstance_ID=0

Nothing shown in console though.

What is the expected or desired behavior?

Should work without errors.

@metas-dh metas-dh added the type:bug label Dec 13, 2017

metas-ts added a commit that referenced this issue Dec 14, 2017

solve problem by *not* using legacy constructor with id=0
..because in ASIs, null does *not* mean new
Cannot save an attribute in orderline #3207

metas-ts added a commit that referenced this issue Dec 14, 2017

update release notes
 [#3207](#3207) Cannot
save an attribute in orderline
[#3208](#3208) console
error when copying MDocType
@metas-lc

This comment has been minimized.

Show comment
Hide comment
@metas-lc

metas-lc Dec 15, 2017

Contributor

IT
repeated the testcase above in both swing and webui: works fine OK

Contributor

metas-lc commented Dec 15, 2017

IT
repeated the testcase above in both swing and webui: works fine OK

@metas-lc metas-lc closed this Dec 15, 2017

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