Index: lams_central/src/java/org/lamsfoundation/lams/web/qb/PrintQbQuestionController.java
===================================================================
diff -u -rc47ad82bb2279af8808f519dfc34a4a0d920d258 -rcb84acde58155494dc2cdf1bae82eac746dadfe2
--- lams_central/src/java/org/lamsfoundation/lams/web/qb/PrintQbQuestionController.java	(.../PrintQbQuestionController.java)	(revision c47ad82bb2279af8808f519dfc34a4a0d920d258)
+++ lams_central/src/java/org/lamsfoundation/lams/web/qb/PrintQbQuestionController.java	(.../PrintQbQuestionController.java)	(revision cb84acde58155494dc2cdf1bae82eac746dadfe2)
@@ -1,14 +1,21 @@
 package org.lamsfoundation.lams.web.qb;
 
+import org.lamsfoundation.lams.qb.model.QbCollection;
 import org.lamsfoundation.lams.qb.model.QbOption;
 import org.lamsfoundation.lams.qb.model.QbQuestion;
+import org.lamsfoundation.lams.qb.service.IQbService;
+import org.lamsfoundation.lams.security.ISecurityService;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Controller;
 import org.springframework.ui.Model;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RequestParam;
 import org.springframework.web.servlet.support.RequestContextUtils;
 
 import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
@@ -19,37 +26,60 @@
 @Controller
 @RequestMapping("/qb")
 public class PrintQbQuestionController {
+    @Autowired
+    IQbService qbService;
 
-    @GetMapping("/printQuestions.do")
+    @GetMapping("/printQuestions")
     public String printQuestions(HttpServletRequest request, Model model) {
 	// read Flash Attributes set by another module and put them in the model
 	Map<String, ?> inputFlashMap = RequestContextUtils.getInputFlashMap(request);
 	if (inputFlashMap != null) {
+	    fillModelWithQuestions(inputFlashMap, model);
+	}
+	return "qb/printQuestions";
+    }
 
-	    Map<Long, List<String>> randomisedOptions = new HashMap<>();
-	    model.addAttribute("printRandomisedOptions", randomisedOptions);
-	    for (String key : inputFlashMap.keySet()) {
-		if (key.startsWith("print")) {
-		    model.addAttribute(key, inputFlashMap.get(key));
+    @GetMapping("/printQbCollectionQuestions")
+    public String printQbCollectionQuestions(@RequestParam Long collectionUid, Model model,
+	    HttpServletResponse response) throws IOException {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
+	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
+	    return null;
+	}
 
-		    if (key.equals("printQuestions")) {
-			Collection<QbQuestion> questions = (Collection<QbQuestion>) inputFlashMap.get(key);
-			for (QbQuestion question : questions) {
-			    if (question.getType().equals(QbQuestion.TYPE_MATCHING_PAIRS) || question.getType()
-				    .equals(QbQuestion.TYPE_ORDERING)) {
-				// randomise the order of options when correct answers are hidden
-				List<String> optionNames = new ArrayList<>();
-				for (QbOption option : question.getQbOptions()) {
-				    optionNames.add(option.getName());
-				}
-				Collections.shuffle(optionNames);
-				randomisedOptions.put(question.getUid(), optionNames);
+	Map<String, Object> input = new HashMap<>();
+	Collection<QbQuestion> questions = qbService.getCollectionQuestions(collectionUid);
+	input.put("printQuestions", questions);
+	QbCollection collection = qbService.getCollectionByUid(collectionUid);
+	input.put("printTitleSuffix", collection.getName());
+
+	fillModelWithQuestions(input, model);
+	return "qb/printQuestions";
+    }
+
+    private static void fillModelWithQuestions(Map<String, ?> input, Model model) {
+	Map<Long, List<String>> randomisedOptions = new HashMap<>();
+	model.addAttribute("printRandomisedOptions", randomisedOptions);
+	for (String key : input.keySet()) {
+	    if (key.startsWith("print")) {
+		model.addAttribute(key, input.get(key));
+
+		if (key.equals("printQuestions")) {
+		    Collection<QbQuestion> questions = (Collection<QbQuestion>) input.get(key);
+		    for (QbQuestion question : questions) {
+			if (question.getType().equals(QbQuestion.TYPE_MATCHING_PAIRS) || question.getType()
+				.equals(QbQuestion.TYPE_ORDERING)) {
+			    // randomise the order of options when correct answers are hidden
+			    List<String> optionNames = new ArrayList<>();
+			    for (QbOption option : question.getQbOptions()) {
+				optionNames.add(option.getName());
 			    }
+			    Collections.shuffle(optionNames);
+			    randomisedOptions.put(question.getUid(), optionNames);
 			}
 		    }
 		}
 	    }
 	}
-	return "qb/printQuestions";
     }
 }
\ No newline at end of file
Index: lams_central/src/java/org/lamsfoundation/lams/web/qb/QbCollectionController.java
===================================================================
diff -u -r308be95be92e1e96e2be7e4d041ef7b92fcc7b37 -rcb84acde58155494dc2cdf1bae82eac746dadfe2
--- lams_central/src/java/org/lamsfoundation/lams/web/qb/QbCollectionController.java	(.../QbCollectionController.java)	(revision 308be95be92e1e96e2be7e4d041ef7b92fcc7b37)
+++ lams_central/src/java/org/lamsfoundation/lams/web/qb/QbCollectionController.java	(.../QbCollectionController.java)	(revision cb84acde58155494dc2cdf1bae82eac746dadfe2)
@@ -93,7 +93,7 @@
     @RequestMapping("/showOne")
     public String showOneCollection(@RequestParam long collectionUid, Model model, HttpServletResponse response)
 	    throws Exception {
-	if (!hasUserAccessToCollection(collectionUid)) {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return null;
 	}
@@ -112,7 +112,7 @@
     @ResponseBody
     public String getCollectionGridData(@RequestParam long collectionUid, @RequestParam String view,
 	    HttpServletRequest request, HttpServletResponse response) throws IOException {
-	if (!hasUserAccessToCollection(collectionUid)) {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return null;
 	}
@@ -148,7 +148,7 @@
     @ResponseBody
     public void removeCollectionQuestion(@RequestParam long collectionUid, @RequestParam int qbQuestionId,
 	    HttpServletResponse response) throws IOException {
-	if (!hasUserAccessToCollection(collectionUid)) {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return;
 	}
@@ -159,7 +159,7 @@
     @ResponseBody
     public String removeCollectionQuestions(@RequestParam long collectionUid,
 	    @RequestParam("qbQuestionIds[]") int[] qbQuestionIds, HttpServletResponse response) throws IOException {
-	if (!hasUserAccessToCollection(collectionUid)) {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return null;
 	}
@@ -179,7 +179,7 @@
 	if (!Configuration.getAsBoolean(ConfigurationKeys.QB_COLLECTIONS_TRANSFER_ALLOW)) {
 	    throw new SecurityException("Transfering questions between collections is disabled");
 	}
-	if (!hasUserAccessToCollection(targetCollectionUid)) {
+	if (!qbService.hasUserAccessToCollection(targetCollectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return;
 	}
@@ -212,7 +212,7 @@
     @ResponseBody
     public String changeCollectionName(@RequestParam(name = "pk") long collectionUid,
 	    @RequestParam(name = "value") String name, HttpServletResponse response) throws IOException {
-	if (!hasUserAccessToCollection(collectionUid)) {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return null;
 	}
@@ -236,7 +236,7 @@
     @RequestMapping(path = "/removeCollection", method = RequestMethod.POST)
     @ResponseBody
     public void removeCollection(@RequestParam long collectionUid, HttpServletResponse response) throws IOException {
-	if (!hasUserAccessToCollection(collectionUid)) {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return;
 	}
@@ -247,7 +247,7 @@
     @ResponseBody
     public void shareCollection(@RequestParam long collectionUid, @RequestParam int organisationId,
 	    HttpServletResponse response) throws IOException {
-	if (!hasUserAccessToCollection(collectionUid)) {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return;
 	}
@@ -258,7 +258,7 @@
     @ResponseBody
     public void unshareCollection(@RequestParam long collectionUid, @RequestParam int organisationId,
 	    HttpServletResponse response) throws IOException {
-	if (!hasUserAccessToCollection(collectionUid)) {
+	if (!qbService.hasUserAccessToCollection(collectionUid)) {
 	    response.sendError(HttpServletResponse.SC_FORBIDDEN, "The user does not have access to given collection");
 	    return;
 	}
@@ -330,16 +330,4 @@
 	UserDTO user = (UserDTO) ss.getAttribute(AttributeNames.USER);
 	return user != null ? user.getUserID() : null;
     }
-
-    private boolean hasUserAccessToCollection(long collectionUid) {
-	Integer userId = getUserId();
-	if (userId == null) {
-	    return false;
-	}
-	if (securityService.isAppadmin(userId, "access QB collection", true)) {
-	    return true;
-	}
-	Collection<QbCollection> collections = qbService.getUserCollections(userId);
-	return collections.stream().map(QbCollection::getUid).anyMatch(uid -> uid.equals(collectionUid));
-    }
 }
\ No newline at end of file
Index: lams_central/web/qb/collection.jsp
===================================================================
diff -u -r308be95be92e1e96e2be7e4d041ef7b92fcc7b37 -rcb84acde58155494dc2cdf1bae82eac746dadfe2
--- lams_central/web/qb/collection.jsp	(.../collection.jsp)	(revision 308be95be92e1e96e2be7e4d041ef7b92fcc7b37)
+++ lams_central/web/qb/collection.jsp	(.../collection.jsp)	(revision cb84acde58155494dc2cdf1bae82eac746dadfe2)
@@ -3,560 +3,571 @@
 <c:set var="hasQuestions" value="${questionCount > 0}" />
 
 <lams:html>
-<lams:head>
-	<title><fmt:message key="label.qb.collection" /></title>
-	<meta name="viewport" content="width=device-width, initial-scale=1">
-	
-	<lams:css/>
-	<link type="text/css" href="<lams:LAMSURL/>css/free.ui.jqgrid.min.css" rel="stylesheet">
-	<link type="text/css" href="<lams:LAMSURL/>css/thickbox.css" rel="stylesheet">
-	<link rel="stylesheet" href="<lams:LAMSURL/>css/x-editable.css"> 
-	<link rel="stylesheet" href="<lams:LAMSURL/>css/x-editable-lams.css"> 
-	<style>
-		#edit-collection-button {
-			float: right;
-		}
-		
-		.row {
-			margin-top: 10px;
-		}
-				
-		.middle-cell {
-			padding-top: 6px;
-			display: inline-block;
-		}
+	<lams:head>
+		<title><fmt:message key="label.qb.collection" /></title>
+		<meta name="viewport" content="width=device-width, initial-scale=1">
 
-		.header-column {
-			font-weight: bold;
-		}
+		<lams:css/>
+		<link type="text/css" href="<lams:LAMSURL/>css/free.ui.jqgrid.min.css" rel="stylesheet">
+		<link type="text/css" href="<lams:LAMSURL/>css/thickbox.css" rel="stylesheet">
+		<link rel="stylesheet" href="<lams:LAMSURL/>css/x-editable.css">
+		<link rel="stylesheet" href="<lams:LAMSURL/>css/x-editable-lams.css">
+		<style>
+			#edit-collection-button {
+				float: right;
+			}
 
-		.grid-question-count {
-			margin-left: 10px;
-		}
-		
-		.grid-collection-private {
-			color: red !important;
-			font-size: smaller;
-		}
-		
-		#grid-question-create {
-			display: inline-block;
-			float: right;
-			margin-right: 50px;
-		}
-		
-		#collection-name-row {
-			min-height: 20px; 
-			margin-top: 20px;
-		}
-		
-		#collection-name:hover+span+i {
-			visibility:visible
-		}
-		#collection-name+span+i {
-			visibility:hidden
-		}
-		
-		a.thickbox {
-			color: black;
-			padding-left: 8px;
-		}
-		
-		select {
-			height: auto !important;
-			width: auto !important;
-		}
-		
-		.all-learning-outcomes {
-			display: none;
-		}
-		
-		.all-learning-outcomes-show {
-			cursor: pointer;
-		}
-	</style>
-	
-	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/jquery.js"></script>
-	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/jquery-ui.js"></script>
-	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/bootstrap.min.js"></script>
-	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/free.jquery.jqgrid.min.js"></script>
-	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/thickbox.js"></script>
-	<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/x-editable.js"></script>
+			.row {
+				margin-top: 10px;
+			}
+
+			.middle-cell {
+				padding-top: 6px;
+				display: inline-block;
+			}
+
+			.header-column {
+				font-weight: bold;
+			}
+
+			.grid-question-count {
+				margin-left: 10px;
+			}
+
+			.grid-collection-private {
+				color: red !important;
+				font-size: smaller;
+			}
+
+			#grid-question-create {
+				display: inline-block;
+				float: right;
+				margin-right: 50px;
+			}
+
+			#collection-name-row {
+				min-height: 20px;
+				margin-top: 20px;
+			}
+
+			#collection-name:hover+span+i {
+				visibility:visible
+			}
+			#collection-name+span+i {
+				visibility:hidden
+			}
+
+			a.thickbox {
+				color: black;
+				padding-left: 8px;
+			}
+
+			select {
+				height: auto !important;
+				width: auto !important;
+			}
+
+			.all-learning-outcomes {
+				display: none;
+			}
+
+			.all-learning-outcomes-show {
+				cursor: pointer;
+			}
+		</style>
+
+		<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/jquery.js"></script>
+		<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/jquery-ui.js"></script>
+		<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/bootstrap.min.js"></script>
+		<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/free.jquery.jqgrid.min.js"></script>
+		<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/thickbox.js"></script>
+		<script type="text/javascript" src="<lams:LAMSURL/>includes/javascript/x-editable.js"></script>
 		<lams:JSImport src="includes/javascript/common.js" />
-	<script type="text/javascript">
-		<%-- Add jqGrid internationalisation snippet --%>
-		<%@ include file="/jqGrid.i18n.jsp"%>
-			
-		$(document).ready(function(){
-			var collectionGrid = $('#collection-grid'),	
-				isPublicCollection = ${empty collection.userId};
-			
-			collectionGrid.jqGrid({
-				guiStyle: "bootstrap",
-				iconSet: 'fontAwesome',
-				caption: '<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.title" /></spring:escapeBody>',
-			    datatype: "xml",
-			    url: "<lams:LAMSURL />qb/collection/getCollectionGridData.do?view=single&collectionUid=${collection.uid}",
-			    height: "100%",
-			    autowidth:true,
-				shrinkToFit: true,
-				<%-- Do not allow batch removing questions from the public collection --%>
-				multiselect: !isPublicCollection,
-				multiPageSelection: !isPublicCollection,
-				viewrecords: true,
-			    cellEdit: false,
-			    cmTemplate: { title: false, search: false },
-			    sortorder: "asc", 
-			    sortname: "name", 
-			    pager: true,
-			    rowList:[10,20,30,40,50,100],
-			    rowNum: 10,
-			    viewrecords: true,
-			    recordpos: "left",
-			    hidegrid: false,
-			    colNames:[
-			    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.id" /></spring:escapeBody>',
-			    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.name" /></spring:escapeBody>',
-			    	"questionType",
-			    	"questionVersion",
-			    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.outcomes" /></spring:escapeBody>',
-			    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.usage" /></spring:escapeBody>',
-			    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.actions" /></spring:escapeBody>',
-			    	"hasVersions"
-			    ],
-			    colModel:[
-			    	{name:'id', index:'uid', sortable:true, hidden:false, width: 10},
-			    	{name:'name', index:'name', sortable:true, search:true, autoencode:true, formatter: questionNameFormatter},
-			    	{name:'questionType', index:'questionType', width:0, hidden: true},
-			    	{name:'questionVersion', index:'questionVersion', width:0, hidden: true},
-			    	{name: 'learningOutcomes', index: 'learningOutcomes', sortable:true, search:true, autoencode: true, width: 25, formatter: learningOutcomeFormatter},
-			    	{name: 'usage', index: 'usage', sortable:true, width: 15, align: "center"},
-			      	// formatter gets just question uid and creates a button
-			    	{name:'actions', index:'actions', classes: "stats-cell", sortable:false, width: 13, align: "center", formatter: actionsFormatter},
-					{name:'hasVersions', index:'hasVersions', width:0, hidden: true}
-			    ],
-				beforeSelectRow: function(rowid, e) {
-					// do not select rows at all
-				    return !isPublicCollection;
-				},
-				loadComplete: function(data) {
-					//init thickbox
-					tb_init('a.thickbox');
-			    },
-			    loadError: function(xhr,st,err) {
-			    	collectionGrid.clearGridData();
-				   	alert('<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.error" /></spring:escapeBody>');
-			    },
-				subGrid : true,
-				subGridOptions: {
-				    hasSubgrid: function (options) {
-				        return options.data.hasVersions == 'true';
-				    }
-				 },
-				subGridRowExpanded: function(subgrid_id, row_id) {
-				    var subgrid_table_id = subgrid_id + "_t",
-				   	    rowData = jQuery("#" + subgrid_id.substring(0, subgrid_id.lastIndexOf('_'))).getRowData(row_id),
-				   	    questionId = rowData["id"].split("_")[0];
-					jQuery("#"+subgrid_id).html("<table id='"+subgrid_table_id+"' class='scroll archive'></table><div id='"+subgrid_table_id+"_pager' class='scroll' ></div>");
-					jQuery("#"+subgrid_table_id).jqGrid({
-							 guiStyle: "bootstrap",
-							 iconSet: 'fontAwesome',
-							 autoencode:false,
-							 autowidth: true,
-						     datatype: "xml",
-						     url: "<lams:LAMSURL />qb/collection/getQuestionVersionGridData.do?view=version&qbQuestionId=" + questionId,
-						     height: "100%",
-						     cmTemplate: { title: false },
-						     cellEdit:false,
-						     pager: false,
-						     colNames: [
-						    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.id" /></spring:escapeBody>',
-						    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.name" /></spring:escapeBody>',
-						    	"questionType",
-						    	"questionVersion",
-						    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.outcomes" /></spring:escapeBody>',
-						    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.usage" /></spring:escapeBody>',
-						    	'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.actions" /></spring:escapeBody>'
-						     ],
-						     colModel: [
-						    	{name:'id', index:'question_id', sortable:false, hidden:true, width: 10},
-						    	{name:'name', index:'name', sortable:false, search:false, autoencode:false, formatter: questionNameFormatter},
-						    	{name:'questionType', index:'questionType', width:0, hidden: true},
-						    	{name:'questionVersion', index:'questionVersion', width:0, hidden: true},
-						    	{name: 'learningOutcomes', index: 'learningOutcomes', width:0, hidden: true},
-						    	{name: 'usage', index: 'usage', sortable:false, width: 9, align: "center"},
-						      	// formatter gets just question uid and creates a button
-						    	{name:'actions', index:'actions', classes: "stats-cell", sortable:false, width: 12, align: "center", formatter: actionsFormatter}
-						     ],
-						     loadError: function(xhr,st,err) {
-						    	jQuery("#"+subgrid_table_id).clearGridData();
-						    	alert('<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.error" /></spring:escapeBody>');
-						     }
-					  	});
+		<script type="text/javascript">
+			<%-- Add jqGrid internationalisation snippet --%>
+			<%@ include file="/jqGrid.i18n.jsp"%>
+
+			$(document).ready(function(){
+				var collectionGrid = $('#collection-grid'),
+						isPublicCollection = ${empty collection.userId};
+
+				collectionGrid.jqGrid({
+					guiStyle: "bootstrap",
+					iconSet: 'fontAwesome',
+					caption: '<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.title" /></spring:escapeBody>',
+					datatype: "xml",
+					url: "<lams:LAMSURL />qb/collection/getCollectionGridData.do?view=single&collectionUid=${collection.uid}",
+					height: "100%",
+					autowidth:true,
+					shrinkToFit: true,
+					<%-- Do not allow batch removing questions from the public collection --%>
+					multiselect: !isPublicCollection,
+					multiPageSelection: !isPublicCollection,
+					viewrecords: true,
+					cellEdit: false,
+					cmTemplate: { title: false, search: false },
+					sortorder: "asc",
+					sortname: "name",
+					pager: true,
+					rowList:[10,20,30,40,50,100],
+					rowNum: 10,
+					viewrecords: true,
+					recordpos: "left",
+					hidegrid: false,
+					colNames:[
+						'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.id" /></spring:escapeBody>',
+						'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.name" /></spring:escapeBody>',
+						"questionType",
+						"questionVersion",
+						'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.outcomes" /></spring:escapeBody>',
+						'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.usage" /></spring:escapeBody>',
+						'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.actions" /></spring:escapeBody>',
+						"hasVersions"
+					],
+					colModel:[
+						{name:'id', index:'uid', sortable:true, hidden:false, width: 10},
+						{name:'name', index:'name', sortable:true, search:true, autoencode:true, formatter: questionNameFormatter},
+						{name:'questionType', index:'questionType', width:0, hidden: true},
+						{name:'questionVersion', index:'questionVersion', width:0, hidden: true},
+						{name: 'learningOutcomes', index: 'learningOutcomes', sortable:true, search:true, autoencode: true, width: 25, formatter: learningOutcomeFormatter},
+						{name: 'usage', index: 'usage', sortable:true, width: 15, align: "center"},
+						// formatter gets just question uid and creates a button
+						{name:'actions', index:'actions', classes: "stats-cell", sortable:false, width: 13, align: "center", formatter: actionsFormatter},
+						{name:'hasVersions', index:'hasVersions', width:0, hidden: true}
+					],
+					beforeSelectRow: function(rowid, e) {
+						// do not select rows at all
+						return !isPublicCollection;
+					},
+					loadComplete: function(data) {
+						//init thickbox
+						tb_init('a.thickbox');
+					},
+					loadError: function(xhr,st,err) {
+						collectionGrid.clearGridData();
+						alert('<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.error" /></spring:escapeBody>');
+					},
+					subGrid : true,
+					subGridOptions: {
+						hasSubgrid: function (options) {
+							return options.data.hasVersions == 'true';
+						}
+					},
+					subGridRowExpanded: function(subgrid_id, row_id) {
+						var subgrid_table_id = subgrid_id + "_t",
+								rowData = jQuery("#" + subgrid_id.substring(0, subgrid_id.lastIndexOf('_'))).getRowData(row_id),
+								questionId = rowData["id"].split("_")[0];
+						jQuery("#"+subgrid_id).html("<table id='"+subgrid_table_id+"' class='scroll archive'></table><div id='"+subgrid_table_id+"_pager' class='scroll' ></div>");
+						jQuery("#"+subgrid_table_id).jqGrid({
+							guiStyle: "bootstrap",
+							iconSet: 'fontAwesome',
+							autoencode:false,
+							autowidth: true,
+							datatype: "xml",
+							url: "<lams:LAMSURL />qb/collection/getQuestionVersionGridData.do?view=version&qbQuestionId=" + questionId,
+							height: "100%",
+							cmTemplate: { title: false },
+							cellEdit:false,
+							pager: false,
+							colNames: [
+								'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.id" /></spring:escapeBody>',
+								'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.name" /></spring:escapeBody>',
+								"questionType",
+								"questionVersion",
+								'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.outcomes" /></spring:escapeBody>',
+								'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.usage" /></spring:escapeBody>',
+								'<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.actions" /></spring:escapeBody>'
+							],
+							colModel: [
+								{name:'id', index:'question_id', sortable:false, hidden:true, width: 10},
+								{name:'name', index:'name', sortable:false, search:false, autoencode:false, formatter: questionNameFormatter},
+								{name:'questionType', index:'questionType', width:0, hidden: true},
+								{name:'questionVersion', index:'questionVersion', width:0, hidden: true},
+								{name: 'learningOutcomes', index: 'learningOutcomes', width:0, hidden: true},
+								{name: 'usage', index: 'usage', sortable:false, width: 9, align: "center"},
+								// formatter gets just question uid and creates a button
+								{name:'actions', index:'actions', classes: "stats-cell", sortable:false, width: 12, align: "center", formatter: actionsFormatter}
+							],
+							loadError: function(xhr,st,err) {
+								jQuery("#"+subgrid_table_id).clearGridData();
+								alert('<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.grid.error" /></spring:escapeBody>');
+							}
+						});
 					}
-			}).jqGrid('filterToolbar');
+				}).jqGrid('filterToolbar');
 
-			if (!isPublicCollection) {
-				// turn to inline mode for x-editable.js
-				$.fn.editable.defaults.mode = 'inline';
-				// do not cancel on clicking outside of box
-				$.fn.editable.defaults.onblur = 'ignore';
-				//enable renaming of lesson title  
-				$('#collection-name').editable({
-				    type: 'text',
-				    pk: ${collection.uid},
-				    url: "<lams:LAMSURL />qb/collection/changeCollectionName.do?<csrf:token/>",
-				    validate: function(value) {
-					    //close editing area on validation failure
-			            if (!value.trim()) {
-			                $('.editable-open').editableContainer('hide', 'cancel');
-			                return '<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.name.blank.error" /></spring:escapeBody>';
-			            }
-			        },
-				    //assume server response: 200 Ok {status: 'error', msg: 'field cannot be empty!'}
-				    success: function(response, newValue) {
-						if (response.created == 'false') {
-							alert('<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.name.duplicate.error" /></spring:escapeBody>');
+				if (!isPublicCollection) {
+					// turn to inline mode for x-editable.js
+					$.fn.editable.defaults.mode = 'inline';
+					// do not cancel on clicking outside of box
+					$.fn.editable.defaults.onblur = 'ignore';
+					//enable renaming of lesson title
+					$('#collection-name').editable({
+						type: 'text',
+						pk: ${collection.uid},
+						url: "<lams:LAMSURL />qb/collection/changeCollectionName.do?<csrf:token/>",
+						validate: function(value) {
+							//close editing area on validation failure
+							if (!value.trim()) {
+								$('.editable-open').editableContainer('hide', 'cancel');
+								return '<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.name.blank.error" /></spring:escapeBody>';
+							}
+						},
+						//assume server response: 200 Ok {status: 'error', msg: 'field cannot be empty!'}
+						success: function(response, newValue) {
+							if (response.created == 'false') {
+								alert('<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.name.duplicate.error" /></spring:escapeBody>');
+							}
 						}
-				    }
-			    //hide and show pencil on showing and hiding editing widget
-				}).on('shown', function(e, editable) {
-					$(this).nextAll('i.fa-pencil').hide();
-				}).on('hidden', function(e, reason) {
-					$(this).nextAll('i.fa-pencil').show();
+						//hide and show pencil on showing and hiding editing widget
+					}).on('shown', function(e, editable) {
+						$(this).nextAll('i.fa-pencil').hide();
+					}).on('hidden', function(e, reason) {
+						$(this).nextAll('i.fa-pencil').show();
+					});
+				}
+			});
+
+			function learningOutcomeFormatter(cellvalue) {
+				if (!cellvalue) {
+					return '';
+				}
+
+				var outcomes = cellvalue.split('<br>'),
+						cellhtml = '',
+						firstOutcome = null;
+				outcomes.forEach(function(outcome){
+					cellhtml += '<span class="alert-info btn-xs">' + outcome + '</span><br>';
+					if (!firstOutcome) {
+						firstOutcome = cellhtml;
+					}
 				});
+
+				// if there is only one outcome, just show it
+				// if there is more, show the first one and a button to show all
+				return outcomes.length === 1 ? cellhtml :
+						firstOutcome + '<a class="all-learning-outcomes-show" onClick="javascript:showLearningOutcomes(this)">'
+						+ '(show all)</a><div class="all-learning-outcomes">' + cellhtml + '</div>';
 			}
-		});
-		
-		function learningOutcomeFormatter(cellvalue) {
-			if (!cellvalue) {
-				return '';
+
+			function showLearningOutcomes(button) {
+				var cell = $(button).closest('td'),
+						fullText = $('.all-learning-outcomes', cell).html();
+				cell.html(fullText);
 			}
-			
-			var outcomes = cellvalue.split('<br>'),
-				cellhtml = '',
-				firstOutcome = null;
-			outcomes.forEach(function(outcome){
-				cellhtml += '<span class="alert-info btn-xs">' + outcome + '</span><br>';
-				if (!firstOutcome) {
-					firstOutcome = cellhtml;
+
+			// auxiliary formatter for jqGrid's question statistics column
+			function actionsFormatter(cellvalue){
+				var cellhtml = "<i class='fa fa-bar-chart' onClick='javascript:window.open(\"<lams:LAMSURL/>qb/stats/show.do?qbQuestionUid=" + cellvalue
+						+ "\", \"_blank\")' title='<spring:escapeBody javaScriptEscape="true"><fmt:message key='label.qb.collection.action.stats' /></spring:escapeBody>'></i>";
+
+				cellhtml += "<a  title='<spring:escapeBody javaScriptEscape="true"><fmt:message key='label.edit' /></spring:escapeBody>' href='<c:url value='/qb/edit/editQuestion.do'/>?qbQuestionUid="
+						+ cellvalue + "&oldCollectionUid=${collection.uid}&KeepThis=true&TB_iframe=true&modal=true' class='thickbox'>";
+				cellhtml += 	"<i class='fa fa-pencil'></i>";
+				cellhtml += "</a>";
+
+				return cellhtml;
+			}
+
+			//auxiliary formatter for jqGrid's question column
+			function questionNameFormatter (cellvalue, options, rowObject) {
+				var questionTypeInt = rowObject[2].textContent;
+				var questionType;
+				switch (questionTypeInt) {
+					case '1':
+						questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.multiple.choice" /></spring:escapeBody>";
+						break;
+					case '2':
+						questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.matching.pairs" /></spring:escapeBody>";
+						break;
+					case '3':
+						questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.short.answer" /></spring:escapeBody>";
+						break;
+					case '4':
+						questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.numerical" /></spring:escapeBody>";
+						break;
+					case '5':
+						questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.true.false" /></spring:escapeBody>";
+						break;
+					case '6':
+						questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.essay" /></spring:escapeBody>";
+						break;
+					case '7':
+						questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.ordering" /></spring:escapeBody>";
+						break;
+					case '8':
+						questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.mark.hedging" /></spring:escapeBody>";
+						break;
 				}
-			});
-			
-			// if there is only one outcome, just show it
-			// if there is more, show the first one and a button to show all
-			return outcomes.length === 1 ? cellhtml :
-				   	firstOutcome + '<a class="all-learning-outcomes-show" onClick="javascript:showLearningOutcomes(this)">'
-					+ '(show all)</a><div class="all-learning-outcomes">' + cellhtml + '</div>';
-		}
-		
-		function showLearningOutcomes(button) {
-			var cell = $(button).closest('td'),
-				fullText = $('.all-learning-outcomes', cell).html();
-			cell.html(fullText);
-		}
-		
-		// auxiliary formatter for jqGrid's question statistics column
-		function actionsFormatter(cellvalue){
-			var cellhtml = "<i class='fa fa-bar-chart' onClick='javascript:window.open(\"<lams:LAMSURL/>qb/stats/show.do?qbQuestionUid=" + cellvalue 
-					+ "\", \"_blank\")' title='<spring:escapeBody javaScriptEscape="true"><fmt:message key='label.qb.collection.action.stats' /></spring:escapeBody>'></i>";
 
-			cellhtml += "<a  title='<spring:escapeBody javaScriptEscape="true"><fmt:message key='label.edit' /></spring:escapeBody>' href='<c:url value='/qb/edit/editQuestion.do'/>?qbQuestionUid="
-						+ cellvalue + "&oldCollectionUid=${collection.uid}&KeepThis=true&TB_iframe=true&modal=true' class='thickbox'>"; 
-			cellhtml += 	"<i class='fa fa-pencil'></i>";
-			cellhtml += "</a>";
+				var questionVersion = rowObject[3].textContent;
 
-			return cellhtml;
-		}
-		
-		//auxiliary formatter for jqGrid's question column
-		function questionNameFormatter (cellvalue, options, rowObject) {
-	       	var questionTypeInt = rowObject[2].textContent;
-	       	var questionType;
-	       	switch (questionTypeInt) {
-	        case '1':
-	        	questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.multiple.choice" /></spring:escapeBody>";
-	          	break;
-	        case '2':
-	        	questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.matching.pairs" /></spring:escapeBody>";
-	          	break;
-	        case '3':
-	        	questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.short.answer" /></spring:escapeBody>";
-	          	break;
-	        case '4':
-	        	questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.numerical" /></spring:escapeBody>";
-	          	break;
-	        case '5':
-	        	questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.true.false" /></spring:escapeBody>";
-	          	break;
-	        case '6':
-	        	questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.essay" /></spring:escapeBody>";
-	          	break;
-	        case '7':
-	        	questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.ordering" /></spring:escapeBody>";
-	          	break;
-	        case '8':
-	        	questionType = "<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.question.type.mark.hedging" /></spring:escapeBody>";
-	          	break;
-	      	}
+				var text = cellvalue ? "<a target='_blank' title='Show stats' href='<lams:LAMSURL/>qb/stats/show.do?qbQuestionUid="
+						+ options.rowId + "'>" + cellvalue  + "</a>" : "";
+				text += "<span class='pull-right alert-info btn-xs loffset5'>";
+				text += "v. " + questionVersion;
+				text += "</span>";
+				if (questionType) {
+					text += "<span class='pull-right alert-info btn-xs'>";
+					text += questionType;
+					text += "</span>";
+				}
 
-	       	var questionVersion = rowObject[3].textContent;
+				return text;
+			}
 
-	       	var text = cellvalue ? "<a target='_blank' title='Show stats' href='<lams:LAMSURL/>qb/stats/show.do?qbQuestionUid=" 
-	       						  + options.rowId + "'>" + cellvalue  + "</a>" : "";
-	        text += "<span class='pull-right alert-info btn-xs loffset5'>";
-	       	text += "v. " + questionVersion;
-	        text += "</span>";
-	        if (questionType) {
-		       	text += "<span class='pull-right alert-info btn-xs'>";
-		       	text += questionType;
-		        text += "</span>";
-	        }
-	        	
-			return text;
-		}
-		
-		// remove a collection
-		function removeCollection() {
-			<fmt:message key="label.qb.collection.remove.confirm" var="REMOVECOLLVAR">
-			    <fmt:param>${collection.name}</fmt:param>
-            </fmt:message>
-			
-			if (confirm('<spring:escapeBody javaScriptEscape="true">${REMOVECOLLVAR}</spring:escapeBody>')) {
+			// remove a collection
+			function removeCollection() {
+				<fmt:message key="label.qb.collection.remove.confirm" var="REMOVECOLLVAR">
+				<fmt:param>${collection.name}</fmt:param>
+				</fmt:message>
+
+				if (confirm('<spring:escapeBody javaScriptEscape="true">${REMOVECOLLVAR}</spring:escapeBody>')) {
+					$.ajax({
+						'url'  : '<lams:LAMSURL />qb/collection/removeCollection.do',
+						'type' : 'POST',
+						'dataType' : 'text',
+						'data' : {
+							'collectionUid' : ${collection.uid},
+							"<csrf:tokenname/>" : "<csrf:tokenvalue/>"
+						},
+						'cache' : false
+					}).done(function(){
+						document.location.href = '<lams:LAMSURL />qb/collection/show.do';
+					});
+				}
+			}
+
+			// share a collection with authors of an organisation
+			function shareCollection(organisationId) {
+				var grid =  $('#collection-grid');
+
 				$.ajax({
-					'url'  : '<lams:LAMSURL />qb/collection/removeCollection.do',
+					'url'  : '<lams:LAMSURL />qb/collection/shareCollection.do',
 					'type' : 'POST',
 					'dataType' : 'text',
 					'data' : {
 						'collectionUid' : ${collection.uid},
+						'organisationId' : organisationId,
 						"<csrf:tokenname/>" : "<csrf:tokenvalue/>"
 					},
 					'cache' : false
 				}).done(function(){
-					document.location.href = '<lams:LAMSURL />qb/collection/show.do';
+					document.location.reload();
 				});
 			}
-		}
-		
-		// share a collection with authors of an organisation
-		function shareCollection(organisationId) {
-			var grid =  $('#collection-grid');
-			
-			$.ajax({
-				'url'  : '<lams:LAMSURL />qb/collection/shareCollection.do',
-				'type' : 'POST',
-				'dataType' : 'text',
-				'data' : {
-					'collectionUid' : ${collection.uid},
-					'organisationId' : organisationId,
-					"<csrf:tokenname/>" : "<csrf:tokenvalue/>"
-				},
-				'cache' : false
-			}).done(function(){
-				document.location.reload();
-			});
-		}
-		
-		// stop sharing a collection with authors of an organisation
-		function unshareCollection(organisationId) {
-			var grid =  $('#collection-grid');
-			
-			$.ajax({
-				'url'  : '<lams:LAMSURL />qb/collection/unshareCollection.do',
-				'type' : 'POST',
-				'dataType' : 'text',
-				'data' : {
-					'collectionUid' : ${collection.uid},
-					'organisationId' : organisationId,
-					"<csrf:tokenname/>" : "<csrf:tokenvalue/>"
-				},
-				'cache' : false
-			}).done(function(){
-				document.location.reload();
-			});
-		}
 
-	    function importWordQuestions(){
-	    	window.open('<lams:LAMSURL/>questions/questionFile.jsp?importType=word',
-				'QuestionFile','width=500,height=370,scrollbars=yes');
-	    }
+			// stop sharing a collection with authors of an organisation
+			function unshareCollection(organisationId) {
+				var grid =  $('#collection-grid');
 
+				$.ajax({
+					'url'  : '<lams:LAMSURL />qb/collection/unshareCollection.do',
+					'type' : 'POST',
+					'dataType' : 'text',
+					'data' : {
+						'collectionUid' : ${collection.uid},
+						'organisationId' : organisationId,
+						"<csrf:tokenname/>" : "<csrf:tokenvalue/>"
+					},
+					'cache' : false
+				}).done(function(){
+					document.location.reload();
+				});
+			}
+
+			function importWordQuestions(){
+				window.open('<lams:LAMSURL/>questions/questionFile.jsp?importType=word',
+						'QuestionFile','width=500,height=370,scrollbars=yes');
+			}
+
 			<c:if test="${isAiEnabled}">
 			function importOpenAi() {
 				launchPopup('<lams:LAMSURL/>ai/authoring/ratMcq.do', 'OpenAiForm');
 			}
 			</c:if>
 
 
-	    function importQTI(){
-	    	window.open('<lams:LAMSURL/>questions/questionFile.jsp',
-				'QuestionFile','width=500,height=370,scrollbars=yes');
-	    }
-		
-	    function saveQTI(formHTML, formName) {
-	    	var form = $($.parseHTML(formHTML));
-			$.ajax({
-				url: '<c:url value="/imsqti/saveQTI.do" />?<csrf:token/>&collectionUid=${collection.uid}',
-				data: form.serializeArray(),
-				type: "POST",
-				success: function() {
-					location.reload();
-				}
-			});
-	    }
+			function importQTI(){
+				window.open('<lams:LAMSURL/>questions/questionFile.jsp',
+						'QuestionFile','width=500,height=370,scrollbars=yes');
+			}
 
-	    function exportQTI(){
-			//dynamically create a form and submit it
-			var exportExcelUrl = '<c:url value="/imsqti/exportCollectionAsQTI.do" />?<csrf:token/>&collectionUid=${collection.uid}';
-			var form = $('<form method="post" action="' + exportExcelUrl + '"></form>');
-		    $(document.body).append(form);
-		    form.submit();
-	    }
-		
-		function exportQuestionsXml(){
-			//dynamically create a form and submit it
-			var exportExcelUrl = "<c:url value='/xmlQuestions/exportQuestionsXml.do'/>?<csrf:token/>&collectionUid=${collection.uid}&reqID="+(new Date()).getTime();
-			var form = $('<form method="post" action="' + exportExcelUrl + '"></form>');
-		    $(document.body).append(form);
-		    form.submit();
-		}
+			function saveQTI(formHTML, formName) {
+				var form = $($.parseHTML(formHTML));
+				$.ajax({
+					url: '<c:url value="/imsqti/saveQTI.do" />?<csrf:token/>&collectionUid=${collection.uid}',
+					data: form.serializeArray(),
+					type: "POST",
+					success: function() {
+						location.reload();
+					}
+				});
+			}
 
+			function exportQTI(){
+				//dynamically create a form and submit it
+				var exportExcelUrl = '<c:url value="/imsqti/exportCollectionAsQTI.do" />?<csrf:token/>&collectionUid=${collection.uid}';
+				var form = $('<form method="post" action="' + exportExcelUrl + '"></form>');
+				$(document.body).append(form);
+				form.submit();
+			}
 
-		function removeQuestions(){
-			let collectionGrid = $('#collection-grid'), 
-				questionsToRemove = collectionGrid.jqGrid('getGridParam','selarrrow');
-			if (questionsToRemove.length == 0){
-				return;
+			function exportQuestionsXml(){
+				//dynamically create a form and submit it
+				var exportExcelUrl = "<c:url value='/xmlQuestions/exportQuestionsXml.do'/>?<csrf:token/>&collectionUid=${collection.uid}&reqID="+(new Date()).getTime();
+				var form = $('<form method="post" action="' + exportExcelUrl + '"></form>');
+				$(document.body).append(form);
+				form.submit();
 			}
-			let questionIdsToRemove = [];
-			questionsToRemove.forEach(function(questionUid) {
-				let questionId = $('tr#' + questionUid + ' > td[aria-describedby="collection-grid_id"]', collectionGrid).text();
-				if (questionId) {
-					questionIdsToRemove.push(questionId);
+
+
+			function removeQuestions(){
+				let collectionGrid = $('#collection-grid'),
+						questionsToRemove = collectionGrid.jqGrid('getGridParam','selarrrow');
+				if (questionsToRemove.length == 0){
+					return;
 				}
-			});
-			
-			$.ajax({
-				'url'  : '<lams:LAMSURL />qb/collection/removeCollectionQuestions.do',
-				'type' : 'POST',
-				'dataType' : 'text',
-				'data' : {
-					'collectionUid' : ${collection.uid},
-	 				'qbQuestionIds' : questionIdsToRemove,
-					"<csrf:tokenname/>" : "<csrf:tokenvalue/>"
-				},
-				'cache' : false
-			}).done(function(response){
-				if (response == 'fail') {
-					// not all questions were removed
-					alert('<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.remove.questions.fail"/></spring:escapeBody>');
-				}
-				document.location.reload();
-			});
-		}
-		
-		//create proper href for "Create question" button
-		function initLinkHref() {
-			var questionType = document.getElementById("question-type").selectedIndex + 1;
-			$("#create-question-href").attr("href", 
-					"<c:url value='/qb/edit/initNewQuestion.do'/>?questionType=" + questionType 
-					+ "&oldCollectionUid=${collection.uid}" 
-					+ "&KeepThis=true&TB_iframe=true&modal=true");
-		};
-		
-	    //this method gets invoked after question has been edited and saved
-		function refreshThickbox(){
-			location.reload();
-		};
-	</script>
-</lams:head>
-<body class="stripes">
-<lams:Page title="Collection" type="admin">
-	
-	<div>
-		<button class="btn btn-default btn-sm" onClick="javascript:document.location.href='<lams:LAMSURL />qb/collection/show.do'">
-			<i class="fa fa-angle-double-left"></i>
-			<fmt:message key="label.qb.collection.management" />
-		</button>
-	</div>
-	
-	<div id="collection-name-row" class="row h4">
-		
-		<div class="col-xs-12 col-sm-6">
+				let questionIdsToRemove = [];
+				questionsToRemove.forEach(function(questionUid) {
+					let questionId = $('tr#' + questionUid + ' > td[aria-describedby="collection-grid_id"]', collectionGrid).text();
+					if (questionId) {
+						questionIdsToRemove.push(questionId);
+					}
+				});
+
+				$.ajax({
+					'url'  : '<lams:LAMSURL />qb/collection/removeCollectionQuestions.do',
+					'type' : 'POST',
+					'dataType' : 'text',
+					'data' : {
+						'collectionUid' : ${collection.uid},
+						'qbQuestionIds' : questionIdsToRemove,
+						"<csrf:tokenname/>" : "<csrf:tokenvalue/>"
+					},
+					'cache' : false
+				}).done(function(response){
+					if (response == 'fail') {
+						// not all questions were removed
+						alert('<spring:escapeBody javaScriptEscape="true"><fmt:message key="label.qb.collection.remove.questions.fail"/></spring:escapeBody>');
+					}
+					document.location.reload();
+				});
+			}
+
+			//create proper href for "Create question" button
+			function initLinkHref() {
+				var questionType = document.getElementById("question-type").selectedIndex + 1;
+				$("#create-question-href").attr("href",
+						"<c:url value='/qb/edit/initNewQuestion.do'/>?questionType=" + questionType
+						+ "&oldCollectionUid=${collection.uid}"
+						+ "&KeepThis=true&TB_iframe=true&modal=true");
+			};
+
+			//this method gets invoked after question has been edited and saved
+			function refreshThickbox(){
+				location.reload();
+			};
+
+			function showQuestionsPrintPage() {
+				let url = '<c:url value="/qb/printQbCollectionQuestions.do?collectionUid=${collection.uid}"/>';
+				window.open(url, "_blank");
+			}
+		</script>
+	</lams:head>
+	<body class="stripes">
+	<lams:Page title="Collection" type="admin">
+
+		<div>
+			<button class="btn btn-default btn-sm" onClick="javascript:document.location.href='<lams:LAMSURL />qb/collection/show.do'">
+				<i class="fa fa-angle-double-left"></i>
+				<fmt:message key="label.qb.collection.management" />
+			</button>
+		</div>
+
+		<div id="collection-name-row" class="row h4">
+
+			<div class="col-xs-12 col-sm-4">
 			<span id="collection-name">
 				<strong>
 					<c:out value="${collection.name}" />
 				</strong>
 			</span>
-			<c:if test="${not empty collection.userId}">
-				<span>&nbsp;</span><i class='fa fa-sm fa-pencil'></i>
-			</c:if>
-			
-			<c:if test="${collection.personal}">
+				<c:if test="${not empty collection.userId}">
+					<span>&nbsp;</span><i class='fa fa-sm fa-pencil'></i>
+				</c:if>
+
+				<c:if test="${collection.personal}">
 				<span class="grid-collection-private small">
 					<i class="fa fa-lock"></i> <fmt:message key="label.qb.collection.private" />
 				</span>
-			</c:if>
-		</div>
-		
-		<div class=" col-xs-12 col-sm-6">
-			<a class="btn btn-default btn-xs loffset10 pull-right"
-			   onClick="javascript:removeQuestions()"
-			   title="<fmt:message key="label.qb.collection.remove.questions.tooltip" />">
-				<fmt:message key="label.qb.collection.remove.questions" />&nbsp;<i class="fa fa-trash"></i>
-			</a>
-			
-			<%-- Do not display button for public and private collections --%>
-			<c:if test="${not empty collection.userId and not collection.personal}">
-				<div class="btn-group-xs pull-right loffset10">
-					<c:if test="${not hasQuestions}">
-						<button class="btn btn-default" onClick="javascript:removeCollection()" title="Remove collection">
-							<i class="fa fa-trash"></i>
-							<fmt:message key="label.qb.collection.remove" />
-						</button>
-					</c:if>
-				</div>
-			</c:if>
-		
-			<div class="btn-group btn-group-xs pull-right loffset10" role="group">
-				<a class="btn btn-default btn-xs disabled" aria-disabled="true"><fmt:message key="label.qb.collection.xml" /></a>
-			
-				<a class="btn btn-default btn-xs loffset5 thickbox" title="<fmt:message key="label.import.xml" />"
-					href="<c:url value='/xmlQuestions/initImportQuestionsXml.do'/>?collectionUid=${collection.uid}&KeepThis=true&TB_iframe=true&modal=true" >  
-					<i class="fa fa-upload"></i>
-				</a>
-				
-				<c:if test="${isQtiExportEnabled && hasQuestions}">
-					<a onclick="javascript:exportQuestionsXml();" class="btn btn-default btn-xs" title="<fmt:message key="label.export.xml" />">  
-						<i class="fa fa-download"></i>
-					</a>
 				</c:if>
 			</div>
-			
-			<div class="btn-group btn-group-xs loffset10 pull-right" role="group">
-				<a class="btn btn-default btn-xs disabled" aria-disabled="true"><fmt:message key="label.qb.collection.qti" /></a>
-			
-				<a href="#nogo" onClick="javascript:importQTI()" class="btn btn-default" title="<fmt:message key='label.import.qti'/>">
-					<i class="fa fa-upload"></i>
+
+			<div class=" col-xs-12 col-sm-8">
+				<button onclick="javascript:showQuestionsPrintPage()" type="button"
+						class="btn btn-default btn-xs loffset10 pull-right">
+					<i class="fa fa-print" aria-hidden="true"></i>
+					<fmt:message key="label.print" />
+				</button>
+
+				<a class="btn btn-default btn-xs loffset10 pull-right"
+				   onClick="javascript:removeQuestions()"
+				   title="<fmt:message key="label.qb.collection.remove.questions.tooltip" />">
+					<fmt:message key="label.qb.collection.remove.questions" />&nbsp;<i class="fa fa-trash"></i>
 				</a>
-				
-				<c:if test="${isQtiExportEnabled && hasQuestions}">
-					<a href="#nogo" onClick="javascript:exportQTI()" class="btn btn-default" title="<fmt:message key='label.export.qti'/>">
-						<i class="fa fa-download"></i>
-					</a>
+
+					<%-- Do not display button for public and private collections --%>
+				<c:if test="${not empty collection.userId and not collection.personal}">
+					<div class="btn-group-xs pull-right loffset10">
+						<c:if test="${not hasQuestions}">
+							<button class="btn btn-default" onClick="javascript:removeCollection()" title="Remove collection">
+								<i class="fa fa-trash"></i>
+								<fmt:message key="label.qb.collection.remove" />
+							</button>
+						</c:if>
+					</div>
 				</c:if>
-			</div>
-			
-			<div class="btn-group btn-group-xs loffset10 pull-right" role="group">
-				<a class="btn btn-default btn-xs disabled" aria-disabled="true">
-					<fmt:message key='label.qb.collection.word'/>
-				</a>
-			
-				<a href="#nogo" onClick="javascript:importWordQuestions()" class="btn btn-default" title="<fmt:message key='label.import.word'/>">
-					<i class="fa fa-upload"></i>
-				</a>
-			</div>
-			
+
+				<div class="btn-group btn-group-xs pull-right loffset10" role="group">
+					<a class="btn btn-default btn-xs disabled" aria-disabled="true"><fmt:message key="label.qb.collection.xml" /></a>
+
+					<a class="btn btn-default btn-xs loffset5 thickbox" title="<fmt:message key="label.import.xml" />"
+					   href="<c:url value='/xmlQuestions/initImportQuestionsXml.do'/>?collectionUid=${collection.uid}&KeepThis=true&TB_iframe=true&modal=true" >
+						<i class="fa fa-upload"></i>
+					</a>
+
+					<c:if test="${isQtiExportEnabled && hasQuestions}">
+						<a onclick="javascript:exportQuestionsXml();" class="btn btn-default btn-xs" title="<fmt:message key="label.export.xml" />">
+							<i class="fa fa-download"></i>
+						</a>
+					</c:if>
+				</div>
+
+				<div class="btn-group btn-group-xs loffset10 pull-right" role="group">
+					<a class="btn btn-default btn-xs disabled" aria-disabled="true"><fmt:message key="label.qb.collection.qti" /></a>
+
+					<a href="#nogo" onClick="javascript:importQTI()" class="btn btn-default" title="<fmt:message key='label.import.qti'/>">
+						<i class="fa fa-upload"></i>
+					</a>
+
+					<c:if test="${isQtiExportEnabled && hasQuestions}">
+						<a href="#nogo" onClick="javascript:exportQTI()" class="btn btn-default" title="<fmt:message key='label.export.qti'/>">
+							<i class="fa fa-download"></i>
+						</a>
+					</c:if>
+				</div>
+
+				<div class="btn-group btn-group-xs loffset10 pull-right" role="group">
+					<a class="btn btn-default btn-xs disabled" aria-disabled="true">
+						<fmt:message key='label.qb.collection.word'/>
+					</a>
+
+					<a href="#nogo" onClick="javascript:importWordQuestions()" class="btn btn-default" title="<fmt:message key='label.import.word'/>">
+						<i class="fa fa-upload"></i>
+					</a>
+				</div>
+
 				<c:if test="${isAiEnabled}">
 					<div class="btn-group btn-group-xs loffset10 pull-right" role="group">
 						<a class="btn btn-default btn-xs disabled" aria-disabled="true">
@@ -570,52 +581,52 @@
 					</div>
 				</c:if>
 
-			<div class="btn-group-xs pull-right" style="display: flex;">
-				<select id="question-type" class="form-control btn-xs">
-					<option selected="selected"><fmt:message key="label.question.type.multiple.choice" /></option>
-					<option><fmt:message key="label.question.type.matching.pairs" /></option>
-					<option><fmt:message key="label.question.type.short.answer" /></option>
-					<option><fmt:message key="label.question.type.numerical" /></option>
-					<option><fmt:message key="label.question.type.true.false" /></option>
-					<option><fmt:message key="label.question.type.essay" /></option>
-					<option><fmt:message key="label.question.type.ordering" /></option>
-					<option><fmt:message key="label.question.type.mark.hedging" /></option>
-				</select>&nbsp;
-						
-				<a onclick="initLinkHref();return false;" href="" class="btn btn-default thickbox" id="create-question-href">  
-					<i class="fa fa-plus-circle" aria-hidden="true" title="<fmt:message key="label.create.question" />"></i>
-				</a>
+				<div class="btn-group-xs pull-right" style="display: flex;">
+					<select id="question-type" class="form-control btn-xs">
+						<option selected="selected"><fmt:message key="label.question.type.multiple.choice" /></option>
+						<option><fmt:message key="label.question.type.matching.pairs" /></option>
+						<option><fmt:message key="label.question.type.short.answer" /></option>
+						<option><fmt:message key="label.question.type.numerical" /></option>
+						<option><fmt:message key="label.question.type.true.false" /></option>
+						<option><fmt:message key="label.question.type.essay" /></option>
+						<option><fmt:message key="label.question.type.ordering" /></option>
+						<option><fmt:message key="label.question.type.mark.hedging" /></option>
+					</select>&nbsp;
+
+					<a onclick="initLinkHref();return false;" href="" class="btn btn-default thickbox" id="create-question-href">
+						<i class="fa fa-plus-circle" aria-hidden="true" title="<fmt:message key="label.create.question" />"></i>
+					</a>
+				</div>
+
 			</div>
-			
 		</div>
-	</div>
-				
-	<c:choose>
-		<c:when test="${hasQuestions}">			
-			<%-- jqGrid placeholder with some useful attributes --%>
-			<div class="voffset20" >
-				<table id="collection-grid"></table>
-			</div>
-		</c:when>
-		<c:otherwise>
-			<div class="alert alert-warning">
-				<fmt:message key="label.qb.collection.questions.none" />
-			</div>
-		</c:otherwise>
-	</c:choose>
 
-	<%-- Do not display links for collection manipulation for public and private collections --%>
-	<c:if test="${(isAppadmin or (userId eq collection.userId)) and not collection.personal
+		<c:choose>
+			<c:when test="${hasQuestions}">
+				<%-- jqGrid placeholder with some useful attributes --%>
+				<div class="voffset20" >
+					<table id="collection-grid"></table>
+				</div>
+			</c:when>
+			<c:otherwise>
+				<div class="alert alert-warning">
+					<fmt:message key="label.qb.collection.questions.none" />
+				</div>
+			</c:otherwise>
+		</c:choose>
+
+		<%-- Do not display links for collection manipulation for public and private collections --%>
+		<c:if test="${(isAppadmin or (userId eq collection.userId)) and not collection.personal
 		and (not empty collection.organisations or not empty availableOrganisations)}">
-		<div class="panel panel-default voffset20">
-			<div class="panel-heading">
-				<fmt:message key="label.qb.collection.share.title" />
-			</div>
-			<div class="panel-body">
-				<table class="table table-striped table-condensed table-responsive" style="max-width: 400px;">
-					<tbody>
-					
-						<%-- Already shared organisations--%>
+			<div class="panel panel-default voffset20">
+				<div class="panel-heading">
+					<fmt:message key="label.qb.collection.share.title" />
+				</div>
+				<div class="panel-body">
+					<table class="table table-striped table-condensed table-responsive" style="max-width: 400px;">
+						<tbody>
+
+							<%-- Already shared organisations--%>
 						<c:forEach var="organisation" items="${collection.organisations}">
 							<tr  class="info">
 								<td>
@@ -629,8 +640,8 @@
 								</td>
 							</tr>
 						</c:forEach>
-							
-						<%-- Not yet shared ones --%>
+
+							<%-- Not yet shared ones --%>
 						<c:forEach var="organisation" items="${availableOrganisations}">
 							<tr>
 								<td>
@@ -641,17 +652,17 @@
 										<fmt:message key="label.qb.collection.share" />
 									</button>
 								</td>
-							</tr>						
+							</tr>
 						</c:forEach>
-					</tbody>
-				</table>
-			
+						</tbody>
+					</table>
+
+				</div>
 			</div>
-		</div>
-	</c:if>
-	
-	<!-- Dummy div for question save to work properly -->
-	<div id="itemArea" class="hidden"></div>
-</lams:Page>
-</body>
+		</c:if>
+
+		<!-- Dummy div for question save to work properly -->
+		<div id="itemArea" class="hidden"></div>
+	</lams:Page>
+	</body>
 </lams:html>
\ No newline at end of file
Index: lams_central/web/qb/printQuestions.jsp
===================================================================
diff -u -r32a2a44474f2a866af0ce193d299f5ac3554865a -rcb84acde58155494dc2cdf1bae82eac746dadfe2
--- lams_central/web/qb/printQuestions.jsp	(.../printQuestions.jsp)	(revision 32a2a44474f2a866af0ce193d299f5ac3554865a)
+++ lams_central/web/qb/printQuestions.jsp	(.../printQuestions.jsp)	(revision cb84acde58155494dc2cdf1bae82eac746dadfe2)
@@ -14,6 +14,7 @@
 			body {
 				text-align: center;
 				overflow: visible;
+				padding-left: 1rem;
 			}
 
 			#question-list {
@@ -51,6 +52,7 @@
 
 				body {
 					text-align: left;
+					padding-left: 0;
 				}
 
 				.question-item, img {
Index: lams_common/src/java/org/lamsfoundation/lams/commonContext.xml
===================================================================
diff -u -r52a6369ce66a2ea70d38bfa2e627dfcde6354b10 -rcb84acde58155494dc2cdf1bae82eac746dadfe2
--- lams_common/src/java/org/lamsfoundation/lams/commonContext.xml	(.../commonContext.xml)	(revision 52a6369ce66a2ea70d38bfa2e627dfcde6354b10)
+++ lams_common/src/java/org/lamsfoundation/lams/commonContext.xml	(.../commonContext.xml)	(revision cb84acde58155494dc2cdf1bae82eac746dadfe2)
@@ -551,6 +551,7 @@
 		<property name="userManagementService"><ref bean="userManagementService"/></property>
 		<property name="logEventService"><ref bean="logEventService"/></property>
         <property name="toolService"><ref bean="lamsToolService"/></property>
+        <property name="securityService"><ref bean="securityService"/></property>
 	</bean>
 	
 	<bean id="qbService" class="org.springframework.transaction.interceptor.TransactionProxyFactoryBean">
Index: lams_common/src/java/org/lamsfoundation/lams/qb/service/IQbService.java
===================================================================
diff -u -re7cbf85417c8226f95995e5adb34f7eeb386a98c -rcb84acde58155494dc2cdf1bae82eac746dadfe2
--- lams_common/src/java/org/lamsfoundation/lams/qb/service/IQbService.java	(.../IQbService.java)	(revision e7cbf85417c8226f95995e5adb34f7eeb386a98c)
+++ lams_common/src/java/org/lamsfoundation/lams/qb/service/IQbService.java	(.../IQbService.java)	(revision cb84acde58155494dc2cdf1bae82eac746dadfe2)
@@ -183,4 +183,6 @@
     TreeSet<QbQuestionUnit> getUnitsFromRequest(HttpServletRequest request, boolean isForSaving);
 
     Long allocateVSAnswerToOption(Long qbQuestionUid, Long targetOptionUid, Long previousOptionUid, String answer);
+
+    boolean hasUserAccessToCollection(long collectionUid);
 }
\ No newline at end of file
Index: lams_common/src/java/org/lamsfoundation/lams/qb/service/QbService.java
===================================================================
diff -u -r1e85a36d6397d66fa9cc46b334e2ef2bef8af4f6 -rcb84acde58155494dc2cdf1bae82eac746dadfe2
--- lams_common/src/java/org/lamsfoundation/lams/qb/service/QbService.java	(.../QbService.java)	(revision 1e85a36d6397d66fa9cc46b334e2ef2bef8af4f6)
+++ lams_common/src/java/org/lamsfoundation/lams/qb/service/QbService.java	(.../QbService.java)	(revision cb84acde58155494dc2cdf1bae82eac746dadfe2)
@@ -49,6 +49,7 @@
 import org.lamsfoundation.lams.qb.model.QbQuestion;
 import org.lamsfoundation.lams.qb.model.QbQuestionUnit;
 import org.lamsfoundation.lams.qb.model.QbToolQuestion;
+import org.lamsfoundation.lams.security.ISecurityService;
 import org.lamsfoundation.lams.tool.ToolContent;
 import org.lamsfoundation.lams.tool.service.ILamsCoreToolService;
 import org.lamsfoundation.lams.tool.service.ILamsToolService;
@@ -84,9 +85,11 @@
 
     private ILamsToolService toolService;
 
-    public static final Comparator<QbCollection> COLLECTION_NAME_COMPARATOR = Comparator
-	    .comparing(QbCollection::getName);
+    private ISecurityService securityService;
 
+    public static final Comparator<QbCollection> COLLECTION_NAME_COMPARATOR = Comparator.comparing(
+	    QbCollection::getName);
+
     @Override
     public QbQuestion getQuestionByUid(Long qbQuestionUid) {
 	return qbDAO.getQuestionByUid(qbQuestionUid);
@@ -268,8 +271,9 @@
 	QbStatsActivityDTO activityDTO = new QbStatsActivityDTO();
 	activityDTO.setActivity(activity);
 
-	String monitorUrl = "/lams/" + lamsCoreToolService.getToolMonitoringURL(lessonId, activity)
-		+ "&contentFolderID=" + learningDesign.getContentFolderID();
+	String monitorUrl =
+		"/lams/" + lamsCoreToolService.getToolMonitoringURL(lessonId, activity) + "&contentFolderID="
+			+ learningDesign.getContentFolderID();
 	activityDTO.setMonitorURL(monitorUrl);
 
 	// if there is only 1 participant, there is no point in calculating question indexes
@@ -304,8 +308,8 @@
 	    // sort grades by highest mark
 	    Collections.sort(userLessonGrades, Comparator.comparing(GradebookUserLesson::getMark).reversed());
 	    // see how many learners should be in top/bottom 27% of the group
-	    int groupCount = (int) Math
-		    .ceil(Configuration.getAsInt(ConfigurationKeys.QB_STATS_GROUP_SIZE) / 100.0 * participantCount);
+	    int groupCount = (int) Math.ceil(
+		    Configuration.getAsInt(ConfigurationKeys.QB_STATS_GROUP_SIZE) / 100.0 * participantCount);
 
 	    // go through each grade and gather data for indexes
 	    for (int userIndex = 0; userIndex < participantCount; userIndex++) {
@@ -634,12 +638,12 @@
     @Override
     public List<QbCollection> getUserCollections(int userId) {
 	Set<QbCollection> collections = new LinkedHashSet<>();
-	
+
 	// even though it is covered by #getUserOwnCollections(),
 	// it creates user private collection when necessary
 	QbCollection privateCollection = getUserPrivateCollection(userId);
 	collections.add(privateCollection);
-	
+
 	collections.addAll(getUserOwnCollections(userId));
 
 	QbCollection publicCollection = getPublicCollection();
@@ -689,17 +693,17 @@
     }
 
     /**
-     * Is the question in a learning design which is in user's private folder, public folder
-     * or a course folder where user is a monitor.
+     * Is the question in a learning design which is in user's private folder, public folder or a course folder where
+     * user is a monitor.
      */
     @Override
     public boolean isQuestionInUserMonitoredOrganisationFolder(int qbQuestionId, int userId) {
 	return qbDAO.isQuestionInUserMonitoredOrganisationFolder(qbQuestionId, userId);
     }
 
     /**
-     * Cascades in QbToolQuestion, QbQuestion and QbOptions do not seem to work on insert.
-     * New QbQuestions need to be saved step by step.
+     * Cascades in QbToolQuestion, QbQuestion and QbOptions do not seem to work on insert. New QbQuestions need to be
+     * saved step by step.
      */
     @Override
     public void insertQuestion(QbQuestion qbQuestion) {
@@ -712,14 +716,14 @@
 	    qbQuestion.setContentFolderId(FileUtil.generateUniqueContentFolderID());
 	}
 
-	Collection<QbOption> qbOptions = qbQuestion.getQbOptions() == null ? null
-		: new ArrayList<>(qbQuestion.getQbOptions());
+	Collection<QbOption> qbOptions =
+		qbQuestion.getQbOptions() == null ? null : new ArrayList<>(qbQuestion.getQbOptions());
 	if (qbOptions != null) {
 	    qbQuestion.getQbOptions().clear();
 	}
 
-	Collection<QbQuestionUnit> units = qbQuestion.getUnits() == null ? null
-		: new ArrayList<>(qbQuestion.getUnits());
+	Collection<QbQuestionUnit> units =
+		qbQuestion.getUnits() == null ? null : new ArrayList<>(qbQuestion.getUnits());
 	if (units != null) {
 	    qbQuestion.getUnits().clear();
 	}
@@ -833,7 +837,7 @@
      * Allocate learner's answer into one of the available answer groups.
      *
      * @return if present, it contains optionUid of the option group containing duplicate (added there presumably by
-     *         another teacher working in parallel)
+     * 	another teacher working in parallel)
      */
     @Override
     public Long allocateVSAnswerToOption(Long toolQuestionUid, Long targetOptionUid, Long previousOptionUid,
@@ -874,8 +878,9 @@
 
 	if (!targetOptionUid.equals(-1L) && targetOption == null) {
 	    // front end provided incorrect target option UID
-	    log.error("Target option with UID " + targetOptionUid + " was not found in question with UID "
-		    + qbQuestionUid + " to allocate answer " + answer);
+	    log.error(
+		    "Target option with UID " + targetOptionUid + " was not found in question with UID " + qbQuestionUid
+			    + " to allocate answer " + answer);
 	    return null;
 	}
 
@@ -886,7 +891,8 @@
 
 	    Set<String> nameWithoutUserAnswer = new LinkedHashSet<>(List.of(alternatives));
 	    nameWithoutUserAnswer.remove(answer);
-	    name = nameWithoutUserAnswer.isEmpty() ? ""
+	    name = nameWithoutUserAnswer.isEmpty()
+		    ? ""
 		    : nameWithoutUserAnswer.stream().filter(a -> QbUtils.normaliseVSAnswer(a, isExactMatch) != null)
 			    .collect(Collectors.joining(QbUtils.VSA_ANSWER_DELIMITER));
 	    previousOption.setName(name);
@@ -931,9 +937,9 @@
     /**
      * Extract web form content to QB question.
      *
-     * BE CAREFUL: This method will copy necessary info from request form to an old or new AssessmentQuestion
-     * instance. It gets all info EXCEPT AssessmentQuestion.createDate, which need be set when
-     * persisting this assessment Question.
+     * BE CAREFUL: This method will copy necessary info from request form to an old or new AssessmentQuestion instance.
+     * It gets all info EXCEPT AssessmentQuestion.createDate, which need be set when persisting this assessment
+     * Question.
      *
      * @return qbQuestionModified
      */
@@ -1008,9 +1014,9 @@
 	}
 
 	// set options
-	if ((type == QbQuestion.TYPE_MULTIPLE_CHOICE) || (type == QbQuestion.TYPE_ORDERING)
-		|| (type == QbQuestion.TYPE_MATCHING_PAIRS) || (type == QbQuestion.TYPE_VERY_SHORT_ANSWERS)
-		|| (type == QbQuestion.TYPE_NUMERICAL) || (type == QbQuestion.TYPE_MARK_HEDGING)) {
+	if ((type == QbQuestion.TYPE_MULTIPLE_CHOICE) || (type == QbQuestion.TYPE_ORDERING) || (type
+		== QbQuestion.TYPE_MATCHING_PAIRS) || (type == QbQuestion.TYPE_VERY_SHORT_ANSWERS) || (type
+		== QbQuestion.TYPE_NUMERICAL) || (type == QbQuestion.TYPE_MARK_HEDGING)) {
 	    Set<QbOption> optionList = getOptionsFromRequest(request, true);
 	    List<QbOption> options = new ArrayList<>();
 	    int displayOrder = 0;
@@ -1055,15 +1061,16 @@
      *
      * @param request
      * @param isForSaving
-     *            whether the blank options will be preserved or not
+     * 	whether the blank options will be preserved or not
      */
     @Override
     public TreeSet<QbOption> getOptionsFromRequest(HttpServletRequest request, boolean isForSaving) {
 	Map<String, String> paramMap = splitRequestParameter(request, QbConstants.ATTR_OPTION_LIST);
 
 	int count = NumberUtils.toInt(paramMap.get(QbConstants.ATTR_OPTION_COUNT));
 	int questionType = WebUtil.readIntParam(request, QbConstants.ATTR_QUESTION_TYPE);
-	Integer correctOptionIndex = (paramMap.get(QbConstants.ATTR_OPTION_CORRECT) == null) ? null
+	Integer correctOptionIndex = (paramMap.get(QbConstants.ATTR_OPTION_CORRECT) == null)
+		? null
 		: NumberUtils.toInt(paramMap.get(QbConstants.ATTR_OPTION_CORRECT));
 
 	TreeSet<QbOption> optionList = new TreeSet<>();
@@ -1086,8 +1093,8 @@
 	    }
 	    option.setDisplayOrder(NumberUtils.toInt(displayOrder));
 
-	    if ((questionType == QbQuestion.TYPE_MULTIPLE_CHOICE)
-		    || (questionType == QbQuestion.TYPE_VERY_SHORT_ANSWERS)) {
+	    if ((questionType == QbQuestion.TYPE_MULTIPLE_CHOICE) || (questionType
+		    == QbQuestion.TYPE_VERY_SHORT_ANSWERS)) {
 		String name = paramMap.get(QbConstants.ATTR_OPTION_NAME_PREFIX + i);
 		if (name == null && isForSaving && !(questionType == QbQuestion.TYPE_VERY_SHORT_ANSWERS && i < 2)) {
 		    continue;
@@ -1204,6 +1211,19 @@
 	return unitList;
     }
 
+    @Override
+    public boolean hasUserAccessToCollection(long collectionUid) {
+	Integer userId = getUserId();
+	if (userId == null) {
+	    return false;
+	}
+	if (securityService.isAppadmin(userId, "access QB collection", true)) {
+	    return true;
+	}
+	Collection<QbCollection> collections = getUserCollections(userId);
+	return collections.stream().map(QbCollection::getUid).anyMatch(uid -> uid.equals(collectionUid));
+    }
+
     private Map<String, String> splitRequestParameter(HttpServletRequest request, String parameterName) {
 	String list = request.getParameter(parameterName);
 	if (list == null) {
@@ -1256,4 +1276,8 @@
     public void setToolService(ILamsToolService toolService) {
 	this.toolService = toolService;
     }
+
+    public void setSecurityService(ISecurityService securityService) {
+	this.securityService = securityService;
+    }
 }
\ No newline at end of file