Merge tag '0.7.2-beta' into develop

Add an uglyUrls option that disables URL rewriting (fixes #88)
youtube-dl update
This commit is contained in:
Pierre Rudloff 2017-01-10 23:41:37 +01:00
commit 764110ac49
6 changed files with 21 additions and 17 deletions

View file

@ -77,7 +77,8 @@ class Config
/**
* Disable URL rewriting.
* @var boolean
*
* @var bool
*/
public $uglyUrls = false;

View file

@ -2,10 +2,11 @@
/**
* UglyRouter class.
*/
namespace Alltube;
use Slim\Router;
use Psr\Http\Message\ServerRequestInterface;
use Slim\Router;
/**
* Extend Slim's router class in order to disable URL rewriting.
@ -13,7 +14,7 @@ use Psr\Http\Message\ServerRequestInterface;
class UglyRouter extends Router
{
/**
* Dispatch router for HTTP request
* Dispatch router for HTTP request.
*
* @param ServerRequestInterface $request The current HTTP request object
*
@ -36,23 +37,23 @@ class UglyRouter extends Router
}
/**
* Build the path for a named route including the base path
* Build the path for a named route including the base path.
*
* @param string $name Route name
* @param array $data Named argument replacement data
* @param array $queryParams Optional query string parameters
*
* @return string
*
* @throws \RuntimeException If named route does not exist
* @throws \InvalidArgumentException If required data not provided
*
* @return string
*/
public function pathFor($name, array $data = [], array $queryParams = [])
{
$url = str_replace('/', '/?page=', $this->relativePathFor($name, $data, $queryParams));
if ($this->basePath) {
$url = $this->basePath . $url;
$url = $this->basePath.$url;
}
return $url;

View file

@ -1,8 +1,8 @@
<?php
require_once __DIR__.'/vendor/autoload.php';
use Alltube\Controller\FrontController;
use Alltube\Config;
use Alltube\Controller\FrontController;
use Alltube\UglyRouter;
if (isset($_SERVER['REQUEST_URI']) && strpos($_SERVER['REQUEST_URI'], '/index.php') !== false) {

View file

@ -1,7 +1,7 @@
{
"name": "alltube",
"description": "HTML GUI for youtube-dl",
"version": "0.7.1",
"version": "0.7.2-beta",
"author": "Pierre Rudloff",
"bugs": "https://github.com/Rudloff/alltube/issues",
"dependencies": {

View file

@ -6,11 +6,11 @@
namespace Alltube\Test;
use Alltube\UglyRouter;
use Slim\Http\Request;
use Slim\Http\Uri;
use Slim\Http\Headers;
use Slim\Http\Environment;
use Slim\Http\Headers;
use Slim\Http\Request;
use Slim\Http\Stream;
use Slim\Http\Uri;
/**
* Unit tests for the UglyRouter class.
@ -35,6 +35,7 @@ class UglyRouterTest extends \PHPUnit_Framework_TestCase
/**
* Test the dispatch() function.
*
* @return void
*/
public function testDispatch()
@ -56,6 +57,7 @@ class UglyRouterTest extends \PHPUnit_Framework_TestCase
/**
* Test the pathFor() function.
*
* @return void
*/
public function testPathFor()
@ -68,11 +70,11 @@ class UglyRouterTest extends \PHPUnit_Framework_TestCase
/**
* Test the pathFor() function with a base path.
*
* @return void
*/
public function testPathForWithBasePath()
{
$this->router->setBasePath('/bar');
$this->assertEquals(
'/bar/?page=foo',