Support Laravel 5.4+

[cschalenborgh]

share() is removed in Laravel 5.4.
Here's the fix:

php artisan vendor:publish --provider="Padosoft\AffiliateNetwork\AffiliateNetworkServiceProvider"
   Symfony\Component\Debug\Exception\FatalThrowableError  : Call to undefined method Illuminate\Foundation\Application::share()

[Copilot]

Pull request overview

Updates the package’s Laravel service provider registration to be compatible with Laravel 5.4+, where Application::share() is no longer available.

Changes:

• Replaces the deprecated $this->app->share(...) binding with a $this->app->singleton(...) binding in the service provider.

Comments suppressed due to low confidence (1)

src/AffiliateNetworkServiceProvider.php:41

• The container binding here registers a singleton for Connection::class, but this class isn’t imported/defined in this package (so Connection::class resolves to Padosoft\AffiliateNetwork\Connection and will fatally error). It also doesn’t match the instance being returned (new NetworkManager()) or the subsequent alias/provides entries that reference NetworkManager. Bind the singleton under 'NetworkManager' (or NetworkManager::class) and align alias()/provides() to the same abstract.

        $this->app->singleton(Connection::class, function ($app) {
            return new NetworkManager();
        });
        $this->app->alias('NetworkManager', NetworkManager::class);
    }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.