Working on scheduled reports

app/Console/Commands/SendReminders.php

@@ -76,11 +76,6 @@ class SendReminders extends Command
 
         $this->info('Done');
 
-        if (\Utils::isNinjaDev()) {
-            $this->info('Stopping early on ninja dev');
-            exit;
-        }
-
         if ($errorEmail = env('ERROR_EMAIL')) {
             \Mail::raw('EOM', function ($message) use ($errorEmail, $database) {
                 $message->to($errorEmail)

app/Http/Controllers/ReportController.php

@@ -11,6 +11,7 @@ use Input;
 use Utils;
 use View;
 use Carbon;
+use Validator;
 
 /**
  * Class ReportController.
@@ -133,6 +134,14 @@ class ReportController extends BaseController
 
     private function schedule($params, $options)
     {
+        $validator = Validator::make(request()->all(), [
+            'frequency' => 'required|in:daily,weekly,biweekly,monthly',
+            'send_date' => 'required',
+        ]);
+
+        if ($validator->fails()) {
+            session()->now('message', trans('texts.scheduled_report_error'));
+        } else {
             $options['report_type'] = $params['reportType'];
             $options['range'] = request('range');
             $options['start_date_offset'] = $options['range'] ? '' : Carbon::parse($params['startDate'])->diffInDays(null, false); // null,false to get the relative/non-absolute diff
@@ -150,6 +159,7 @@ class ReportController extends BaseController
 
             session()->now('message', trans('texts.created_scheduled_report'));
         }
+    }
 
     private function cancelSchdule()
     {

app/Jobs/ExportReportResults.php

@@ -99,6 +99,7 @@ class ExportReportResults extends Job
                 $sheet->setAutoSize(true);
             });
 
+            if (count($summary)) {
                 $excel->sheet(trans("texts.totals"), function($sheet) use($report, $summary, $format) {
                     $sheet->setOrientation('landscape');
                     $sheet->freezeFirstRow();
@@ -118,6 +119,7 @@ class ExportReportResults extends Job
                     });
                     $sheet->setAutoSize(true);
                 });
+            }
 
         });
     }

app/Ninja/Reports/ActivityReport.php

@@ -18,8 +18,8 @@ class ActivityReport extends AbstractReport
     {
         $account = Auth::user()->account;
 
-        $startDate = $this->startDate->format('Y-m-d');
-        $endDate = $this->endDate->format('Y-m-d');
+        $startDate = $this->startDate;;
+        $endDate = $this->endDate;
 
         $activities = Activity::scope()
             ->with('client.contacts', 'user', 'invoice', 'payment', 'credit', 'task', 'expense', 'account')
@@ -32,7 +32,7 @@ class ActivityReport extends AbstractReport
                 $activity->present()->createdAt,
                 $client ? ($this->isExport ? $client->getDisplayName() : $client->present()->link) : '',
                 $activity->present()->user,
-                $activity->getMessage(),
+                $this->isExport ? strip_tags($activity->getMessage()) : $activity->getMessage(),
             ];
         }
 

resources/lang/en/texts.php

@@ -2553,7 +2553,7 @@ $LANG = array(
     'created_scheduled_report' => 'Successfully scheduled report',
     'deleted_scheduled_report' => 'Successfully canceled scheduled report',
     'scheduled_report_attached' => 'Your scheduled :type report is attached.',
-
+    'scheduled_report_error' => 'Failed to create schedule report',
 );
 
 return $LANG;

resources/views/reports/report_builder.blade.php

@@ -101,10 +101,11 @@
 
     {!! Former::open()->addClass('report-form')->rules(['start_date' => 'required', 'end_date' => 'required']) !!}
 
+
     <div style="display:none">
-		{!! Former::text('action') !!}
-		{!! Former::text('range') !!}
-		{!! Former::text('scheduled_report_id') !!}
+		{!! Former::text('action')->forceValue('') !!}
+		{!! Former::text('range')->forceValue('') !!}
+		{!! Former::text('scheduled_report_id')->forceValue('') !!}
     </div>
 
     {!! Former::populateField('start_date', $startDate) !!}
@@ -212,7 +213,7 @@
 				->appendIcon(Icon::create('remove')) !!}
 
 		{!! Button::primary(trans('texts.schedule'))
-				->withAttributes(['id'=>'scheduleButton', 'onclick' => 'showScheduleModal()'])
+				->withAttributes(['id'=>'scheduleButton', 'onclick' => 'showScheduleModal()', 'style' => 'display:none'])
 				->appendIcon(Icon::create('time')) !!}
 
 	 	</span> &nbsp;&nbsp;
@@ -433,6 +434,13 @@
             }
         });
 
+		$('#format').change(function() {
+			var val = $('#format').val();
+            if (isStorageSupported() && val != 'zip') {
+                localStorage.setItem('last:report_format', val);
+            }
+        });
+
         $('#report_type').change(function() {
 			var val = $('#report_type').val();
 			setFiltersShown();
@@ -498,6 +506,12 @@
 				widgets: ['zebra', 'uitheme'],
 			}).show();
 
+			setFiltersShown();
+			setDocumentZipShown();
+			setTimeout(function() {
+				setScheduleButton();
+			}, 1);
+
 			if (isStorageSupported()) {
 				var lastReportType = localStorage.getItem('last:report_type');
 				if (lastReportType) {
@@ -507,11 +521,13 @@
 				if (lastDocumentFilter) {
 					$('#document_filter').val(lastDocumentFilter);
 				}
+				var lastFormat = localStorage.getItem('last:report_format');
+				if (lastFormat) {
+					setTimeout(function() {
+						$('#format').val(lastFormat);
+					}, 1);
+				}
 			}
-
-			setFiltersShown();
-			setDocumentZipShown();
-			setScheduleButton();
 		});
     })