[Sluggable] Passing null to `strpos()` is deprecated

Docs to `Sluggable` [say](https://github.com/doctrine-extensions/DoctrineExtensions/blob/main/doc/sluggable.md#regenerating-slug):

> "In case if you want the slug to regenerate itself based on sluggable fields, set the slug to null."

However, this produces a deprecation under PHP >= 8.1:

```
Deprecated: strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in vendor/gedmo/doctrine-extensions/src/Sluggable/SluggableListener.php on line 368
```

https://github.com/doctrine-extensions/DoctrineExtensions/blob/eb53dfcb2b592327b76ac5226fbb003d32aea37e/src/Sluggable/SluggableListener.php#L367-L370

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

[Sluggable] Passing null to `strpos()` is deprecated #2995

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	// if slug should not be updated, skip it
	if (!$options['updatable'] && !$isInsert && (!isset($changeSet[$slugField]) \|\| 0 === strpos($slug, '__sluggable_placeholder__'))) {
	continue;
	}

Uh oh!

[Sluggable] Passing null to strpos() is deprecated #2995

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

[Sluggable] Passing null to `strpos()` is deprecated #2995