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

Update Image.php according to the last DAM-Update #45

Merged
merged 4 commits into from
May 16, 2017

Conversation

zuernBernhard
Copy link
Contributor

When I try to import images via Drag'nDrop from DAM or DEV-DAM into mylife I get an Error, that the mandatory attribute "status" is not avaiable. (Tested with https://dev-dam.burda.com/dcx/api/document/doc6t3knlo6a86naei27g1?x-doctype=documenttype-image for example).
If I move status from mandatory to optional it works without Problems. Can we merge this simple change or does it mean we get problems at another place that I don't know of ?

When I try to import images via Drag'nDrop from DAM or DEV-DAM into mylife I get an Error, that the mandatory attribute "status" is not avaiable. (Tested with https://dev-dam.burda.com/dcx/api/document/doc6t3knlo6a86naei27g1?x-doctype=documenttype-image for example).
If I move status from mandatory to optional it works without Problems. Can we merge this simple change or does it mean we get problems at another place that I don't know of ?
@zuernBernhard
Copy link
Contributor Author

The reason we have no status is that : "dcxapi:tm_topic/rightsusage-Online" is not present after the last DAM-Update. I'm in contact with Christian Stange about that.

@zuernBernhard zuernBernhard deleted the patch-1 branch April 27, 2017 09:24
@zuernBernhard zuernBernhard restored the patch-1 branch April 27, 2017 09:24
Update computation of Status according to the last DAM-Update
@zuernBernhard zuernBernhard reopened this Apr 27, 2017
@zuernBernhard
Copy link
Contributor Author

After further investigation with Christian Stange we came to the conclusion that status can remain as a mandatory attribute. But the computation of status changed a little bit since the last DAM-Update. This pull-request is updated according to that.

typo
@chrfritsch chrfritsch merged commit 603ce42 into BurdaMagazinOrg:8.x-2.x May 16, 2017
@zuernBernhard
Copy link
Contributor Author

Danke !

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

Successfully merging this pull request may close these issues.

2 participants