Make school term query arg optional and adapt to core

person & schoolTerm are both objects now in personOverviewCardMixin.

aleksis/apps/alsijil/frontend/components/coursebook/statistics/StatisticsForPersonCard.vue

@@ -52,6 +52,16 @@ export default {
   apollo: {
     statistics: {
       query: statisticsByPerson,
+      variables() {
+        const term = this.schoolTerm
+              ? { term: this.schoolTerm.id }
+              : {};
+
+        return {
+          person: this.person.id,
+          ...term,
+        };
+      },
     },
   },
 };

aleksis/apps/alsijil/frontend/components/coursebook/statistics/statistics.graphql

@@ -28,7 +28,7 @@ fragment statistics on StatisticsByPersonType {
 
 query statisticsByPerson (
   $person: ID!
-  $term: ID!
+  $term: ID
 ) {
   statistics: statisticsByPerson(
     person: $person
@@ -40,7 +40,7 @@ query statisticsByPerson (
 
 query statisticsByGroup (
   $group: ID!
-  $term: ID!
+  $term: ID
 ) {
   statistics: statisticsByGroup(
     group: $group

aleksis/apps/alsijil/schema/__init__.py

@@ -51,13 +51,13 @@ class Query(graphene.ObjectType):
 
     statistics_by_person = graphene.Field(
         StatisticsByPersonType,
-        person=graphene.ID(),
-        term=graphene.ID(),
+        person=graphene.ID(required=True),
+        term=graphene.ID(required=False),
     )
     statistics_by_group = graphene.List(
         StatisticsByPersonType,
-        group=graphene.ID(),
-        term=graphene.ID(),
+        group=graphene.ID(required=True),
+        term=graphene.ID(required=False),
     )
 
     def resolve_documentations_by_course_id(root, info, course_id, **kwargs):
@@ -183,12 +183,12 @@ class Query(graphene.ObjectType):
         return lessons_for_person
 
     @staticmethod
-    def resolve_statistics_by_person(root, info, person, term):
+    def resolve_statistics_by_person(root, info, person, term=None):
         # TODO: Annotate person with necessary information for term.
         return Person.objects.get(id=person)
 
     @staticmethod
-    def resolve_statistics_by_group(root, info, group, term):
+    def resolve_statistics_by_group(root, info, group, term=None):
         # TODO: Annotate persons with necessary information for term.
         return Group.objects.get(id=group).members.all()