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

Define an abstract record structure #23

Closed
clemens-tolboom opened this issue Jan 2, 2014 · 7 comments
Closed

Define an abstract record structure #23

clemens-tolboom opened this issue Jan 2, 2014 · 7 comments

Comments

@clemens-tolboom
Copy link
Collaborator

I think it would be nice to allow not only graphviz to use our UML data.

The transformation of our labels into #5 HTML like text is a post processing phase. Only needed for the dot language.

Other tools like thejit, d3js or graph_phyz allow for other table-like structures.

@clemens-tolboom
Copy link
Collaborator Author

I added a milestone to see how that works too.

Would the DOMDocument fill our needs as it's a tree with attributes.

@clue
Copy link
Owner

clue commented Jan 3, 2014

Thanks, certainly makes sense to track this a as dedicated ticket.

Would the DOMDocument fill our needs as it's a tree with attributes.

I'd rather not introduce that dependency. I'll see what I can come up with.

@clemens-tolboom
Copy link
Collaborator Author

I understand your hesitation for dependencies. But the other side is writing your own code.

Is https://github.com/nicmart/Tree useful?

@clemens-tolboom
Copy link
Collaborator Author

There exists http://www.php.net/manual/en/class.recursiveiterator.php which helps some I guess.

@clue
Copy link
Owner

clue commented Jul 7, 2016

I'm not sure I get the motivation for this change, perhaps you can elaborate? What kind of information does this "record structure" hold that can not be seen by plain reflection?

I understand that exports to other formats could be nice, but I'm unsure if this worth the hassle tbh.

@clemens-tolboom
Copy link
Collaborator Author

Not sure now :( ... won't fix?

@clue
Copy link
Owner

clue commented Jul 7, 2016

Alright, let's close this one until we find a good reason to reopen :-)

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

No branches or pull requests

2 participants