If deleting multiple recordings the user right are checked now.

This problem and a patch solving this was reported in bug entry #1317.
a-mair · Apr 4, 2013 · 7d29dbb · 7d29dbb
1 parent c8a125a
commit 7d29dbb
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/pages/recordings.ecpp b/pages/recordings.ecpp
@@ -64,12 +64,17 @@ if (!todel.empty()) {
 	deletions.push_back(todel);
 }
 for (deletions_type::const_iterator it = deletions.begin(); it != deletions.end(); ++it) {
-	RemoveRecordingTask task(*it);
-	LiveTaskManager().Execute(task);
-	if (!task.Result())
-		deleteResult += string() + tr("ERROR:") + " " + task.Error() + "<br/>";
-	else
-		deleteResult += string() + tr("Deleted recording:") + " " + StringReplace(task.RecName(), "~", "/") + "<br/>";
+	if (cUser::CurrentUserHasRightTo(UR_DELRECS)) {
+		RemoveRecordingTask task(*it);
+		LiveTaskManager().Execute(task);
+		if (!task.Result())
+			deleteResult += string() + tr("ERROR:") + " " + task.Error() + "<br/>";
+		else
+			deleteResult += string() + tr("Deleted recording:") + " " + StringReplace(task.RecName(), "~", "/") + "<br/>";
+	}
+	else {
+		throw HtmlError( tr("Sorry, no permission. Please contact your administrator!") );
+	}
 }
 deletions.clear();