Filament•15mo ago•

3 replies

Best practice for multiple dynamic properties

Looking for advice for the following scenario:

Select::make('product_id')
  ->options(Product::all()->pluck('name', 'id')),
TextInput::make('quantity')
  ->numeric()
  ->step(1)
  ->maxValue(function(Get $get) {
    $productId = $get('product_id');
    if (!$productId) {
      return;
    }

    return Product::find($productId)->quanity;
  })
  ->helperText(function(Get $get {
    // Basically the same logic as maxValue
    return "Available: $stock";
  }),

Select::make('product_id')
  ->options(Product::all()->pluck('name', 'id')),
TextInput::make('quantity')
  ->numeric()
  ->step(1)
  ->maxValue(function(Get $get) {
    $productId = $get('product_id');
    if (!$productId) {
      return;
    }

    return Product::find($productId)->quanity;
  })
  ->helperText(function(Get $get {
    // Basically the same logic as maxValue
    return "Available: $stock";
  }),

My form also has several places where I will be using the

Product

Product

data. Do I really need to execute the

Product::find($productId)

Product::find($productId)

on every

dynamic

dynamic

property?

Example:
After the

TextInput::make('quantity')

TextInput::make('quantity')

->live(onBlur: true)

->live(onBlur: true)

I have another field that will show the

Total

Total

which is

quantity * $product->price

quantity * $product->price

. Calling

Product::find...

Product::find...

for the 3rd time seems a bit much.

is there a more efficient approach that I might be not considering, cause calling the

Product

Product

model every time doesn't seem efficient.

Thanks in advance for the advice

Best practice for multiple dynamic properties

Looking for advice for the following scenario:

Select::make('product_id')
  ->options(Product::all()->pluck('name', 'id')),
TextInput::make('quantity')
  ->numeric()
  ->step(1)
  ->maxValue(function(Get $get) {
    $productId = $get('product_id');
    if (!$productId) {
      return;
    }

    return Product::find($productId)->quanity;
  })
  ->helperText(function(Get $get {
    // Basically the same logic as maxValue
    return "Available: $stock";
  }),

Select::make('product_id')
  ->options(Product::all()->pluck('name', 'id')),
TextInput::make('quantity')
  ->numeric()
  ->step(1)
  ->maxValue(function(Get $get) {
    $productId = $get('product_id');
    if (!$productId) {
      return;
    }

    return Product::find($productId)->quanity;
  })
  ->helperText(function(Get $get {
    // Basically the same logic as maxValue
    return "Available: $stock";
  }),

My form also has several places where I will be using the

Product

Product

data. Do I really need to execute the

Product::find($productId)

Product::find($productId)

on every

dynamic

dynamic

property?

Example:
After the

TextInput::make('quantity')

TextInput::make('quantity')

->live(onBlur: true)

->live(onBlur: true)

I have another field that will show the

Total

Total

which is

quantity * $product->price

quantity * $product->price

. Calling

Product::find...

Product::find...

for the 3rd time seems a bit much.

is there a more efficient approach that I might be not considering, cause calling the

Product

Product

model every time doesn't seem efficient.

Thanks in advance for the advice

Best practice for multiple dynamic properties

Similar Threads

Best practice for multiple dynamic properties

Similar Threads

Similar Threads

Similar Threads