Streamline one more exception for the bridges regarding option classes

Take for example the GoIpTransport, it contains the following lines:

symfony/src/Symfony/Component/Notifier/Bridge/GoIp/GoIpTransport.php

Lines 65 to 67 in c98b02a

    
           if (($options = $message->getOptions()) && !$options instanceof GoIpOptions) { 
        
               throw new LogicException(\sprintf('The "%s" transport only supports an instance of the "%s" as an option class.', __CLASS__, GoIpOptions::class)); 
        
           }

I would like to have a dedicated exception class UnsupportedOptionsException, basically the same as I did in

[Notifier] [DX] UnsupportedMessageTypeException for notifier transports #39365

This could be done for all bridges which already use options.

Note

It is important to also raise the notifier version to 7.3 in the composer.json files where this new exception is used.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	if (($options = $message->getOptions()) && !$options instanceof GoIpOptions) {
	throw new LogicException(\sprintf('The "%s" transport only supports an instance of the "%s" as an option class.', __CLASS__, GoIpOptions::class));
	}

Uh oh!

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions