Ragesoss has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/114476

Change subject: Remove the "field" and "level" options in the course edit view
......................................................................

Remove the "field" and "level" options in the course edit view

Remove "field" (of study) and "level" (of students) from the form
for editing a course. The data for those fields aren't displayed
anywhere, and they are optional, so they don't serve a purpose.
Removing them from the edit interface but leaving the tables alone
will be minimally invasive.

Change-Id: I599044687ea44fb796e644f4da10326728571f78
---
M includes/actions/EditCourseAction.php
1 file changed, 0 insertions(+), 18 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/EducationProgram 
refs/changes/76/114476/1

diff --git a/includes/actions/EditCourseAction.php 
b/includes/actions/EditCourseAction.php
index fc80cb0..18cc446 100644
--- a/includes/actions/EditCourseAction.php
+++ b/includes/actions/EditCourseAction.php
@@ -225,24 +225,6 @@
                                'required' => true,
                        );
 
-                       $fieldFields = $this->table->selectFields( 'field', 
array(), array( 'DISTINCT' ) );
-                       $fieldFields = array_merge( array( '' => '' ), 
$fieldFields );
-                       $fields['field'] = array(
-                               'class' => 'EducationProgram\HTMLCombobox',
-                               'label-message' => 'ep-course-edit-field',
-                               'required' => true,
-                               'options' => array_combine( $fieldFields, 
$fieldFields ),
-                       );
-
-                       $levels = $this->table->selectFields( 'level', array(), 
array( 'DISTINCT' ) );
-                       $levels = array_merge( array( '' => '' ), $levels );
-                       $fields['level'] = array(
-                               'class' => 'EducationProgram\HTMLCombobox',
-                               'label-message' => 'ep-course-edit-level',
-                               'required' => true,
-                               'options' => array_combine( $levels, $levels ),
-                       );
-
                        $langOptions = Utils::getLanguageOptions( 
$this->getLanguage()->getCode() );
                        $fields['lang'] = array(
                                'type' => 'select',

-- 
To view, visit https://gerrit.wikimedia.org/r/114476
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I599044687ea44fb796e644f4da10326728571f78
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/EducationProgram
Gerrit-Branch: master
Gerrit-Owner: Ragesoss <rages...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to