Merge branch 'add_export_to_filter' of github.com:/connortechnology/ZoneMinder into add_export_to_filter

web/ajax/event.php

@@ -39,7 +39,8 @@ if ( canView('Events') ) {
       {
         require_once(ZM_SKIN_PATH.'/includes/export_functions.php');
 
-# We use session vars in here, so we need to restart the session because we stopped it in index.php to improve concurrency.
+        # We use session vars in here, so we need to restart the session
+        # because we stopped it in index.php to improve concurrency.
         session_start();
 
         if ( !empty($_REQUEST['exportDetail']) )
@@ -72,6 +73,11 @@ if ( canView('Events') ) {
         else
           $exportFormat = '';
 
+        if ( !empty($_REQUEST['exportCompress']) )
+          $exportCompress = $_SESSION['export']['compress'] = $_REQUEST['exportCompress'];
+        else
+          $exportCompress = false;
+
         session_write_close();
 
         $exportIds = !empty($_REQUEST['eids'])?$_REQUEST['eids']:$_REQUEST['id'];
@@ -83,7 +89,8 @@ if ( canView('Events') ) {
           $exportImages,
           $exportVideo,
           $exportMisc,
-          $exportFormat
+          $exportFormat,
+          $exportCompress
         ) )
           ajaxResponse(array('exportFile'=>$exportFile));
         else
@@ -100,7 +107,8 @@ if ( canView('Events') ) {
         if ( $exportFile = exportEvents(
           $exportIds,
           (isset($_REQUEST['connkey'])?$_REQUEST['connkey']:''),
-          false,false, false, $exportVideo, false, $exportFormat, $exportStructure ) )
+          false,false, false,
+          $exportVideo, false, $exportFormat, $exportStructure ) )
           ajaxResponse(array('exportFile'=>$exportFile));
         else
           ajaxError('Export Failed');

web/lang/en_gb.php

@@ -323,6 +323,7 @@ $SLANG = array(
     'Events'                => 'Events',
     'Exclude'               => 'Exclude',
     'Execute'               => 'Execute',
+    'ExportCompress'        => 'Use Compression',
     'ExportDetails'         => 'Export Event Details',
     'ExportMatches'         => 'Export Matches',
     'Exif'                  => 'Embed EXIF data into image',

web/skins/classic/includes/export_functions.php

@@ -826,6 +826,7 @@ function exportEvents(
   $exportVideo,
   $exportMisc,
   $exportFormat,
+  $exportCompressed,
   $exportStructure = false
 ) {
 
@@ -912,28 +913,30 @@ function exportEvents(
   chdir(ZM_DIR_EXPORTS);
   $archive = '';
   if ( $exportFormat == 'tar' ) {
-    $archive = ZM_DIR_EXPORTS.'/'.$export_root.($connkey?'_'.$connkey:'').'.tar.gz';
+    $archive = ZM_DIR_EXPORTS.'/'.$export_root.($connkey?'_'.$connkey:'').'.tar';
-    @unlink($archive);
+    $command = 'tar --create --dereference';
-    $command = 'nice -10 tar --create --gzip --dereference --file='.escapeshellarg($archive).' zmExport_'.$connkey.'/';
+    if ( $exportCompressed ) {
-    #$command = 'nice -10 tar --create --gzip --file='.escapeshellarg($archive).' --files-from='.escapeshellarg($listFile);
+      $archive .= '.gz';
+      $command .= ' --gzip';
+      $exportFormat .= '.gz';
+    }
     if ( $exportStructure == 'flat' ) {
       //strip file paths if we 
       $command .= " --xform='s#^.+/##x'";
     }
+    $command .= ' --file='.escapeshellarg($archive);
   } elseif ( $exportFormat == 'zip' ) {
     $archive = ZM_DIR_EXPORTS.'/'.$export_root.($connkey?'_'.$connkey:'').'.zip';
-    @unlink($archive);
+    $command = 'zip ';
-    if ( $exportStructure == 'flat' ) {
+    $command = ($exportStructure == 'flat' ? ' -j ' : ' -r ' ).escapeshellarg($archive);
-      $command = 'nice -10 zip -j '.escapeshellarg($archive).' zmExport_'.$connkey.'/';
+    $command .= $exportCompressed ? ' -9' : ' -0';
-      #$command = 'cat '.escapeshellarg($listFile).' | nice -10 zip -q -j '.escapeshellarg($archive).' -@';
-    } else {
-      $command = 'nice -10 zip -r '.escapeshellarg($archive).' zmExport_' . $connkey.'/';
-      #$command = 'cat '.escapeshellarg($listFile).' | nice -10 zip -q '.escapeshellarg($archive).' -@';
-    }
   } else {
     Error("No exportFormat specified.");
     return false;
   } // if $exportFormat
+
+  @unlink($archive);
+  $command .= ' zmExport_' . $connkey.'/';
   Logger::Debug("Command is $command");
   exec($command, $output, $status);
   if ( $status ) {

web/skins/classic/views/export.php

@@ -36,12 +36,15 @@ if ( isset($_SESSION['export']) ) {
     $_REQUEST['exportMisc'] = $_SESSION['export']['misc'];
   if ( isset($_SESSION['export']['format']) )
     $_REQUEST['exportFormat'] = $_SESSION['export']['format'];
+  if ( isset($_SESSION['export']['compress']) )
+    $_REQUEST['exportCompress'] = $_SESSION['export']['compress'];
 } else {
   $_REQUEST['exportDetail'] =
   $_REQUEST['exportFrames'] =
   $_REQUEST['exportImages'] =
   $_REQUEST['exportVideo'] =
   $_REQUEST['exportMisc'] = 1;
+  $_REQUEST['exportCompress'] = 0;
 }
 
 $focusWindow = true;
@@ -74,11 +77,11 @@ if ( $user['MonitorIds'] ) {
 }
 
 if ( isset($_REQUEST['eid']) and $_REQUEST['eid'] ) {
-  Logger::Debug("Loading events by single eid");
+  Logger::Debug('Loading events by single eid');
   $eventsSql .= ' AND E.Id=?';
   $eventsValues[] = $_REQUEST['eid'];
 } elseif ( isset($_REQUEST['eids']) and count($_REQUEST['eids']) > 0 ) {
-  Logger::Debug("Loading events by eids");
+  Logger::Debug('Loading events by eids');
   $eventsSql .= ' AND E.Id IN ('.implode(',', array_map(function(){return '?';}, $_REQUEST['eids'])). ')';
   $eventsValues += $_REQUEST['eids'];
 } else if ( isset($_REQUEST['filter']) ) {
@@ -185,6 +188,15 @@ while ( $event_row = dbFetchNext($results) ) {
                 <input type="radio" id="exportFormatZip" name="exportFormat" value="zip"<?php if ( isset($_REQUEST['exportFormat']) && $_REQUEST['exportFormat'] == "zip" ) { ?> checked="checked"<?php } ?> onclick="configureExportButton( this )"/><label for="exportFormatZip"><?php echo translate('ExportFormatZip') ?></label>
               </td>
             </tr>
+            <tr>
+              <th scope="row"><?php echo translate('ExportCompress') ?></th>
+              <td>
+                <input type="radio" id="exportCompress1" name="exportCompress" value="1"<?php echo ( isset($_REQUEST['exportCompress']) && $_REQUEST['exportCompress'] ) ? ' checked="checked"' : '' ?> onclick="configureExportButton(this)"/>
+                <label for="exportCompress1"><?php echo translate('Yes') ?></label>
+                <input type="radio" id="exportCompress0" name="exportCompress" value="0"<?php echo ( isset($_REQUEST['exportCompress']) && $_REQUEST['exportCompress'] ) ? '' : ' checked="checked"' ?> onclick="configureExportButton(this)"/>
+                <label for="exportCompress0"><?php echo translate('No') ?></label>
+              </td>
+            </tr>
           </tbody>
         </table>
         <button type="button" id="exportButton" name="exportButton" value="Export" onclick="exportEvent(this.form);" disabled="disabled"><?php echo translate('Export') ?></button>

web/views/archive.php

@@ -28,10 +28,14 @@ $connkey = isset($_REQUEST['connkey'])?$_REQUEST['connkey']:'';
 
 if ( $archivetype ) {
   switch ($archivetype) {
-  case 'tar':
+  case 'tar.gz':
     $mimetype = 'gzip';
     $file_ext = 'tar.gz';
     break;
+  case 'tar':
+    $mimetype = 'tar';
+    $file_ext = 'tar';
+    break;
   case 'zip':
     $mimetype = 'zip';
     $file_ext = 'zip';