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

Sort files to allow for some basic dependency ordering through file names. #51

Closed
wants to merge 1 commit into from

Conversation

simesy
Copy link

@simesy simesy commented Dec 7, 2015

To allow a simple work around for https://www.drupal.org/node/2630014

@larowlan
Copy link
Owner

larowlan commented Dec 7, 2015

There is a graph sort already, based on hierarchy. If that's not working,
there is a bug in core serialization. I know that term parents field is
broken and pretty sure menu link parents are too, both of those use custom
storage instead of the normal ER ones. So those need either custom
normalizers that work around core bugs, or we wait for the core bugs to be
fixed. I have one in Entity Pilot for terms, we can copy that here and also
add one for menu. Will paste links to core bugs here when back at desk.
On 7 Dec 2015 4:57 pm, "Simon Hobbs" notifications@github.com wrote:

To allow a simple work around for https://www.drupal.org/node/2630014

You can view, comment on, or merge this pull request online at:

#51
Commit Summary

  • Sort files to allow for some basic dependency ordering through file
    names.

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#51.

@larowlan
Copy link
Owner

larowlan commented Dec 7, 2015

@simesy
Copy link
Author

simesy commented Dec 8, 2015

Thanks Lee

@simesy simesy closed this Dec 8, 2015
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