github-mirror/framework
2
0
Fork 0
mirror of https://github.com/flarum/framework.git synced 2024-11-25 09:41:49 +08:00
Code Issues Actions 46 Packages Projects Releases Wiki Activity

Throw HTTP 403 on extension validation error

Browse Source 
The way I read it, HTTP 405 is a generic statement about the
resource. Once a language pack is not the default, this is not
true anymore, so I figured 403 is more correct.
This commit is contained in:
Franz Liedke 2017-02-03 20:25:21 +01:00
parent 9349ed13fc
commit d3753d94ae
No known key found for this signature in database
GPG Key ID: 9A0231A879B055F4
2 changed files with 25 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/Core/Listener/ExtensionValidator.php
View File

@ -12,7 +12,7 @@
namespace Flarum\Core\Listener;
use Flarum\Event\ExtensionWillBeDisabled;
use Flarum\Http\Exception\MethodNotAllowedException;
use Flarum\Http\Exception\ForbiddenException;
use Illuminate\Contracts\Events\Dispatcher;
class ExtensionValidator
@ -27,7 +27,7 @@ class ExtensionValidator
/**
* @param ExtensionWillBeDisabled $event
* @throws MethodNotAllowedException
* @throws ForbiddenException
*/
public function whenExtensionWillBeDisabled(ExtensionWillBeDisabled $event)
{
@ -35,7 +35,7 @@ class ExtensionValidator
$default_locale = $this->app->make('flarum.settings')->get('default_locale');
$locale = array_get($event->extension->extra, 'flarum-locale.code');
if ($locale === $default_locale) {
throw new MethodNotAllowedException('You cannot disable the default language pack!');
throw new ForbiddenException('You cannot disable the default language pack!');
}
}
}

22
src/Http/Exception/ForbiddenException.php Normal file
View File

@ -0,0 +1,22 @@
<?php
/*
* This file is part of Flarum.
*
* (c) Toby Zerner <toby.zerner@gmail.com>
*
* For the full copyright and license information, please view the LICENSE
* file that was distributed with this source code.
*/
namespace Flarum\Http\Exception;
use Exception;
class ForbiddenException extends Exception
{
public function __construct($message = null, $code = 403, Exception $previous = null)
{
parent::__construct($message, $code, $previous);
}
}