Fix for email form plugin when attching as PDF. Need to reload the fo…

…rm's data, so it's in getData() format for display.

plugins/fabrik_element/fileupload/fileupload.php

@@ -269,7 +269,7 @@ public function elementJavascript($repeatCounter)
 
 		$singleCrop = false;
 
-		if (array_key_exists('params', $value))
+		if (is_array($value) && array_key_exists('params', $value))
 		{
 			$singleCrop = true;
 			$imgParams = (array) FArrayHelper::getValue($value, 'params');

plugins/fabrik_form/email/email.php

@@ -436,6 +436,7 @@ protected function pdfAttachment(&$thisAttachments)
 
 		/** @var FabrikFEModelForm $model */
 		$model = $this->getModel();
+		$model->setRowId($this->data['rowid']);
 		$document = JFactory::getDocument();
 		$docType = $document->getType();
 		$document->setType('pdf');
@@ -481,8 +482,13 @@ protected function pdfAttachment(&$thisAttachments)
 			 * here instead of poking in to the _model, but I don't think there is a setModel for controllers?
 			 */
 			$controller->_model = $model;
-			$controller->_model->data = $this->getProcessData();
-			//$controller->_model->data = null;
+
+			/**
+			 * Unfortunately, we need to reload the data, so it's in the right format.  Can't use the
+			 * submitted data.  "One of these days" we need to have a serious look at normalizing the data formats,
+			 * so submitted data is in the same format (once processed) as data read from the database.
+			 */
+			$controller->_model->data = $this->model->getData();
 
 			// Store in output buffer
 			ob_start();