added support for newer php versions and small improvement on renderer

Author: Ricardo Fiorani

.travis.yml

@@ -8,6 +8,8 @@ php:
   - 5.5
   - 5.6
   - 7.0
+  - 7.1
+  - 7.2
 
 matrix:
   include:

src/Renderer/DefaultRenderer.php

@@ -1,10 +1,8 @@
 <?php
 /**
- * Created by PhpStorm.
- * User: Ricardo Fiorani
- * Date: 30/08/2015
- * Time: 00:55.
+ * @author Ricardo Fiorani
  */
+
 namespace RicardoFiorani\Renderer;
 
 class DefaultRenderer implements EmbedRendererInterface
@@ -13,18 +11,22 @@ class DefaultRenderer implements EmbedRendererInterface
      * A friend once told me that html inside strings
      * isn't a good thing, but god knows I trying
      * to make it not dependant of some html element generator
-     * and too lazy to make one of these on my own.
-     *
-     *
+     * and I'm too lazy/unnecessary to make one of these on my own.
+     * Edit: The best I could do is do an sprintf, so...
      *
      * @param string $embedUrl
-     * @param int    $width
-     * @param int    $height
+     * @param int $width
+     * @param int $height
      *
      * @return string
      */
     public function renderVideoEmbedCode($embedUrl, $width, $height)
     {
-        return '<iframe width="'.$width.'" height="'.$height.'" src="'.addslashes($embedUrl).'" frameborder="0" webkitAllowFullScreen mozallowfullscreen allowFullScreen></iframe>';
+        return sprintf(
+            '<iframe width="%s" height="%s" src="%s" frameborder="0" webkitAllowFullScreen mozallowfullscreen allowFullScreen></iframe>',
+            $width,
+            $height,
+            addslashes($embedUrl)
+        );
     }
 }