Skip to content

Commit a68451f

Browse files
Merge pull request #12 from cordoval/improvements
General Improvements (PSR4, MetaData, Travis)
2 parents dbce535 + cdf4a6c commit a68451f

15 files changed

+99
-100
lines changed

.gitignore

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
/phpunit.xml
2-
*~
1+
phpunit.xml
32
composer.lock
4-
/vendor/
3+
/vendor

.travis.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,5 @@ php:
44
- 5.3
55
- 5.4
66

7-
before_script:
8-
- curl -s http://getcomposer.org/installer | php
9-
- php composer.phar install --dev
7+
install:
8+
- composer install

Converter/MissingParamException.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
<?php
2+
23
/*
34
* This file is part of ToInlineStyleEmailBundle.
45
*
@@ -15,6 +16,7 @@
1516
*
1617
* @author Roberto Trunfio <[email protected]>
1718
*/
18-
class MissingParamException extends \Exception {
19+
class MissingParamException extends \Exception
20+
{
1921

2022
}

Converter/MissingTemplatingEngineException.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
<?php
2+
23
/*
34
* This file is part of ToInlineStyleEmailBundle.
45
*
@@ -15,6 +16,7 @@
1516
*
1617
* @author Roberto Trunfio <[email protected]>
1718
*/
18-
class MissingTemplatingEngineException extends \Exception {
19+
class MissingTemplatingEngineException extends \Exception
20+
{
1921

2022
}

Converter/ToInlineStyleEmailConverter.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
<?php
2+
23
/*
34
* This file is part of ToInlineStyleEmailBundle.
45
*
@@ -25,8 +26,8 @@
2526
*
2627
* @author Roberto Trunfio <[email protected]>
2728
*/
28-
class ToInlineStyleEmailConverter {
29-
29+
class ToInlineStyleEmailConverter
30+
{
3031
/**
3132
* Container is used to get template engine instead of direct injection.
3233
* Direct injection is not used to avoid circular reference exception when rendering using twig tag

DependencyInjection/Configuration.php

Lines changed: 0 additions & 29 deletions
This file was deleted.

DependencyInjection/RobertoTruToInlineStyleEmailExtension.php

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,28 @@
11
<?php
22

3+
/*
4+
* This file is part of ToInlineStyleEmailBundle.
5+
*
6+
* (c) Roberto Trunfio <[email protected]>
7+
*
8+
* For the full copyright and license information, please view the LICENSE
9+
* file that was distributed with this source code.
10+
*/
11+
312
namespace RobertoTru\ToInlineStyleEmailBundle\DependencyInjection;
413

514
use Symfony\Component\DependencyInjection\ContainerBuilder;
615
use Symfony\Component\Config\FileLocator;
716
use Symfony\Component\HttpKernel\DependencyInjection\Extension;
817
use Symfony\Component\DependencyInjection\Loader;
918

10-
/**
11-
* This is the class that loads and manages your bundle configuration
12-
*
13-
* To learn more see {@link http://symfony.com/doc/current/cookbook/bundles/extension.html}
14-
*/
1519
class RobertoTruToInlineStyleEmailExtension extends Extension
1620
{
1721
/**
1822
* {@inheritDoc}
1923
*/
2024
public function load(array $configs, ContainerBuilder $container)
2125
{
22-
$configuration = new Configuration();
23-
$config = $this->processConfiguration($configuration, $configs);
24-
2526
$loader = new Loader\XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
2627
$loader->load('services.xml');
2728
}

README.md

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,18 +4,17 @@ ToInlineStyleEmailBundle
44
**ToInlineStyleEmailBundle** is a _Symfony2_ bundle to use the **CssToInlineStyles** translator by _Tijs Verkoyen_ (see
55
https://github.com/tijsverkoyen/CssToInlineStyles for the repository)
66

7-
87
Requirements
9-
===========
8+
============
109
**ToInlineStyleEmailBundle** is only supported on **PHP 5.3.3** and up.
1110

1211
Installation
13-
===========
12+
============
1413
Please, use the _Composer_ to install this bundle in your Symfony2 app.
1514

1615
The following lines should be added in your ```composer.json```
1716

18-
```
17+
``` json
1918
"require": {
2019
"robertotru/to-inline-style-email-bundle": "dev-master"
2120
},
@@ -28,7 +27,7 @@ new RobertoTru\ToInlineStyleEmailBundle\RobertoTruToInlineStyleEmailBundle(),
2827
```
2928

3029
Documentation and Examples
31-
===========
30+
==========================
3231
The bundle provides a service named **css_to_inline_email_converter**. Use it in a controller to have a nice shortcut to the
3332
converter developed by _Tijs Verkoyen_. E.g.:
3433

@@ -91,10 +90,10 @@ Paths relative to bundle are supported as well:
9190
Read the docs in the files for further details on the usage of the service.
9291

9392
Contributing
94-
===========
93+
============
9594
**ToInlineStyleEmailBundle** is an open source project. Contributions are encouraged.
9695
Feel free to contribute to improve this bundle.
9796

9897
About the author of the bundle
99-
===========
100-
**ToInlineStyleEmailBundle** has been originally developed and is mantained by Roberto Trunfio (see more on the author at www.trunfio.it)
98+
==============================
99+
**ToInlineStyleEmailBundle** has been originally developed and is maintained by Roberto Trunfio (see more on the author at www.trunfio.it)

Resources/doc/index.rst

Whitespace-only changes.
Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,22 @@
11
<?php
22

3+
/*
4+
* This file is part of ToInlineStyleEmailBundle.
5+
*
6+
* (c) Roberto Trunfio <[email protected]>
7+
*
8+
* For the full copyright and license information, please view the LICENSE
9+
* file that was distributed with this source code.
10+
*/
11+
312
namespace RobertoTru\ToInlineStyleEmailBundle\Tests;
413

514
use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
615

7-
class ToInlineStyleEmailBundleTest extends WebTestCase {
8-
public function testIndex() {
9-
10-
//$client = static::createClient();
11-
12-
//$crawler = $client->request('GET', '/hello/Fabien');
16+
class ToInlineStyleEmailBundleTest extends WebTestCase
17+
{
18+
public function testIndex()
19+
{
1320

14-
//$this->assertTrue($crawler->filter('html:contains("Hello Fabien")')->count() > 0);
1521
}
1622
}

0 commit comments

Comments
 (0)