From b036f49c3a9cca5ef3044370ae7bec859cade03a Mon Sep 17 00:00:00 2001
From: "Carlos Alexandre S. da Fonseca" <bozohhot@hotmail.com>
Date: Mon, 20 Aug 2018 13:24:01 -0300
Subject: [PATCH] fix #206 - error on activity creation

---
 mod_form.php | 33 ++++++++++++++++++---------------
 1 file changed, 18 insertions(+), 15 deletions(-)

diff --git a/mod_form.php b/mod_form.php
index 0b7e7be..37b228a 100644
--- a/mod_form.php
+++ b/mod_form.php
@@ -48,13 +48,6 @@ public function definition() {
         // Design Options.
         $mform->addElement('header', 'designoptions', get_string('designoptions', 'simplecertificate'));
 
-        // Certificate image file.
-        $mform->addElement('filemanager', 'certificateimage',
-            get_string('certificateimage', 'simplecertificate'), null,
-            $this->get_filemanager_options_array()
-        );
-        $mform->addHelpButton('certificateimage', 'certificateimage', 'simplecertificate');
-
         // Certificate Text HTML editor.
         $mform->addElement('editor', 'certificatetext',
             get_string('certificatetext', 'simplecertificate'), null,
@@ -63,6 +56,13 @@ public function definition() {
         $mform->addRule('certificatetext', get_string('error'), 'required', null, 'client');
         $mform->addHelpButton('certificatetext', 'certificatetext', 'simplecertificate');
 
+        // Certificate image file.
+        $mform->addElement('filemanager', 'certificateimage',
+            get_string('certificateimage', 'simplecertificate'), null,
+            $this->get_filemanager_options_array()
+        );
+        $mform->addHelpButton('certificateimage', 'certificateimage', 'simplecertificate');
+
         // Certificate Width.
         $mform->addElement('text', 'width', get_string('width', 'simplecertificate'), array('size' => '5'));
         $mform->setType('width', PARAM_INT);
@@ -92,20 +92,15 @@ public function definition() {
         $mform->addHelpButton('certificatetexty', 'textposition', 'simplecertificate');
 
         // Second page.
-        $mform->addElement('header', 'secondpageoptions', get_string('secondpageoptions', 'simplecertificate'));
+        $mform->addElement('header', 'secondpageoptions', get_string(
+            'secondpageoptions', 'simplecertificate')
+        );
         // Enable back page text.
 
         $mform->addElement('selectyesno', 'enablesecondpage', get_string('enablesecondpage', 'simplecertificate'));
         $mform->setDefault('enablesecondpage', get_config('simplecertificate', 'enablesecondpage'));
         $mform->addHelpButton('enablesecondpage', 'enablesecondpage', 'simplecertificate');
 
-        // Certificate secondimage file.
-        $mform->addElement('filemanager', 'secondimage',
-            get_string('secondimage', 'simplecertificate'), null,
-            $this->get_filemanager_options_array());
-        $mform->addHelpButton('secondimage', 'secondimage', 'simplecertificate');
-        $mform->disabledIf('secondimage', 'enablesecondpage', 'eq', 0);
-
         // Certificate secondText HTML editor.
         $mform->addElement('editor', 'secondpagetext',
             get_string('secondpagetext', 'simplecertificate'), null,
@@ -113,6 +108,14 @@ public function definition() {
         $mform->addHelpButton('secondpagetext', 'certificatetext', 'simplecertificate');
         $mform->disabledIf('secondpagetext', 'enablesecondpage', 'eq', 0);
 
+        // Certificate secondimage file.
+        $mform->addElement('filemanager', 'secondimage',
+            get_string('secondimage', 'simplecertificate'), null,
+            $this->get_filemanager_options_array()
+        );
+        $mform->addHelpButton('secondimage', 'secondimage', 'simplecertificate');
+        // ...$mform->disabledIf('secondimage', 'enablesecondpage', 'eq', 0);.
+
         // Certificate Position X.
         $mform->addElement('text', 'secondpagex', get_string('secondpagex', 'simplecertificate'), array('size' => '5'));
         $mform->setType('secondpagex', PARAM_INT);