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

export -> properties -> property should be minOccurs=0 #116

Closed
merrycoder opened this issue Jan 2, 2020 · 1 comment · Fixed by #140
Closed

export -> properties -> property should be minOccurs=0 #116

merrycoder opened this issue Jan 2, 2020 · 1 comment · Fixed by #140
Assignees
Labels
Bug Broken functionality
Milestone

Comments

@merrycoder
Copy link
Collaborator

Problem description
It should be possible to create an empty importfile to remove all properties of a given owner from a platform. This is currently possible with IDX.

Desired Solution
Make export -> properties -> property minOccurs=0 instead of minOccurs=1

@jensstalder
Copy link
Collaborator

Thats correct that was in the original implementation. Its important that <properties /> is present however, this would represent a empty delivery.

aka.

<export>
   <projects></projects>
</export>

Would delete all projects but would leave properties alone. This will make possible seperation of project and property (or some other future resources) delivery in the future possible.

<export>
   <properties></properties>
</export>

Would delete all the properties

<export>
</export>

Would do nothing.

What do you think? This would probably be the result in having minOccurs=0 instead of =1 .

@jensstalder jensstalder added the Bug Broken functionality label Oct 16, 2020
@jensstalder jensstalder self-assigned this Oct 16, 2020
@jensstalder jensstalder linked a pull request Oct 16, 2020 that will close this issue
@jensstalder jensstalder added this to the 2.1.0 milestone Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Broken functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants