Receiving duplicated values in checkboxes

[Arkokeando]

Hi! I'm new working with web dev and I need help with this:
When I change any checkbox value and I send it, I receive both value, in this case 0 and 1 then, I cannot send the real value to the database, can someone give me a tip with it?

Example: I have 5 product and I want to change the 2 first from 0 to 1 the value of $pre_orderable-checkbox, I will get in request :

#parameters: array:4 [▼
     "pre_orderable-checkbox" => array:10 [▼
       0 => "0"
       1 => "1"
       2 => "0"
       3 => "1"
       4 => "0"
       5 => "0"
       6 => "0"
     ]

Instead of this :

#parameters: array:4 [▼
      "pre_orderable-checkbox" => array:10 [▼
        0 => "1"
        1 => "1"
        2 => "0"
        3 => "0"
        4 => "0"
      ]

So it means that The first , third and fifth will be now 0 and the second and the fourth will be 1.

This is the class where "magic happens"

<?php

namespace App\Orchid\Screens;

use App\Models\Dropshipper;
use App\Models\Product;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Str;
use Orchid\Screen\Action;
use Orchid\Screen\Actions\Button;
use Orchid\Screen\Actions\ModalToggle;
use Orchid\Screen\Fields\CheckBox;
use Orchid\Screen\Fields\Input;
use Orchid\Screen\Fields\Select;
use Orchid\Screen\Repository;
use Orchid\Screen\Screen;
use Orchid\Screen\TD;
use Orchid\Support\Color;
use Orchid\Support\Facades\Layout;
use Orchid\Support\Facades\Toast;

class ProductsScreen extends Screen
{
    /**
     * Query data.
     *
     * @return array
     */
    public function query(): iterable
    {
        $products_list=DB::table('products')->get();
        $data= array();
        foreach ($products_list as $product){
            $data[] = new Repository([
                'id' => $product->id,
                'name' => $product->name,
                'sku' => $product->sku,
                'quantity' => $product->quantity,
                'dropshipper_id' => $product->dropshipper_id,
                'barcode' => $product->barcode,
                'shop_id' => $product->shop_id,
                'pre_orderable' => $product->pre_orderable,
                'inventory_management_shopify' => $product->inventory_management_shopify
            ]);
        }

        return [
            'table' => $data
        ];
    }

    /**
     * Display header name.
     *
     * @return string|null
     */
    public function name(): ?string
    {
        return 'Products Screen';
    }

    public function description(): ?string
    {
        return 'Products information';
    }

    /**
     * Button commands.
     *
     * @return Action[]
     */
    public function commandBar(): iterable
    {
        return [
            Button::make('Edit Products')
                ->method('saveProductChanges')
                ->type(Color::SUCCESS()),
        ];
    }

    /**
     * Views.
     *
     * @return \Orchid\Screen\Layout[]|string[]
     */
    public function layout(): iterable
    {
        return [
            Layout::modal('changeDropshipperModal', Layout::rows([
                Select::make('select')
                    ->title('Dropshipper')
                    ->empty('')
                    ->fromModel(Dropshipper::class , 'name'),
            ]))->title('Change Dropshipper')
                ->async('asyncGetProduct'),
            Layout::table('table', [
                TD::make('name', 'Name')
                    ->width('150')
                    ->render(fn (Repository $model) => Str::limit($model->get('name'), 200)),
                TD::make('pre_orderable','Pre Orderable')
                    ->render(fn (Repository $model) => CheckBox::make('pre_orderable-checkbox[]')

                        ->value($model->get('pre_orderable'))),
                TD::make('inventory_management_shopify','Inventory (shopify)')
                    ->render(fn (Repository $model) => CheckBox::make('inventory_management_shopify-checkbox[]')

                        ->value($model->get('inventory_management_shopify'))),
                TD::make('id','Id')
                    ->render(fn (Repository $model) => Input::make('productIds[]')
                        ->value($model->get('id'))->hidden()),
                TD::make('sku', 'SKU')
                    ->render(fn (Repository $model) => Str::limit($model->get('sku'), 200)),
                TD::make('barcode', 'Barcode')
                    ->render(fn (Repository $model) => Str::limit($model->get('barcode'), 200)),
                TD::make('quantity', 'Quantity')
                    ->render(fn (Repository $model) => Str::limit($model->get('quantity'), 200)),
                TD::make('shop_id', 'Shop ID')
                    ->render(fn (Repository $model) => Str::limit($model->get('shop_id'), 200)),
                TD::make('dropshipper', 'Dropshipper')
                    ->render(fn (Repository $model) => ModalToggle::make($this->getDropshipperName($model))
                        ->modal('changeDropshipperModal')
                        ->method('changeDropshipper')
                        ->asyncParameters(['id' => $model->get('id')])
                        ->type(Color::DEFAULT())
                        ->icon('refresh')),
            ]),

        ];
    }

    /**
     * @param Request $request
     * @param Product $product
     * @return void
     */
    public function changeDropshipper(Request $request, Product $product): void
    {
        $request->validate(['select' => 'required']);
        $selected_dropshipper=['dropshipper_id'=>$request->input('select')] ;
        $product->fill($selected_dropshipper)->save();

        Toast::info(__('Dropshipper changed successfully.'));
    }


    public function saveProductChanges(Request $request)
    {
        $productIds = $request->input('productIds');
        $preOrderable = $request->input('pre_orderable-checkbox');
        $inventoryManagementShopify = $request->input('inventory_management_shopify-checkbox');

        for ($i = 0; $i < count($productIds); $i++) {
            
            Product::where('id', $productIds[$i])
                ->update([
                    'pre_orderable' => $preOrderable[$i],
                    'inventory_management_shopify' => $inventoryManagementShopify[$i]
                ]);
        }

        Toast::success('Product changes saved successfully!');

    }

    /**
     * @param Repository $model
     * @return string
     */
    public function getDropshipperName(Repository $model): string
    {
        $dropshipper_info=Dropshipper::query()->findOrFail($model->get('dropshipper_id'));

        return $dropshipper_info['name'];
    }

    /**
     * @param Product $model
     * @return iterable
     */
    public function asyncGetProduct(Product $model): iterable
    {
        return ['model' => $model];
    }

}

[tabuna]

Change the field name to be an array of predefined keys:

TD::make('pre_orderable','Pre Orderable')
->render(fn (Repository $model) => CheckBox::make("pre_orderable-checkbox[$model->id]")
    ->sendTrueOrFalse()
    ->value($model->get('pre_orderable'))),

[Arkokeando]

Hi, thanks for the reply!
I try this but I still get the same in the request:

"pre_orderable-checkbox" => array:8 [▼
        0 => "0"
        1 => "1"
        2 => "0"
        3 => "1"
        4 => "0"
        5 => "1"
        6 => "0"
        7 => "0"
      ]
      "inventory_management_shopify-checkbox" => array:8 [▼
        0 => "0"
        1 => "1"
        2 => "0"
        3 => "1"
        4 => "0"
        5 => "1"
        6 => "0"
        7 => "0"
      ]
      "productIds" => array:5 [▼
        0 => "2"
        1 => "3"
        2 => "4"
        3 => "5"
        4 => "6"

It is sending the "old" value and the new one, I mean, when you change from 0 to 1 (uncheck to check), I get both values, but in the other way, if I have all checked and I want to change it to uncheck (from 1 to 0) I receive this:

"pre_orderable-checkbox" => array:5 [▼
        0 => "0"
        1 => "0"
        2 => "0"
        3 => "0"
        4 => "0"
      ]
      "inventory_management_shopify-checkbox" => array:5 [▼
        0 => "0"
        1 => "0"
        2 => "0"
        3 => "0"
        4 => "0"
      ]
      "productIds" => array:5 [▼
        0 => "2"
        1 => "3"
        2 => "4"
        3 => "5"
        4 => "6"
      ]

So, the problem is that I'm only receiving both values when I change have a true (check).
(I was also using ->sendTrueOrFalse() method before too but when I edited the post I don't know why I deleted it )

[Arkokeando]

Finally I created a function that manage this behavior from the request. I post it here for maybe help others facing the same problem in future

function filter_array(array $array_to_filter): array {
        $final_array = [];
        for ($i = 0; $i < count($array_to_filter); $i++) {
            if ($i +1 < count($array_to_filter)){
                if ($array_to_filter[$i] == 1) {
                    $final_array[] = $array_to_filter[$i];
                } else if ($array_to_filter[$i] == 0 && $array_to_filter[$i +1] == 0) {
                    $final_array[] = $array_to_filter[$i];
                }
            }else {
                $final_array[] = $array_to_filter[$i];
            }
        }
        return $final_array;
    }