From 453120e940c7af48fa7213773585db804ce4343c Mon Sep 17 00:00:00 2001
From: Damyon Wiese <damyon@moodle.com>
Date: Wed, 15 May 2019 13:46:56 +0800
Subject: [PATCH] MDL-65610 lti: ltiversion field optional

The ltiversion field only exists for some preconfigured tool types.
---
 mod/lti/edit_form.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/mod/lti/edit_form.php b/mod/lti/edit_form.php
index a06170c973e..edf22de615e 100644
--- a/mod/lti/edit_form.php
+++ b/mod/lti/edit_form.php
@@ -352,7 +352,8 @@ class mod_lti_edit_types_form extends moodleform {
 
         $errors = parent::validation($data, $files);
 
-        if ($data['lti_ltiversion'] == LTI_VERSION_1P3) {
+        // LTI2 tools do not contain a ltiversion field.
+        if (isset($data['lti_ltiversion']) && $data['lti_ltiversion'] == LTI_VERSION_1P3) {
             require_once($CFG->dirroot . '/mod/lti/upgradelib.php');
 
             $warning = mod_lti_verify_private_key();