From f5f6c9b11070045ec1b802dd10c283af8af2792d Mon Sep 17 00:00:00 2001
From: srosse <none@none>
Date: Wed, 13 Dec 2017 08:41:48 +0100
Subject: [PATCH] OO-3199: reload the certificates data before every preview

---
 .../certificate/ui/CertificatesOptionsController.java       | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/olat/course/certificate/ui/CertificatesOptionsController.java b/src/main/java/org/olat/course/certificate/ui/CertificatesOptionsController.java
index fb53665191f..f688a99b24e 100644
--- a/src/main/java/org/olat/course/certificate/ui/CertificatesOptionsController.java
+++ b/src/main/java/org/olat/course/certificate/ui/CertificatesOptionsController.java
@@ -304,8 +304,7 @@ public class CertificatesOptionsController extends FormBasicController {
 			}
 		} else if(source == certificateChooserCtrl) {
 			if(event == Event.DONE_EVENT) {
-				CertificateTemplate template = certificateChooserCtrl.getSelectedTemplate();
-				doSetTemplate(template);
+				doSetTemplate(certificateChooserCtrl.getSelectedTemplate());
 			}
 			cmc.deactivate();
 			cleanUp();
@@ -336,6 +335,7 @@ public class CertificatesOptionsController extends FormBasicController {
 	}
 	
 	private void doPreviewTemplate(UserRequest ureq) {
+		selectedTemplate = certificatesManager.getTemplateById(selectedTemplate.getKey());
 		File preview = certificatesManager.previewCertificate(selectedTemplate, entry, getLocale());
 		MediaResource resource = new PreviewMediaResource(preview);
 		ureq.getDispatchResult().setResultingMediaResource(resource);
@@ -345,8 +345,10 @@ public class CertificatesOptionsController extends FormBasicController {
 		this.selectedTemplate = template;
 		if(selectedTemplate == null) {
 			templateCont.contextPut("templateName", translate("default.template"));
+			previewTemplateLink.setEnabled(false);
 		} else {
 			templateCont.contextPut("templateName", template.getName());
+			previewTemplateLink.setEnabled(true);
 		}
 	}
 	
-- 
GitLab