Skip to content

Commit

Permalink
[TASK] Remove the $css parameter from the CssInliner constructor
Browse files Browse the repository at this point in the history
This interface change the is first step toward having CssInliner
inherit from AbstractHtmlProcessor.
  • Loading branch information
oliverklee committed Feb 20, 2018
1 parent e39796f commit b632afa
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Expand Up @@ -13,7 +13,8 @@ This project adheres to [Semantic Versioning](https://semver.org/).
([#513](https://github.com/jjriv/emogrifier/pull/513),
[#516](https://github.com/jjriv/emogrifier/pull/516))
- Add a CssInliner class
([#514](https://github.com/jjriv/emogrifier/pull/514))
([#514](https://github.com/jjriv/emogrifier/pull/514),
[#522](https://github.com/jjriv/emogrifier/pull/522))
- Composer scripts for the various CI build steps
- Validate the composer.json on Travis
([#476](https://github.com/jjriv/emogrifier/pull/476))
Expand Down
4 changes: 1 addition & 3 deletions src/Emogrifier/CssInliner.php
Expand Up @@ -221,12 +221,10 @@ class CssInliner
* The constructor.
*
* @param string $html the HTML to emogrify, must be UTF-8-encoded
* @param string $css the CSS to merge, must be UTF-8-encoded
*/
public function __construct($html = '', $css = '')
public function __construct($html = '')
{
$this->setHtml($html);
$this->setCss($css);
}

/**
Expand Down

0 comments on commit b632afa

Please sign in to comment.