-
Notifications
You must be signed in to change notification settings - Fork 0
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
Inconsistent syntax for droppedVer in API #13
Comments
Found another syntax style that differ from the others: http://api.wpseek.com/?method=wordpress.getfunction&s=get_page
|
And another one: http://api.wpseek.com/?method=wordpress.getfunction&s=install_blog_defaults
|
Found another style: http://api.wpseek.com/?method=wordpress.getfunction&s=install_themes_feature_list
|
This one doesn't even have a dropped version number: http://api.wpseek.com/?method=wordpress.getfunction&s=wp_login
|
Hey Ola, I know this is a huge problem. The WordPress doc blocks are a vast desert when it comes to consistency. Some time ago I created a ticket with patches to address as many of these issues as possible: https://core.trac.wordpress.org/ticket/28806 Unfortunately fixing these will span beyond 4.0 as you can see in the last comment on my ticket. I'm gonna fix the issues with the API itself, but can't do much on the data provided through the WP code until my patches are merged. |
No worries. I had a feeling a lot of the problem was in the WordPress codebase. Let's wait until your patches are merged. |
I'll try to add some "legacy code" to address and consolidate the docblocks' data on my side which I can remove after my patches were merged. |
Consolidated most of the |
Using the API, the syntax for droppedVer is inconsistent.
Examples:
http://api.wpseek.com/?method=wordpress.getfunction&s=debug_fclose
http://api.wpseek.com/?method=wordpress.getfunction&s=automatic_feed_links
http://api.wpseek.com/?method=wordpress.getfunction&s=_fetch_remote_file
This makes it hard to fetch the actual dropped version number.
The text was updated successfully, but these errors were encountered: