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 ability to remove previously parsed resources no longer getting parsed #147

Open
coffee2code opened this issue Dec 19, 2014 · 5 comments

Comments

@coffee2code
Copy link
Contributor

It'd be nice to optionally be able to remove previously parsed resources that were not included as part of a full re-parsing.

Remove may mean delete, but more likely assignment of a non-"publish" post status.

Two examples where this is useful:

  1. Inadvertently (or intentionally, but with later misgivings about doing so) including a file in a parsing that should not have been included. (In this case you may actually want the parsed resources deleted.)
  2. Functions, hooks, classes, etc that no longer exist and that you no longer want to have listed/available as if they did. (In this case you probably don't want to delete as there may be meta data added to these resources after they were parsed, such as comments, etc.)

This could be a wontfix under the belief that it's the parser user's responsibility to handle this. In which case, the parser user would probably hook 'wp_parser_import_item_post_data' and forcibly set post_modified for everything sent its way (since currently, wp_update_post() is not called if there wasn't a change in the post object data). Then after parsing, a script could remove anything not modified since the known parsing datetime (and anything related to them).

But shouldn't the parser have some responsibility (or capability) here if it does updates?

@JDGrimes
Copy link
Contributor

I think in the case of 2 we probably want to just mark those elements as deleted or removed. This might be similar to marking an element as deprecated in the reference. Then it is up to the user to decide whether they want to display removed elements, or hide them.

@Rarst
Copy link
Contributor

Rarst commented Dec 20, 2014

For reasonable performance we could track of touched IDs, then update post modified with one query at end of the process.

Doing anything with the information probably falls on end user's integration side. At most I can see opt-in un-publishing.

@florianbrinkmann
Copy link

Any news on that? Would be great to have a way to automatically remove/unpublish posts of functions/hooks/classes which are no longer part of the parsed content :)

@sebastienserre
Copy link

Hello, is someone worked around something that make this job?

@csalzano
Copy link

Hi, I wrote two plugins to modify the way phpdoc-parser works and published them with READMEs:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants