Skip to content

Commit f8e3c0c

Browse files
committed
fix event names
1 parent d73d5f2 commit f8e3c0c

File tree

8 files changed

+18
-15
lines changed

8 files changed

+18
-15
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@
22

33
All Notable changes to `laravel-medialibrary` will be documented in this file
44

5+
##3.10.1
6+
- Fixed the event names to make them more readable `CollectionHasBeenCleared`, `ConversionHasBeenCompleted`, `MediaHasBeenAdded`
7+
58
##3.10.0
69
- Added `CollectionClearedEvent`, `ConverionCompleteEvent`, `MediaAddedEvent`
710

src/Events/CollectionClearedEvent.php renamed to src/Events/CollectionHasBeenCleared.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
use Illuminate\Queue\SerializesModels;
66
use Spatie\MediaLibrary\HasMedia\Interfaces\HasMedia;
77

8-
class CollectionClearedEvent
8+
class CollectionHasBeenCleared
99
{
1010
use SerializesModels;
1111

src/Events/ConversionCompleteEvent.php renamed to src/Events/ConversionHasBeenCompleted.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
use Spatie\MediaLibrary\Conversion\Conversion;
77
use Spatie\MediaLibrary\Media;
88

9-
class ConversionCompleteEvent
9+
class ConversionHasBeenCompleted
1010
{
1111
use SerializesModels;
1212

src/Events/MediaAddedEvent.php renamed to src/Events/MediaHasBeenAdded.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
use Illuminate\Queue\SerializesModels;
66
use Spatie\MediaLibrary\Media;
77

8-
class MediaAddedEvent
8+
class MediaHasBeenAdded
99
{
1010
use SerializesModels;
1111

src/FileManipulator.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
use Spatie\MediaLibrary\Conversion\Conversion;
1010
use Spatie\MediaLibrary\Conversion\ConversionCollection;
1111
use Spatie\MediaLibrary\Conversion\ConversionCollectionFactory;
12-
use Spatie\MediaLibrary\Events\ConversionCompleteEvent;
12+
use Spatie\MediaLibrary\Events\ConversionHasBeenCompleted;
1313
use Spatie\MediaLibrary\Helpers\File as MediaLibraryFileHelper;
1414
use Spatie\MediaLibrary\Helpers\Gitignore;
1515
use Spatie\MediaLibrary\Jobs\PerformConversions;
@@ -81,7 +81,7 @@ public function performConversions(ConversionCollection $conversions, Media $med
8181

8282
app(Filesystem::class)->copyToMediaLibrary($renamedFile, $media, true);
8383

84-
$this->events->fire(new ConversionCompleteEvent($media, $conversion));
84+
$this->events->fire(new ConversionHasBeenCompleted($media, $conversion));
8585
}
8686

8787
File::deleteDirectory($tempDirectory);

src/Filesystem.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
use Illuminate\Contracts\Config\Repository as ConfigRepository;
66
use Illuminate\Contracts\Events\Dispatcher;
77
use Illuminate\Contracts\Filesystem\Factory;
8-
use Spatie\MediaLibrary\Events\MediaAddedEvent;
8+
use Spatie\MediaLibrary\Events\MediaHasBeenAdded;
99
use Spatie\MediaLibrary\Helpers\File;
1010
use Spatie\MediaLibrary\Helpers\Gitignore;
1111
use Spatie\MediaLibrary\PathGenerator\PathGeneratorFactory;
@@ -49,7 +49,7 @@ public function add($file, Media $media, $targetFileName = '')
4949
{
5050
$this->copyToMediaLibrary($file, $media, false, $targetFileName);
5151

52-
$this->events->fire(new MediaAddedEvent($media));
52+
$this->events->fire(new MediaHasBeenAdded($media));
5353

5454
app(FileManipulator::class)->createDerivedFiles($media);
5555
}

src/HasMedia/HasMediaTrait.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
use Illuminate\Contracts\Events\Dispatcher;
66
use Spatie\MediaLibrary\Conversion\Conversion;
7-
use Spatie\MediaLibrary\Events\CollectionClearedEvent;
7+
use Spatie\MediaLibrary\Events\CollectionHasBeenCleared;
88
use Spatie\MediaLibrary\Exceptions\MediaDoesNotBelongToModel;
99
use Spatie\MediaLibrary\Exceptions\MediaIsNotPartOfCollection;
1010
use Spatie\MediaLibrary\Exceptions\UrlCouldNotBeOpened;
@@ -237,7 +237,7 @@ public function clearMediaCollection($collectionName = 'default')
237237
$media->delete();
238238
});
239239

240-
app(Dispatcher::class)->fire(new CollectionClearedEvent($this, $collectionName));
240+
app(Dispatcher::class)->fire(new CollectionHasBeenCleared($this, $collectionName));
241241

242242
return $this;
243243
}

tests/Events/EventTest.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22

33
namespace Spatie\MediaLibrary\Test\Events;
44

5-
use Spatie\MediaLibrary\Events\CollectionClearedEvent;
6-
use Spatie\MediaLibrary\Events\ConversionCompleteEvent;
7-
use Spatie\MediaLibrary\Events\MediaAddedEvent;
5+
use Spatie\MediaLibrary\Events\CollectionHasBeenCleared;
6+
use Spatie\MediaLibrary\Events\ConversionHasBeenCompleted;
7+
use Spatie\MediaLibrary\Events\MediaHasBeenAdded;
88
use Spatie\MediaLibrary\Test\TestCase;
99

1010
class EventTest extends TestCase
@@ -14,7 +14,7 @@ class EventTest extends TestCase
1414
*/
1515
public function it_will_fire_the_media_added_event()
1616
{
17-
$this->expectsEvents([MediaAddedEvent::class]);
17+
$this->expectsEvents([MediaHasBeenAdded::class]);
1818

1919
$this->testModel->addMedia($this->getTestJpg())->toMediaLibrary();
2020
}
@@ -24,7 +24,7 @@ public function it_will_fire_the_media_added_event()
2424
*/
2525
public function it_will_fire_the_conversion_complete_event()
2626
{
27-
$this->expectsEvents([ConversionCompleteEvent::class]);
27+
$this->expectsEvents([ConversionHasBeenCompleted::class]);
2828

2929
$this->testModelWithConversion->addMedia($this->getTestJpg())->toCollection('images');
3030
}
@@ -39,7 +39,7 @@ public function it_will_fire_the_collection_cleared_event()
3939
->preservingOriginal()
4040
->toMediaLibrary('images');
4141

42-
$this->expectsEvents([CollectionClearedEvent::class]);
42+
$this->expectsEvents([CollectionHasBeenCleared::class]);
4343

4444
$this->testModel->clearMediaCollection('images');
4545
}

0 commit comments

Comments
 (0)