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

Add missing XSD definitions #2244

Merged
merged 2 commits into from
Sep 9, 2021
Merged

Conversation

phansys
Copy link
Collaborator

@phansys phansys commented Aug 27, 2021

Update XSD in order to add missing definitions:

  • tree-path type;
  • hard-delete and time-aware attributes to "soft-deleteable" type;

See

$config['timeAware'] = false;
if ($this->_isAttributeSet($xml->{'soft-deleteable'}, 'time-aware')) {
$config['timeAware'] = $this->_getBooleanAttribute($xml->{'soft-deleteable'}, 'time-aware');
}
$config['hardDelete'] = true;
if ($this->_isAttributeSet($xml->{'soft-deleteable'}, 'hard-delete')) {
$config['hardDelete'] = $this->_getBooleanAttribute($xml->{'soft-deleteable'}, 'hard-delete');
}

} elseif (isset($mapping->{'tree-path'})) {

@phansys phansys marked this pull request as ready for review August 27, 2021 22:01
@phansys phansys changed the title Add missing attributes to "soft-deleteable" type in XSD Add missing XSD definitions Aug 30, 2021
@AkenRoberts AkenRoberts merged commit 98fdbf5 into doctrine-extensions:main Sep 9, 2021
@AkenRoberts
Copy link
Member

Thank you 🥳

@phansys phansys deleted the xsd branch September 9, 2021 02:27
paxal pushed a commit to sportmium/DoctrineExtensions that referenced this pull request Mar 11, 2022
* Add missing attributes to "soft-deleteable" type in XSD

* Add missing XSD entry for "tree-path"
@fri0z
Copy link

fri0z commented Jul 11, 2022

And when is it planned to update http://gediminasm.org/schemas/orm/doctrine-extensions-mapping ?

rotdrop pushed a commit to rotdrop/DoctrineExtensions that referenced this pull request Apr 15, 2023
* Add missing attributes to "soft-deleteable" type in XSD

* Add missing XSD entry for "tree-path"
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.

None yet

3 participants