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

Cache exceptions/User-Agents translation #52

Merged
merged 4 commits into from
Dec 29, 2016
Merged

Cache exceptions/User-Agents translation #52

merged 4 commits into from
Dec 29, 2016

Conversation

timse201
Copy link
Contributor

No description provided.

Zodiac1978
Zodiac1978 previously approved these changes Dec 28, 2016
@bueltge
Copy link
Member

bueltge commented Dec 28, 2016

@timse201 please review this pull request more for the codex, use blanks after a open and closed brackets, like current __('Post/Pages-IDs ... in __( 'Post/Pages-IDs ....
Also please santitize the input from translation files, in this context is esc_html__( the right choice.
But see this cheat sheet https://gist.github.com/bueltge/49c652187ef10a448bfb

@@ -1605,8 +1605,8 @@ private static function _minify_select()
{
return array(
self::MINIFY_DISABLED => esc_html__('No minify', 'cachify'),
self::MINIFY_HTML_ONLY => 'HTML',
self::MINIFY_HTML_JS => 'HTML + Inline JavaScript'
self::MINIFY_HTML_ONLY => esc_html__('HTML', 'cachify'),
Copy link
Member

Choose a reason for hiding this comment

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

Please, I will buy a blank for esc_html__( 'HTML', 'cachify' ),, before and after brackets.

@@ -1776,14 +1776,14 @@ public static function options_page()
<fieldset>
<label for="cachify_without_ids">
<input type="text" name="cachify[without_ids]" id="cachify_without_ids" value="<?php echo esc_attr($options['without_ids']) ?>" />
Post/Pages-IDs
__('Post/Pages-IDs (separated by commas)', 'cachify' );
Copy link
Member

Choose a reason for hiding this comment

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

Also here, I will buy a blank tp esc_html__( ' and a filter function to sanitize the input string.

@Zodiac1978 Zodiac1978 dismissed their stale review December 28, 2016 20:15

Missing coding standards

@bueltge
Copy link
Member

bueltge commented Dec 29, 2016

Really great, thanks a lot for the effort.

@bueltge bueltge added this to the 2.3.0 milestone Dec 29, 2016
@bueltge bueltge merged commit 3e68897 into pluginkollektiv:master Dec 29, 2016
@bueltge
Copy link
Member

bueltge commented Dec 29, 2016

@timse201 can you enhance the CHANGELOG.md with your changes, for the headline Unreleased. Thanks a lot.

@timse201 timse201 deleted the patch-3 branch December 29, 2016 15:35
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