Skip to content

Fixed build log for invalid encoded utf8 characters. Issue #145.#146

Merged
corpsee merged 1 commit intophp-censor:non-utf-8-fixfrom
SimonHeimberg:non-utf-8-fix
Feb 3, 2018
Merged

Fixed build log for invalid encoded utf8 characters. Issue #145.#146
corpsee merged 1 commit intophp-censor:non-utf-8-fixfrom
SimonHeimberg:non-utf-8-fix

Conversation

@SimonHeimberg
Copy link
Contributor

No description provided.

@SimonHeimberg
Copy link
Contributor Author

The logging works with my data. Can still not garante that every case is covered now...

return $outputs;
}

private static function replaceIllegalCharacters($utf8String)
Copy link
Member

Choose a reason for hiding this comment

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

PHPCensor\Helper\CommandExecutor::replaceIllegalCharacters::PHPCensor\Helper\CommandExecutor::replaceIllegalCharacters is missing a docblock.

return $outputs;
}

private static function replaceIllegalCharacters($utf8String)
Copy link
Member

Choose a reason for hiding this comment

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

Class PHPCensor\Helper\CommandExecutor::replaceIllegalCharacters is missing a docblock.

@codecov
Copy link

codecov bot commented Feb 3, 2018

Codecov Report

Merging #146 into non-utf-8-fix will increase coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##             non-utf-8-fix     #146      +/-   ##
===================================================
+ Coverage            16.83%   16.87%   +0.04%     
- Complexity            3133     3134       +1     
===================================================
  Files                  202      202              
  Lines                12736    12743       +7     
===================================================
+ Hits                  2144     2151       +7     
  Misses               10592    10592
Impacted Files Coverage Δ Complexity Δ
src/PHPCensor/Helper/CommandExecutor.php 81.53% <100%> (+1.05%) 46 <1> (+1) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45f8102...48e81ef. Read the comment docs.

@SimonHeimberg
Copy link
Contributor Author

to get cleaner history, this pr can be directly merged into master

@corpsee corpsee merged commit f4c2a03 into php-censor:non-utf-8-fix Feb 3, 2018
@corpsee
Copy link
Member

corpsee commented Feb 3, 2018

Merged. Thanks! 👍

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants