why does PGProblemEditor.pm check for the modify_student_data permission instead of the modify_problem_sets permission?

[ionparticle]

Hello, is there a special reason PGProblemEditor.pm in several instances checks for the modify_student_data permission instead of the modify_problem_sets permission? For example:

webwork2/lib/WeBWorK/ContentGenerator/Instructor/PGProblemEditor.pm

Lines 135 to 142 in cfb690a

	use constant FORM_PERMS => {
	view => "modify_student_data",
	add_problem => "modify_student_data",
	make_local_copy => "modify_student_data",
	save => "modify_student_data",
	save_as => "modify_student_data",
	revert => "modify_student_data",
	};

webwork2/lib/WeBWorK/ContentGenerator/Instructor/PGProblemEditor.pm

Lines 452 to 453 in cfb690a

	return CGI::div({class=>"ResultsWithError"}, "You are not authorized to modify problems.")
	unless $authz->hasPermissions($user, "modify_student_data");

We're trying to create a user role that can edit content but has no power over students/grades. So we'd like to avoid giving this role the modify_student_data permission. Would it be ok to replace this with modify_problem_sets or perhaps modify_set_def_files or manage_course_files? There's also a modify_problem_template_files that still exists in defaults.config but appears to not be used in code.