From 476768a32db5240fc87f637d88e2154456317ea1 Mon Sep 17 00:00:00 2001 From: Karl Cordes Date: Mon, 1 Apr 2013 11:52:31 +1100 Subject: [PATCH] Fixed bug with adding attachments from the same principle. Closes #67 --- app/controllers/documents_controller.php | 2 +- app/views/documents/view.ctp | 4 +++- app/webroot/js/document_attachments.js | 2 +- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/app/controllers/documents_controller.php b/app/controllers/documents_controller.php index 66f55b34..e431a65e 100755 --- a/app/controllers/documents_controller.php +++ b/app/controllers/documents_controller.php @@ -118,7 +118,7 @@ class DocumentsController extends AppController { function saveAttachments() { $this->layout = 'ajax'; - if($this->data) { + if($this->data) { $attachments = array(); foreach($this->data['DocumentAttachment']['attachment_id'] as $i => $attachment_id) { $docID = $this->data['DocumentAttachment']['document_id']; diff --git a/app/views/documents/view.ctp b/app/views/documents/view.ctp index 0c261447..0c846bf4 100755 --- a/app/views/documents/view.ctp +++ b/app/views/documents/view.ctp @@ -46,7 +46,9 @@ echo $form->input("DocPage.content", array('class'=>'page', 'label'=>'Page', 'be $attachment) { ?> - + + input('DocumentAttachment.document_id', + array('type'=>'hidden', 'value'=> $document['Document']['id'])); ?> input("DocumentAttachment.{$index}.id", array('type'=>'checkbox', 'value'=> $attachment['DocumentAttachment']['id'], diff --git a/app/webroot/js/document_attachments.js b/app/webroot/js/document_attachments.js index 7fd8feeb..72f8b21e 100755 --- a/app/webroot/js/document_attachments.js +++ b/app/webroot/js/document_attachments.js @@ -15,7 +15,7 @@ $(function() { buttons: { "Add Attachment": function() { - var attachmentInputs = $('#DocumentAttachmentAddForm').find('input'); + var attachmentInputs = $('#DocumentAttachmentAddForm').find('input:checked, #DocumentAttachmentDocumentId'); $.post('/documents/saveAttachments', attachmentInputs, function(data) { $("#addAttachmentModal").dialog( "close" );