Remove effects

zapisy/apps/effects/models.py

@@ -14,16 +14,3 @@ class CompletedCourses(models.Model):
 
     class Meta:
         unique_together = ('student', 'course', 'program')
-
-    def get_completed_effects(student: Student) -> Set[str]:
-        completed_courses = (
-            CompletedCourses.objects.filter(student=student, program=student.program)
-            .select_related('course').prefetch_related('course__effects')
-        )
-
-        done_effects = set()
-        for record in completed_courses:
-            for effect in record.course.effects.all():
-                done_effects.add(effect.group_name)
-
-        return done_effects

zapisy/apps/enrollment/courses/admin.py

@@ -4,7 +4,6 @@
 from apps.enrollment.courses.models.classroom import Classroom
 from apps.enrollment.courses.models.course_instance import CourseInstance
 from apps.enrollment.courses.models.course_type import Type
-from apps.enrollment.courses.models.effects import Effects
 from apps.enrollment.courses.models.group import Group, GuaranteedSpots
 from apps.enrollment.courses.models.semester import ChangedDay, Freeday, Semester
 from apps.enrollment.courses.models.tag import Tag
@@ -74,26 +73,6 @@ class FreedayAdmin(admin.ModelAdmin):
     ordering = ('-day',)
 
 
-class EffectsListFilter(SimpleListFilter):
-    title = 'Grupa efektów kształcenia'
-
-    # Parameter for the filter that will be used in the URL query.
-    parameter_name = 'effects'
-
-    def lookups(self, request, model_admin):
-        result = []
-        for effect in Effects.objects.all():
-            result.append((str(effect.id), effect))
-
-        return result
-
-    def queryset(self, request, queryset):
-        if self.value():
-            return queryset.filter(effects=self.value())
-        else:
-            return queryset
-
-
 class TermInline(admin.TabularInline):
     model = Term
     extra = 0
@@ -175,7 +154,7 @@ def get_queryset(self, request):
 @admin.register(CourseInstance)
 class CourseInstanceAdmin(admin.ModelAdmin):
     list_filter = ('semester', 'course_type', ('owner', admin.RelatedOnlyFieldListFilter),
-                   'tags', 'effects',)
+                   'tags')
     list_display = ('name', 'owner', 'course_type', 'semester',)
     search_fields = ('name', 'name_en')
     ordering = ('semester', 'owner', 'offer')
@@ -194,7 +173,6 @@ class TypeAdmin(admin.ModelAdmin):
 
 admin.site.register(Group, GroupAdmin)
 admin.site.register(Tag)
-admin.site.register(Effects)
 admin.site.register(Classroom, ClassroomAdmin)
 admin.site.register(Semester, SemesterAdmin)
 admin.site.register(Freeday, FreedayAdmin)

zapisy/apps/enrollment/courses/migrations/0037_auto_20250105_1457.py

@@ -0,0 +1,20 @@
+# Generated by Django 3.1.14 on 2025-01-05 14:57
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('courses', '0036_auto_20211022_1641'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='courseinformation',
+            name='effects',
+        ),
+        migrations.DeleteModel(
+            name='Effects',
+        ),
+    ]

zapisy/apps/enrollment/courses/models/course_information.py

@@ -13,7 +13,6 @@
 from apps.users.models import Employee
 
 from .course_type import Type as CourseType
-from .effects import Effects
 from .tag import Tag
 
 
@@ -73,7 +72,6 @@ class CourseInformation(models.Model):
     discipline = models.CharField("dyscyplina", max_length=100, default="Informatyka")
 
     tags = models.ManyToManyField(Tag, verbose_name="tagi", blank=True)
-    effects = models.ManyToManyField(Effects, verbose_name="grupy efektów kształcenia", blank=True)
 
     created = models.DateTimeField("Data utworzenia", auto_now_add=True)
     modified = models.DateTimeField("Data modyfikacji", auto_now=True)
@@ -120,7 +118,6 @@ def __json__(self):
             'courseType': self.course_type_id,
             'recommendedForFirstYear': self.recommended_for_first_year,
             'owner': self.owner_id,
-            'effects': [effect.pk for effect in self.effects.all()],
             'tags': [tag.pk for tag in self.tags.all()],
         }
 
@@ -130,13 +127,11 @@ def get_short_name(self):
     @staticmethod
     def prepare_filter_data(qs: models.QuerySet) -> Dict:
         """Prepares the data for course filter based on a given queryset."""
-        all_effects = Effects.objects.all().values_list('id', 'group_name', named=True)
         all_tags = Tag.objects.all().values_list('id', 'full_name', named=True)
         all_owners = qs.values_list(
             'owner', 'owner__user__first_name', 'owner__user__last_name', named=True).distinct()
         all_types = qs.values_list('course_type', 'course_type__name', named=True).distinct()
         return {
-            'allEffects': {e.id: e.group_name for e in all_effects},
             'allTags': {t.id: t.full_name for t in all_tags},
             'allOwners': {
                 o.owner: [o.owner__user__first_name, o.owner__user__last_name] for o in all_owners

zapisy/apps/enrollment/courses/models/course_instance.py

@@ -57,7 +57,6 @@ def create_proposal_instance(cls, proposal: Proposal, semester: Semester):
         instance = cls(**proposal_dict)
         instance.save()
         instance.tags.set(proposal.tags.all())
-        instance.effects.set(proposal.effects.all())
         return instance
 
     @classmethod

zapisy/apps/enrollment/courses/templates/courses/course_parts/course_head.html

@@ -76,16 +76,6 @@ <h1 class="d-inline-block">
             </td>
         </tr>
         {% endif %}
-        {% if course.effects.all %}
-        <tr>
-            <th>Grupy efektów kształcenia</th>
-            <td>
-                {% for effect in course.effects.all %}
-                    <span class="badge bg-info me-2">{{ effect }}</span>
-                {% endfor %}
-            </td>
-        </tr>
-        {% endif %}
     </tbody>
 </table>
 

zapisy/apps/enrollment/courses/views.py

@@ -32,7 +32,7 @@ def prepare_courses_list_data(semester: Optional[Semester]):
     """Returns a dict used by course list and filter in various views."""
     qs = CourseInstance.objects.filter(semester=semester).order_by('name')
     courses = []
-    for course in qs.prefetch_related('effects', 'tags'):
+    for course in qs.prefetch_related('tags'):
         course_dict = course.__json__()
         course_dict.update({
             'url': reverse('course-page', args=(course.slug,)),
@@ -68,7 +68,7 @@ def course_view_data(request, slug) -> Tuple[Optional[CourseInstance], Optional[
     course: CourseInstance
     try:
         course = CourseInstance.objects.filter(slug=slug).select_related(
-            'semester', 'course_type').prefetch_related('tags', 'effects').get()
+            'semester', 'course_type').prefetch_related('tags').get()
     except CourseInstance.DoesNotExist:
         return None, None
 
@@ -332,7 +332,7 @@ def get_all_group_ids_for_course_slug(slug, class_type: int = None):
         course = (
             CourseInstance.objects.filter(slug=slug)
             .select_related('semester', 'course_type')
-            .prefetch_related('tags', 'effects')
+            .prefetch_related('tags')
             .get()
         )
     except CourseInstance.DoesNotExist:

zapisy/apps/enrollment/timetable/assets/components/CourseFilter.vue

@@ -19,7 +19,6 @@ export default Vue.extend({
   },
   data: function () {
     return {
-      allEffects: {},
       allTags: {},
       allOwners: [] as MultiselectFilterData<number>,
       allTypes: [] as MultiselectFilterData<number>,
@@ -31,7 +30,6 @@ export default Vue.extend({
     const filtersData = JSON.parse(
       document.getElementById("filters-data")!.innerHTML
     ) as FilterDataJSON;
-    this.allEffects = cloneDeep(filtersData.allEffects);
     this.allTags = cloneDeep(filtersData.allTags);
     this.allOwners = toPairs(filtersData.allOwners)
       .sort(([id, [firstname, lastname]], [id2, [firstname2, lastname2]]) => {
@@ -89,25 +87,16 @@ export default Vue.extend({
           />
         </div>
         <div class="col-md">
-          <MultiSelectFilter
-            filterKey="type-filter"
-            property="courseType"
-            :options="allTypes"
-            title="Rodzaj przedmiotu"
-            placeholder="Wszystkie rodzaje"
-            ref="type-filter"
-          />
-          <hr />
-          <LabelsFilter
-            title="Efekty kształcenia"
-            filterKey="effects-filter"
-            property="effects"
-            :allLabels="allEffects"
-            onClass="bg-info"
-            ref="effects-filter"
-          />
-        </div>
-        <div class="col-md">
+          <div class="col-md">
+            <MultiSelectFilter
+              filterKey="type-filter"
+              property="courseType"
+              :options="allTypes"
+              title="Rodzaj przedmiotu"
+              placeholder="Wszystkie rodzaje"
+              ref="type-filter"
+            />
+          </div>
           <MultiSelectFilter
             filterKey="owner-filter"
             property="owner"

zapisy/apps/enrollment/timetable/assets/models.ts

@@ -195,7 +195,6 @@ export interface PersonDict {
 }
 
 export interface FilterDataJSON {
-  allEffects: KVDict;
   allTags: KVDict;
   allOwners: PersonDict;
   allTypes: KVDict;

zapisy/apps/enrollment/timetable/assets/store/courses.ts

@@ -15,7 +15,6 @@ export interface CourseInfo {
   id: number;
   name: string;
   url: string;
-  effects: Array<number>;
   tags: Array<number>;
   owner: number;
   recommendedForFirstYear: boolean;

zapisy/apps/enrollment/timetable/views.py

@@ -73,7 +73,7 @@ def list_courses_in_semester(semester: Semester):
 
     This list will be used in prototype.
     """
-    qs = CourseInstance.objects.filter(semester=semester).prefetch_related('effects', 'tags')
+    qs = CourseInstance.objects.filter(semester=semester).prefetch_related('tags')
     courses = []
     for course in qs:
         course_dict = course.__json__()

zapisy/apps/offer/proposal/admin.py

@@ -11,7 +11,7 @@
 @admin.register(Proposal)
 class ProposalAdmin(admin.ModelAdmin):
     list_filter = ('status', 'semester', 'course_type', ('owner', admin.RelatedOnlyFieldListFilter),
-                   'modified', 'tags', 'effects', ('courseinstance__semester',
+                   'modified', 'tags', ('courseinstance__semester',
                                                    admin.RelatedOnlyFieldListFilter))
     list_display = ('name', 'owner', 'course_type', 'semester', 'status', 'modified',
                     'last_semester')

zapisy/apps/offer/proposal/assets/components/CourseFilter.vue

@@ -22,7 +22,6 @@ export default Vue.extend({
   },
   data: function () {
     return {
-      allEffects: {},
       allTags: {},
       allOwners: [] as MultiselectFilterData<number>,
       allSemesters: [] as MultiselectFilterData<string>,
@@ -36,7 +35,6 @@ export default Vue.extend({
     const filtersData = JSON.parse(
       document.getElementById("filters-data")!.innerHTML
     ) as FilterDataJSON;
-    this.allEffects = cloneDeep(filtersData.allEffects);
     this.allTags = cloneDeep(filtersData.allTags);
     this.allOwners = toPairs(filtersData.allOwners)
       .sort(([id, [firstname, lastname]], [id2, [firstname2, lastname2]]) => {
@@ -104,25 +102,16 @@ export default Vue.extend({
           />
         </div>
         <div class="col-md">
-          <MultiSelectFilter
-            filterKey="type-filter"
-            property="courseType"
-            :options="allTypes"
-            title="Rodzaj przedmiotu"
-            placeholder="Wszystkie rodzaje"
-            ref="type-filter"
-          />
-          <hr />
-          <LabelsFilter
-            title="Efekty kształcenia"
-            filterKey="effects-filter"
-            property="effects"
-            :allLabels="allEffects"
-            onClass="bg-info"
-            ref="effects-filter"
-          />
-        </div>
-        <div class="col-md">
+          <div class="col-md">
+            <MultiSelectFilter
+              filterKey="type-filter"
+              property="courseType"
+              :options="allTypes"
+              title="Rodzaj przedmiotu"
+              placeholder="Wszystkie rodzaje"
+              ref="type-filter"
+            />
+          </div>
           <MultiSelectFilter
             filterKey="owner-filter"
             property="owner"

zapisy/apps/offer/proposal/templates/proposal/proposal_info.html

@@ -69,16 +69,6 @@
             </td>
         </tr>
         {% endif %}
-        {% if course.effects.all %}
-        <tr>
-            <th>Grupy efektów kształcenia</th>
-            <td>
-                {% for effect in course.effects.all %}
-                    <span class="badge bg-info me-2">{{ effect }}</span>
-                {% endfor %}
-            </td>
-        </tr>
-        {% endif %}
     </tbody>
 </table>
 

zapisy/apps/offer/proposal/views.py

@@ -27,7 +27,7 @@ def offer(request, slug=None):
     filter_statuses = [ProposalStatus.IN_OFFER, ProposalStatus.IN_VOTE, ProposalStatus.WITHDRAWN]
     qs = Proposal.objects.filter(status__in=filter_statuses).order_by('name')
     proposal_list = []
-    for p in qs.prefetch_related('effects', 'tags'):
+    for p in qs.prefetch_related('tags'):
         proposal_dict = p.__json__()
         proposal_dict.update({
             'status': ProposalStatus(p.status)._name_,