Recently viewed session based

In the service provider where you register your composer add this to register method

But it probably doesn't cause that problem.

/**
 * Register the application services.
 *
 * @return void
 */
public function register()
{
    $this->app->singleton(InterestedComposer::class);
}

And I guess you have to check existense of that id in deals.recently_viewed, if it exists there you have to remove it and add it to the end.

@michaloravec Ok i tried it but the problem still exists, do you have an idea where the problem could be? Is it not because of the duplicate session query you mean? I don't understand what you mean with: ".. if it exists there you have to remove it and add it to the end."

So i added the check if deals exists in the session.

@if(session()->has('deals'))

And i added your code to the register method in the appserviceprovider. Can you explain me also where the app is referring to?

<?php

namespace App\Providers;

use App\Models\Category;
use App\Http\View\Composers\CategoryComposer;
use App\Http\View\Composers\CityComposer;
use App\Http\View\Composers\InterestedComposer;
use Illuminate\Support\Facades\View;
use Illuminate\Support\ServiceProvider;
use Illuminate\Support\Facades\Blade;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        $this->app->singleton(InterestedComposer::class);
    }

    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        \View::composer(['includes.categories.*'], CategoryComposer::class);
        \View::composer(['includes.cities.*'], CityComposer::class);
        \View::composer(['includes.interested.*'], InterestedComposer::class);

        Blade::directive('money', function ($amount) {
            return "<?php echo number_format($amount, 2, ',', '.'); ?>";
        });

    }
}

Don't use wildcard *, be more specific to which view you want to add it.

View::composer([
    'includes.interested.SOMETHING'
], InterestedComposer::class);

The code in the controller should be something like this

if (in_array($deal->getKey(), $deals = session()->get('deals.recently_viewed', []))) {
    $index = array_search($deal->getKey(), $deals);

    session()->pull("deals.recently_viewed.{$index}");
}

session()->push('deals.recently_viewed', $deal->getKey());

@michaloravec Ok, thanks man. I will look that up in de docs on how to do that.

@michaloravec Hmm i cant figure it out, i'm trying to do something but i can't get it right, could you give me another hint?

<?php

namespace App\Http\Controllers;

use App\Models\Deal;
use App\Filters\DealFilters;
use App\Models\Category;
use App\Models\City;
use App\Models\Address;
use Carbon\Carbon;
use Illuminate\Support\Str;
use Illuminate\Http\Request;

class DealController
{

    public function show(Request $request, $countrySlug, $citySlug, Category $category, Address $address, Deal $deal)
    {
        $city = City::where('country_slug', $countrySlug)->where('slug', $citySlug)->firstOrFail();

        if (in_array($deal->getKey(), $deals = session()->get('deals', []))) {
            $index = array_search($deal->getKey(), $deals);

            $request->session()->pull("deals.{$index}");
        }

         session()->push('deals', $deal);

        $mediaItems = $deal->getMedia('multi_collection');
        $imagesPerChunk = count($mediaItems);

        return view('deals.show', [
            'city' => $city,
            'category' => $category,
            'address' => $address,
            'deal' => $deal,
            'mediaItems' => $mediaItems,
            'imagesPerChunk' => $imagesPerChunk
        ]);

    }

}

$dealIds = collect(session()->get('deals', []))->pluck('id');

if ($dealIds->contains($deal->getKey()) && $index = $dealIds->search($deal->getKey())) {
    session()->pull("deals.{$index}");
}

session()->push('deals', $deal);

Ok i will