From 653804373d9ef83f83daeaaa766de0fb86c1a34d Mon Sep 17 00:00:00 2001
From: root <root@ilias5-dev.ph-freiburg.de>
Date: Mon, 18 Nov 2019 14:59:49 +0100
Subject: [PATCH] hotfix: column 'virtual' is a reserved word in mysql >=5.7
 and leads to database exception. The hotfix replaces all ->manipulate calls
 with corresponding ->insert/update calls which quotes column identifier
 automatically

---
 .../Glossary/classes/class.ilObjGlossary.php  | 55 +++++++++++--------
 1 file changed, 31 insertions(+), 24 deletions(-)

diff --git a/Modules/Glossary/classes/class.ilObjGlossary.php b/Modules/Glossary/classes/class.ilObjGlossary.php
index 55b91ed9bfd2..8158e5a39738 100755
--- a/Modules/Glossary/classes/class.ilObjGlossary.php
+++ b/Modules/Glossary/classes/class.ilObjGlossary.php
@@ -69,14 +69,16 @@ function create($a_upload = false)
 		{
 			$this->createMetaData();
 		}
+		$this->db->insert('glossary',
+			array(
+				'id' => array('integer', $this->getId()),
+				'is_online' => array('text', 'n'),
+				'virtual' => array('text', $this->getVirtualMode()),
+				'pres_mode' => array('text', 'table'),
+				'snippet_length' => array('integer', 200)
+			)
+		);
 
-		$this->db->manipulate("INSERT INTO glossary (id, is_online, virtual, pres_mode, snippet_length) VALUES (".
-			$this->db->quote($this->getId(), "integer").",".
-			$this->db->quote("n", "text").",".
-			$this->db->quote($this->getVirtualMode(), "text").",".
-			$this->db->quote("table", "text").",".
-			$this->db->quote(200, "integer").
-			")");
 		$this->setPresentationMode("table");
 		$this->setSnippetLength(200);
 
@@ -419,18 +421,22 @@ function update()
 	{
 		$this->updateMetaData();
 
-		$this->db->manipulate("UPDATE glossary SET ".
-			" is_online = ".$this->db->quote(ilUtil::tf2yn($this->getOnline()), "text").",".
-			" virtual = ".$this->db->quote($this->getVirtualMode(), "text").",".
-			" public_xml_file = ".$this->db->quote($this->getPublicExportFile("xml"), "text").",".
-			" public_html_file = ".$this->db->quote($this->getPublicExportFile("html"), "text").",".
-			" glo_menu_active = ".$this->db->quote(ilUtil::tf2yn($this->isActiveGlossaryMenu()), "text").",".
-			" downloads_active = ".$this->db->quote(ilUtil::tf2yn($this->isActiveDownloads()), "text").", ".
-			" pres_mode = ".$this->db->quote($this->getPresentationMode(), "text").", ".
-			" show_tax = ".$this->db->quote((int) $this->getShowTaxonomy(), "integer").", ".
-			" snippet_length = ".$this->db->quote((int)$this->getSnippetLength(), "integer")." ".
-			" WHERE id = ".$this->db->quote($this->getId(), "integer"));
-		
+		$this->db->update('glossary', 
+			array(
+				'is_online' => array('text', ilUtil::tf2yn($this->getOnline())),
+				'virtual' => array('text', $this->getVirtualMode()),
+				'public_xml_file' => array('text', $this->getPublicExportFile("xml")),
+				'public_html_file' => array('text', $this->getPublicExportFile("html")),
+				'glo_menu_active' => array('text', ilUtil::tf2yn($this->isActiveGlossaryMenu())),
+				'downloads_active' => array('text', ilUtil::tf2yn($this->isActiveDownloads())),
+				'pres_mode' => array('text', $this->getPresentationMode()),
+				'show_tax' => array('integer', $this->getShowTaxonomy()),
+				'snippet_length' => array('integer', $this->getSnippetLength())
+			),
+			array(
+				'id' => array('integer', $this->getId())
+			)
+		);
 		include_once("./Services/Style/Content/classes/class.ilObjStyleSheet.php");
 		ilObjStyleSheet::writeStyleUsage($this->getId(), $this->getStyleSheetId());
 
@@ -453,11 +459,12 @@ function updateAutoGlossaries()
 		);
 		foreach ($this->getAutoGlossaries() as $glo_id)
 		{
-			$this->db->manipulate("INSERT INTO glo_glossaries ".
-				"(id, glo_id) VALUES (".
-				$this->db->quote($this->getId(), "integer").",".
-				$this->db->quote($glo_id, "integer").
-				")");
+			$this->db->insert('glo_glossaries',
+				array(
+					'id' => array('integer', $this->getId()),
+					'glo_id' => array('integer', $glo_id)
+				)
+			);
 		}
 	}