QT/InvoiceShelf
1
0
Fork 0
mirror of https://github.com/InvoiceShelf/InvoiceShelf.git synced 2026-08-12 11:10:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

Laravel 11 (#84)

Browse Source 
* Convert string references to `::class`

PHP 5.5.9 adds the new static `class` property which provides the fully qualified class name. This is preferred over using strings for class names since the `class` property references are checked by PHP.

* Use Faker methods

Accessing Faker properties was deprecated in Faker 1.14.

* Convert route options to fluent methods

Laravel 8 adopts the tuple syntax for controller actions. Since the old options array is incompatible with this syntax, Shift converted them to use modern, fluent methods.

* Adopt class based routes

* Remove default `app` files

* Shift core files

* Streamline config files

* Set new `ENV` variables

* Default new `bootstrap/app.php`

* Re-register HTTP middleware

* Consolidate service providers

* Re-register service providers

* Re-register routes

* Re-register scheduled commands

* Bump Composer dependencies

* Use `<env>` tags for configuration

`<env>` tags have a lower precedence than system environment variables making it easier to overwrite PHPUnit configuration values in additional environments, such a CI.

Review this blog post for more details on configuration precedence when testing Laravel: https://jasonmccreary.me/articles/laravel-testing-configuration-precedence/

* Adopt anonymous migrations

* Rename `password_resets` table

* Convert `$casts` property to method

* Adopt Laravel type hints

* Mark base controller as `abstract`

* Remove `CreatesApplication` testing trait

* Shift cleanup

* Fix shift first issues

* Updating Rules for laravel 11, sanctum config and pint

* Fix Carbon issue on dashboard

* Temporary fix for tests while migration is issue fixed on laravel side

* Carbon needs numerical values, not strings

* Minimum php version

* Fix domain installation step not fetching the correct company_id

* Fix Role Policy wasn't properly registered

---------
This commit is contained in:
mchev
2024-06-05 11:33:52 +02:00
committed by GitHub
parent 72311db1bd
commit 3259173066
656 changed files with 4964 additions and 7944 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/Policies/ExpenseCategoryPolicy.php
+11 -11
View File
@@ -1,11 +1,11 @@
<?php
namespace InvoiceShelf\Policies;
namespace App\Policies;
use App\Models\Expense;
use App\Models\ExpenseCategory;
use App\Models\User;
use Illuminate\Auth\Access\HandlesAuthorization;
use InvoiceShelf\Models\Expense;
use InvoiceShelf\Models\ExpenseCategory;
use InvoiceShelf\Models\User;
use Silber\Bouncer\BouncerFacade;
class ExpenseCategoryPolicy
@@ -17,7 +17,7 @@ class ExpenseCategoryPolicy
*
* @return mixed
*/
public function viewAny(User $user)
public function viewAny(User $user): bool
{
if (BouncerFacade::can('view-expense', Expense::class)) {
return true;
@@ -31,7 +31,7 @@ class ExpenseCategoryPolicy
*
* @return mixed
*/
public function view(User $user, ExpenseCategory $expenseCategory)
public function view(User $user, ExpenseCategory $expenseCategory): bool
{
if (BouncerFacade::can('view-expense', Expense::class) && $user->hasCompany($expenseCategory->company_id)) {
return true;
@@ -45,7 +45,7 @@ class ExpenseCategoryPolicy
*
* @return mixed
*/
public function create(User $user)
public function create(User $user): bool
{
if (BouncerFacade::can('view-expense', Expense::class)) {
return true;
@@ -59,7 +59,7 @@ class ExpenseCategoryPolicy
*
* @return mixed
*/
public function update(User $user, ExpenseCategory $expenseCategory)
public function update(User $user, ExpenseCategory $expenseCategory): bool
{
if (BouncerFacade::can('view-expense', Expense::class) && $user->hasCompany($expenseCategory->company_id)) {
return true;
@@ -73,7 +73,7 @@ class ExpenseCategoryPolicy
*
* @return mixed
*/
public function delete(User $user, ExpenseCategory $expenseCategory)
public function delete(User $user, ExpenseCategory $expenseCategory): bool
{
if (BouncerFacade::can('view-expense', Expense::class) && $user->hasCompany($expenseCategory->company_id)) {
return true;
@@ -87,7 +87,7 @@ class ExpenseCategoryPolicy
*
* @return mixed
*/
public function restore(User $user, ExpenseCategory $expenseCategory)
public function restore(User $user, ExpenseCategory $expenseCategory): bool
{
if (BouncerFacade::can('view-expense', Expense::class) && $user->hasCompany($expenseCategory->company_id)) {
return true;
@@ -101,7 +101,7 @@ class ExpenseCategoryPolicy
*
* @return mixed
*/
public function forceDelete(User $user, ExpenseCategory $expenseCategory)
public function forceDelete(User $user, ExpenseCategory $expenseCategory): bool
{
if (BouncerFacade::can('view-expense', Expense::class) && $user->hasCompany($expenseCategory->company_id)) {
return true;