There are so much useless empty lines in your coding… can’t you format it correctly, please ?
The plugin follows WordPress standards for coding and for inline documentation. These standards are in place for both documentation and readability.
If the issue is that every other line is a blank line – that’s the result of either transferring the file as ascii instead of binary or across platforms. That’s not actually in the plugin, which you can view on trac.
There are some missing translations too, “blocked, unblocked…”.
The plugin is in the ww.wp.xz.cn polyglots project and has been for some time now. If there are missing strings in your particular translation, feel free to contribute to the translation. Language packs from ww.wp.xz.cn will only download automatically when the translation reaches 95%, otherwise the fallback is the included language files which are no longer supported.
https://translate.ww.wp.xz.cn/projects/wp-plugins/wp-members
Hi Chad,
Thanks for your answer.
For empty lines, I mean useless ones as you can see for instance line 130 to 133 in:
lib/recaptchalib.php.
For translation, I mean inside the code, for instance in admin/post.php, line 149, replace:
$action = ( $_REQUEST['a'] == 'block' ) ? 'blocked' : 'unblocked';
with
$action = ( $_REQUEST['a'] == 'block' ) ? __( 'Blocked', 'wp-members' ) : __( 'Unblocked', 'wp-members' );
Regards
For empty lines, I mean useless ones asyou can see for instance line 130 to 133 in:
recaptchalib.php.
That file is Google’s code library for reCAPTCHA, so that’s not going to change – it comes that way directly from Google’s reCAPTCHA project. (Additionally, it is for reCAPTCHA 1.0 which is scheduled to be removed from the plugin anyway since it’s no longer supported – no one should be using it.)
The specific translation section you mentioned I’ll update – I think that had not been updated as the intention was to change this entirely at some point since grammatically it may not always make sense the way it would currently display in English. But inclusion of custom post types makes this problematic. So I think for now I’ll just change it to what you suggested.
Hi Chad,
It was just an example for reCAPTCHA… I could notice the same in your own files.
OK, thanks for the next update.
There are some other missing gettext strings, I’ll check and inform you which ones.
Regards.
