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 the names of the functions used in each element #85

Merged
merged 3 commits into from
Apr 8, 2014

Conversation

JDGrimes
Copy link
Contributor

See #79

This is a proof of concept for exporting the functions used by each element. I've made no attempt to import this data here, because we haven't decided the best way of doing that yet. The idea is to maybe go ahead and get this committed so folks can hack away on the importing/display part.

Notes:

  • This only covers calls to regular functions. Class methods are different and can be added later.
  • I chose to export do_action and apply_filter calls, even though they are already being exported as hooks. My thinking here is that they will be weeded out on import or display as needed.

@@ -58,7 +61,9 @@ class File_Reflector extends FileReflector {
* We also check function calls to see if there are any actions or hooks. If
* there are, they are added to the file's hooks if in the global scope, or if
* we are in a function/method, they are added to the queue. They will be
* assinged to the function by leaveNode().
* assinged to the function by leaveNode(). We also check for any other function
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Existing typo here - assigned

Rarst added a commit that referenced this pull request Apr 8, 2014
Export the names of the functions used in each element
@Rarst Rarst merged commit 0a422e0 into WordPress:master Apr 8, 2014
@JDGrimes JDGrimes deleted the export-functions-called branch August 26, 2014 21:04
@JDGrimes JDGrimes mentioned this pull request Sep 29, 2014
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.

3 participants