-
Notifications
You must be signed in to change notification settings - Fork 52
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
Extract all supported functions. #13
Conversation
https://github.com/oscarotero/Gettext requires PHP 5.4! |
Hmm. I simply copied It was added to the script over seven years ago in https://i18n.trac.wordpress.org/changeset/15325. I guess it's a remnant from the earlier days of WordPress. We should remove it from this script as it serves no purpose. |
Actually, I think we can remove support for These functions were added to We can remove them now and re-add them as soon as they actually exist. However, I don't think these functions will ever make it to core in that form |
Extract all supported functions.
See #12 (comment)
Renames function map names to be more meaningful in
WordPress_Code_Extractor
and supports all inWordPress_Functions_Scanner::saveGettextFunctions()
and errors if unmapped found.Also errors if
comments_number_link()
found (not sure what to do here).Adds tests.