The laravel-config's discuss from tarfin-labs

Factories can be loaded from package since Laravel v6.12.0

Publishing factories is not necessery since Laravel v6.12.0. So they can be loaded from package like models.
#31133

publish the LaravelConfig class and the Model

hey i love this package but i want to be able to add traits to the Model such as SoftDeletes , auditing , and caching
i also want to be able to add/edit the functions used for adding/changing the keypairs wethere its updating the description by id or disable caching on some functions or get the audit results i think the LaravelConfig class would work better if it was a trait of the Model and the Model was publishable then it would be easy to add or change any of the functions

the new cast funchon brakes laravel-ide-helper ide-helper:models

i found that if you change src/Casts/ConfigValueCast.php line 12 and line 24 from
line 12:
public function get(Model $model, string $key, mixed $value, array $attributes)
to:
public function get($model, string $key, mixed $value, array $attributes)
and line 24 from:
public function set(Model $model, string $key, mixed $value, array $attributes)
to:
public function set($model, string $key, mixed $value, array $attributes)
it works fixing all the problems
this is the error

Symfony\Component\ErrorHandler\Error\FatalError 

  Declaration of TarfinLabs\LaravelConfig\Casts\ConfigValueCast::get(Illuminate\Database\Eloquent\Model $model, string $key, $value, array $attributes) must be compatible with Illuminate\Contracts\Database\Eloquent\CastsAttributes::get($model, string $key, $value, array $attributes)

  at vendor/tarfin-labs/laravel-config/src/Casts/ConfigValueCast.php:12
      8▕ use TarfinLabs\LaravelConfig\Enums\ConfigDataType;
      9▕ 
     10▕ class ConfigValueCast implements CastsAttributes
     11▕ {
  ➜  12▕     public function get(Model $model, string $key, $value, array $attributes)
     13▕     {
     14▕         return match ($attributes['type']) {
     15▕             ConfigDataType::BOOLEAN->value => (bool) $value,
     16▕             ConfigDataType::INTEGER->value => (int) $value,

      vendor frames
  5   artisan:0
      {main}()

   Whoops\Exception\ErrorException 

  Declaration of TarfinLabs\LaravelConfig\Casts\ConfigValueCast::get(Illuminate\Database\Eloquent\Model $model, string $key, $value, array $attributes) must be compatible with Illuminate\Contracts\Database\Eloquent\CastsAttributes::get($model, string $key, $value, array $attributes)

  at vendor/tarfin-labs/laravel-config/src/Casts/ConfigValueCast.php:12
      8▕ use TarfinLabs\LaravelConfig\Enums\ConfigDataType;
      9▕ 
     10▕ class ConfigValueCast implements CastsAttributes
     11▕ {
  ➜  12▕     public function get(Model $model, string $key, $value, array $attributes)
     13▕     {
     14▕         return match ($attributes['type']) {
     15▕             ConfigDataType::BOOLEAN->value => (bool) $value,
     16▕             ConfigDataType::INTEGER->value => (int) $value,

      vendor frames
  2   [internal]:0
      Whoops\Run::handleShutdown()

tarfin-labs / laravel-config Goto Github PK

laravel-config's Issues

Factories can be loaded from package since Laravel v6.12.0

publish the LaravelConfig class and the Model

the new cast funchon brakes laravel-ide-helper ide-helper:models

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent