add a select-all checkbox to add course page

Alex-Jordan · Dec 19, 2023 · 4d80939 · 4d80939
1 parent 295f12a
commit 4d80939
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 33 deletions.
diff --git a/lib/WeBWorK/ContentGenerator/CourseAdmin.pm b/lib/WeBWorK/ContentGenerator/CourseAdmin.pm
@@ -308,15 +308,6 @@ sub do_add_course ($c) {
 
 	my $copy_from_course = trim_spaces($c->param('copy_from_course')) // '';
 
-	my $copy_templates_html     = $c->param('copy_templates_html')     || '';
-	my $copy_simple_config_file = $c->param('copy_simple_config_file') || '';
-	my $copy_config_file        = $c->param('copy_config_file')        || '';
-	my $copy_non_students       = $c->param('copy_non_students')       || '';
-	my $copy_sets               = $c->param('copy_sets')               || '';
-	my $copy_achievements       = $c->param('copy_achievements')       || '';
-	my $copy_title              = $c->param('copy_title')              || '';
-	my $copy_institution        = $c->param('copy_institution')        || '';
-
 	my $add_dbLayout = trim_spaces($c->param('add_dbLayout')) || '';
 
 	my $ce2 = WeBWorK::CourseEnvironment->new({ courseName => $add_courseID });
@@ -372,15 +363,9 @@ sub do_add_course ($c) {
 	# Include any optional arguments, including a template course and the course title and course institution.
 	my %optional_arguments;
 	if ($copy_from_course ne '') {
-		$optional_arguments{copyFrom}          = $copy_from_course;
-		$optional_arguments{copyTemplatesHtml} = $copy_templates_html;
-		$optional_arguments{copySimpleConfig}  = $copy_simple_config_file;
-		$optional_arguments{copyConfig}        = $copy_config_file;
-		$optional_arguments{copyNonStudents}   = $copy_non_students;
-		$optional_arguments{copySets}          = $copy_sets;
-		$optional_arguments{copyAchievements}  = $copy_achievements;
-		$optional_arguments{copyTitle}         = $copy_title;
-		$optional_arguments{copyInstitution}   = $copy_institution;
+		%optional_arguments             = map { $_ => 1 } ($c->param('copy_component'));
+		$optional_arguments{copyFrom}   = $copy_from_course;
+		$optional_arguments{copyConfig} = $c->param('copy_config_file') || '';
 	}
 	if ($add_courseTitle ne '') {
 		$optional_arguments{courseTitle} = $add_courseTitle;

diff --git a/templates/ContentGenerator/Base/admin_links.html.ep b/templates/ContentGenerator/Base/admin_links.html.ep
@@ -16,13 +16,7 @@
 			% 	{
 			% 		add_admin_users         => 1,
 			% 		copy_from_course        => $ce->{siteDefaults}{default_copy_from_course} || '',
-			% 		copy_templates_html     => 1,
-			% 		copy_simple_config_file => 1,
-			% 		copy_non_students       => 1,
-			% 		copy_sets               => 1,
-			% 		copy_achievements       => 1,
-			% 		copy_title              => 1,
-			% 		copy_institution        => 1,
+			%		copy_component		=> 'copyTemplatesHtml',
 			% 		add_dbLayout            => 'sql_single'
 			% 	}
 			% ],

diff --git a/templates/ContentGenerator/CourseAdmin/add_course_form.html.ep b/templates/ContentGenerator/CourseAdmin/add_course_form.html.ep
@@ -125,49 +125,56 @@
 			</div>
 			<div class="form-check">
 				<label class="form-check-label">
-					<%= check_box 'copy_templates_html' => 1, class => 'form-check-input' =%>
+					<%= check_box select_all => 1, class => 'select-all form-check-input',
+						'data-select-group' => 'copy_component'=%>
+					<%= maketext('select all', 'templates', 'html') =%>
+				</label>
+			</div>
+			<div class="form-check">
+				<label class="form-check-label">
+					<%= check_box copy_component => 'copyTemplatesHtml', class => 'form-check-input' =%>
 					<%= maketext('[_1] and [_2] folders', 'templates', 'html') =%>
 				</label>
 			</div>
 			<div class="form-check">
 				<label class="form-check-label">
-					<%= check_box 'copy_simple_config_file' => 1, class => 'form-check-input' =%>
+					<%= check_box copy_component => 'copySimpleConfig', class => 'form-check-input' =%>
 					<%= maketext('simple configuration file') =%>
 				</label>
 			</div>
 			<div class="form-check">
 				<label class="form-check-label">
-					<%= check_box 'copy_config_file' => 1, class => 'form-check-input' =%>
+					<%= check_box copy_config_file => 1, class => 'form-check-input' =%>
 					<%= maketext('course configuration file') =%>
 				</label>
 			</div>
 			<div class="form-check">
 				<label class="form-check-label">
-					<%= check_box 'copy_non_students' => 1, class => 'form-check-input' =%>
+					<%= check_box copy_component => 'copyNonStudents', class => 'form-check-input' =%>
 					<%= maketext('non-student users') =%>
 				</label>
 			</div>
 			<div class="form-check">
 				<label class="form-check-label">
-					<%= check_box 'copy_sets' => 1, class => 'form-check-input' =%>
+					<%= check_box copy_component => 'copySets', class => 'form-check-input' =%>
 					<%= maketext('assignments/sets') =%>
 				</label>
 			</div>
 			<div class="form-check">
 				<label class="form-check-label">
-					<%= check_box 'copy_achievements' => 1, class => 'form-check-input' =%>
+					<%= check_box copy_component => 'copyAchievements', class => 'form-check-input' =%>
 					<%= maketext('achievements') =%>
 				</label>
 			</div>
 			<div class="form-check">
 				<label class="form-check-label">
-					<%= check_box 'copy_title' => 1, class => 'form-check-input' =%>
+					<%= check_box copy_component => 'copyTitle', class => 'form-check-input' =%>
 					<%= maketext('course title (will override "Course Title" input above)') =%>
 				</label>
 			</div>
 			<div class="form-check">
 				<label class="form-check-label">
-					<%= check_box 'copy_institution' => 1, class => 'form-check-input' =%>
+					<%= check_box copy_component => 'copyInstitution', class => 'form-check-input' =%>
 					<%= maketext('course institution (will override "Institution" input above)') =%>
 				</label>
 			</div>