Call to a member function related() on null

@Sinnbeck Its going to this url: http://localhost:8000/admin/products/0 It should redirect back to the same page: http://localhost:8000/admin/products

I tried

$product = Product::findOrFail($id);
        dd($product);

But its giving page not found

@Sharim 0? I dont think you have product with id = 0 :)

@Sinnbeck I think id = 0 in bulk delete. That is a controller of voyager admin panel. There is 2 different conditions from single item and bulk items like this:

$ids = [];
        if (empty($id)) {
            // Bulk delete, get IDs from POST
            $ids = explode(',', $request->ids);
        } else {
            // Single item delete, get ID from URL
            $ids[] = $id;
        }
        foreach ($ids as $id) {
            $data = call_user_func([$dataType->model_name, 'findOrFail'], $id);

            // Check permission
            $this->authorize('delete', $data);

            $model = app($dataType->model_name);
            if (!($model && in_array(SoftDeletes::class, class_uses_recursive($model)))) {
                $this->cleanup($dataType, $data);
            }
        }

@Sharim But isnt this your code?

//delete related items
        $product = Product::where('id', $id)->first();
        $product->related()->sync($request->related_id);

If that is supposed to bulk delete you need to get multiple items (all I assume?)

@Sinnbeck Yes this code is working on single item delete only but not in bulk delete.

@Sharim Yes. You are doing the bulk logic after you have gotten the product.. Maybe you meant to put it after?

$products = Product::whereIn('id', $ids)->get();
$products->each->related()->sync($request->related_id);

@Sinnbeck You mentioned two different variables $products & $product. It'll work ?

@Sharim Typo. Fixed

@Sinnbeck Getting this error:

Method Illuminate\Database\Eloquent\Collection::sync does not exist.

@Sharim Did you foget the ->each?

@Sinnbeck I put this code after foreach:

$products = Product::whereIn('id', $ids)->get();
$products->each->related()->sync($request->related_id);

@Sharim Can you show the full code ?

@Sinnbeck here is the full method sir:

public function destroy(Request $request, $id)
    {
        $slug = $this->getSlug($request);

        $dataType = Voyager::model('DataType')->where('slug', '=', $slug)->first();
        
        // Init array of IDs
        $ids = [];
        if (empty($id)) {
            // Bulk delete, get IDs from POST
            $ids = explode(',', $request->ids);
        } else {
            // Single item delete, get ID from URL
            $ids[] = $id;
        }
        foreach ($ids as $id) {
            $data = call_user_func([$dataType->model_name, 'findOrFail'], $id);

            // Check permission
            $this->authorize('delete', $data);

            $model = app($dataType->model_name);
            if (!($model && in_array(SoftDeletes::class, class_uses_recursive($model)))) {
                $this->cleanup($dataType, $data);
            }
        }

        $products = Product::whereIn('id', $ids)->get();
        $products->each->related()->sync($request->related_id);

        $displayName = count($ids) > 1 ? $dataType->getTranslatedAttribute('display_name_plural') : $dataType->getTranslatedAttribute('display_name_singular');

        $res = $data->destroy($ids);
        
        $data = $res
            ? [
                'message'    => __('voyager::generic.successfully_deleted')." {$displayName}",
                'alert-type' => 'success',
            ]
            : [
                'message'    => __('voyager::generic.error_deleting')." {$displayName}",
                'alert-type' => 'error',
            ];

        if ($res) {
            event(new BreadDataDeleted($dataType, $data));
        }

        return redirect()->back()->with($data);
    }

@Sharim Weird that should work! I dont understand why you are getting that error. Can you try the old way?

$products = Product::whereIn('id', $ids)->get();
foreach ($products as $product) {
    $product->related()->sync($request->related_id);
}

@Sinnbeck That worked :)