Skip to content

Commit c5bb010

Browse files
authored
Merge pull request #174 from yajra/ferror
fix: export package integration error message
2 parents 9ecfab9 + 9d053db commit c5bb010

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/Services/DataTable.php

+7-2
Original file line numberDiff line numberDiff line change
@@ -489,7 +489,7 @@ protected function buildExcelFile()
489489
}
490490

491491
if (! class_exists(ExcelServiceProvider::class)) {
492-
throw new Exception('Please install maatwebsite/excel to be able to use this function.');
492+
throw new Exception('Please `composer require maatwebsite/excel` to be able to use this function.');
493493
}
494494

495495
if (! new $this->exportClass instanceof DataTablesExportHandler) {
@@ -634,7 +634,7 @@ public function pdf()
634634
public function snappyPdf(): Response
635635
{
636636
if (! class_exists(PdfWrapper::class)) {
637-
throw new Exception('You need to install barryvdh/laravel-snappy to be able to use this feature.');
637+
throw new Exception('Please `composer require barryvdh/laravel-snappy` to be able to use this feature.');
638638
}
639639

640640
/** @var \Barryvdh\Snappy\PdfWrapper $snappy */
@@ -783,9 +783,14 @@ public function fastExcelCallback(): Closure
783783

784784
/**
785785
* @return \Rap2hpoutre\FastExcel\FastExcel
786+
* @throws \Yajra\DataTables\Exceptions\Exception
786787
*/
787788
protected function buildFastExcelFile(): FastExcel
788789
{
790+
if (! class_exists(FastExcel::class)) {
791+
throw new Exception('Please `composer require rap2hpoutre/fast-excel` to be able to use this function.');
792+
}
793+
789794
$query = null;
790795
if (method_exists($this, 'query')) {
791796
/** @var EloquentBuilder|QueryBuilder $query */

0 commit comments

Comments
 (0)