Add PHP-CS-Fixer to linting process and Travis.

This commit is contained in:
michaelachrisco 2017-03-31 11:28:20 -07:00
parent 07453dad8e
commit 5a43121592
5 changed files with 67 additions and 44 deletions

1
.gitignore vendored
View file

@ -3,3 +3,4 @@ composer.phar
composer.lock
.DS_Store
/.idea
.php_cs.cache

View file

@ -5,7 +5,6 @@ php:
- 5.5
- 5.6
- 7.1
- hhvm
env:
global:
@ -22,4 +21,6 @@ install:
- if [[ $setup = 'stable' ]]; then travis_retry composer update --prefer-source --no-interaction --prefer-stable; fi
- if [[ $setup = 'lowest' ]]; then travis_retry composer update --prefer-source --no-interaction --prefer-lowest --prefer-stable; fi
script: ./vendor/bin/kahlan -reporter=verbose
script:
- ./vendor/bin/kahlan -reporter=verbose
- ./vendor/friendsofphp/php-cs-fixer/php-cs-fixer fix src/

View file

@ -15,6 +15,7 @@
},
"require-dev": {
"kahlan/kahlan": "^2.4",
"illuminate/database": ">=5.2.0"
"illuminate/database": ">=5.2.0",
"friendsofphp/php-cs-fixer": "^2.2"
}
}

View file

@ -1,5 +1,6 @@
<?php
namespace MichaelAChrisco\ReadOnly;
use RuntimeException;
class ReadOnlyException extends RuntimeException{}
class ReadOnlyException extends \RuntimeException
{
}

View file

@ -1,13 +1,15 @@
<?php
namespace MichaelAChrisco\ReadOnly;
use Illuminate\Database\Eloquent\Builder;
use MichaelAChrisco\ReadOnly\ReadOnlyException;
trait ReadOnlyTrait {
static function readOnly($class){
throw new ReadOnlyException("Not allowed to persist changes in read-only model {$class}");
}
trait ReadOnlyTrait
{
public static function readOnly($class)
{
throw new ReadOnlyException("Not allowed to persist changes in read-only model {$class}");
}
/**
* throws ReadOnlyException on create
@ -15,8 +17,9 @@ trait ReadOnlyTrait {
* @param array $attributes
*
*/
static function create(array $attributes = []){
ReadOnlyTrait::readOnly(get_called_class());
public static function create(array $attributes = [])
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -25,8 +28,9 @@ trait ReadOnlyTrait {
* @param array $attributes
*
*/
static function forceCreate(array $attributes){
ReadOnlyTrait::readOnly(get_called_class());
public static function forceCreate(array $attributes)
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -35,8 +39,9 @@ trait ReadOnlyTrait {
* @param array $options
*
*/
public function save(array $options = []){
ReadOnlyTrait::readOnly(get_called_class());
public function save(array $options = [])
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -46,8 +51,9 @@ trait ReadOnlyTrait {
* @param $options
*
*/
public function update(array $attributes = [], array $options = []){
ReadOnlyTrait::readOnly(get_called_class());
public function update(array $attributes = [], array $options = [])
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -56,8 +62,9 @@ trait ReadOnlyTrait {
* @param array $arr
*
*/
static function firstOrCreate(array $arr){
ReadOnlyTrait::readOnly(get_called_class());
public static function firstOrCreate(array $arr)
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -66,8 +73,9 @@ trait ReadOnlyTrait {
* @param array $arr
*
*/
static function firstOrNew(array $arr){
ReadOnlyTrait::readOnly(get_called_class());
public static function firstOrNew(array $arr)
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -75,8 +83,9 @@ trait ReadOnlyTrait {
* @method delete
*
*/
public function delete(){
ReadOnlyTrait::readOnly(get_called_class());
public function delete()
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -85,8 +94,9 @@ trait ReadOnlyTrait {
* @param mixed $ids
*
*/
static function destroy($ids){
ReadOnlyTrait::readOnly(get_called_class());
public static function destroy($ids)
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -94,8 +104,9 @@ trait ReadOnlyTrait {
* @method restore
*
*/
public function restore(){
ReadOnlyTrait::readOnly(get_called_class());
public function restore()
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -103,8 +114,9 @@ trait ReadOnlyTrait {
* @method forceDelete
*
*/
public function forceDelete(){
ReadOnlyTrait::readOnly(get_called_class());
public function forceDelete()
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -112,8 +124,9 @@ trait ReadOnlyTrait {
* @method performDeleteOnModel
*
*/
public function performDeleteOnModel(){
ReadOnlyTrait::readOnly(get_called_class());
public function performDeleteOnModel()
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -121,8 +134,9 @@ trait ReadOnlyTrait {
* @method push
*
*/
public function push(){
ReadOnlyTrait::readOnly(get_called_class());
public function push()
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -130,8 +144,9 @@ trait ReadOnlyTrait {
* @method finishSave
*
*/
public function finishSave(array $options){
ReadOnlyTrait::readOnly(get_called_class());
public function finishSave(array $options)
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -139,8 +154,9 @@ trait ReadOnlyTrait {
* @method performUpdate
*
*/
public function performUpdate(Builder $query, array $options = []){
ReadOnlyTrait::readOnly(get_called_class());
public function performUpdate(Builder $query, array $options = [])
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -148,8 +164,9 @@ trait ReadOnlyTrait {
* @method touch
*
*/
public function touch(){
ReadOnlyTrait::readOnly(get_called_class());
public function touch()
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -157,8 +174,9 @@ trait ReadOnlyTrait {
* @method insert
*
*/
public function insert(){
ReadOnlyTrait::readOnly(get_called_class());
public function insert()
{
ReadOnlyTrait::readOnly(get_called_class());
}
/**
@ -166,7 +184,8 @@ trait ReadOnlyTrait {
* @method truncate
*
*/
public function truncate(){
ReadOnlyTrait::readOnly(get_called_class());
public function truncate()
{
ReadOnlyTrait::readOnly(get_called_class());
}
}