Display point stats in assignment overview

Fixes #131

src/components/Assignments/Assignment/AssignmentSync/AssignmentSync.js

@@ -81,7 +81,7 @@ const getSyncMessages = syncInfo => {
   const res = [];
   for (const field in syncMessages) {
     if (!syncInfo[field]) {
-      console.log('Field ' + field + ' not present.');
+      // console.log('Field ' + field + ' not present.');
       continue;
     }
 

src/components/Assignments/Assignment/AssignmentTableRow/AssignmentTableRow.js

@@ -4,6 +4,7 @@ import { Link } from 'react-router';
 import AssignmentStatusIcon from '../AssignmentStatusIcon/AssignmentStatusIcon';
 import { FormattedDate, FormattedTime } from 'react-intl';
 
+import ResourceRenderer from '../../../helpers/ResourceRenderer';
 import withLinks from '../../../../hoc/withLinks';
 import { LocalizedExerciseName } from '../../../helpers/LocalizedNames';
 import { MaybeBonusAssignmentIcon } from '../../../icons';
@@ -23,6 +24,7 @@ const AssignmentTableRow = ({
   },
   status,
   userId,
+  bestSubmission,
   links: {
     ASSIGNMENT_DETAIL_URI_FACTORY,
     ASSIGNMENT_DETAIL_SPECIFIC_USER_URI_FACTORY
@@ -48,6 +50,25 @@ const AssignmentTableRow = ({
       <td>
         {group}
       </td>}
+    {bestSubmission &&
+      <td>
+        <ResourceRenderer resource={bestSubmission}>
+          {data =>
+            data.lastSubmission
+              ? <span>
+                  {data.lastSubmission.evaluation.points}
+                  {data.bonusPoints > 0 &&
+                    <span style={{ color: 'green' }}>
+                      +{data.bonusPoints}
+                    </span>}
+                  {data.bonusPoints < 0 &&
+                    <span style={{ color: 'red' }}>
+                      {data.bonusPoints}
+                    </span>}/{data.maxPoints}
+                </span>
+              : <span />}
+        </ResourceRenderer>
+      </td>}
     <td>
       <FormattedDate value={new Date(firstDeadline * 1000)} />
       {', '}
@@ -76,6 +97,7 @@ AssignmentTableRow.propTypes = {
   }),
   status: PropTypes.string,
   userId: PropTypes.string,
+  bestSubmission: PropTypes.object,
   links: PropTypes.object
 };
 

src/components/Assignments/Assignment/AssignmentsTable/AssignmentsTable.js

@@ -44,11 +44,25 @@ const compareAssignments = (a, b) => {
   }
 };
 
+const displayPoints = (bestSubmissions, assignments) => {
+  const assignmentIds = assignments
+    .filter(isReady)
+    .map(getJsData)
+    .map(assignment => assignment.id);
+  return Object.keys(bestSubmissions)
+    .filter(key => assignmentIds.indexOf(key) >= 0)
+    .reduce((acc, key) => {
+      acc = acc || bestSubmissions[key];
+      return acc;
+    }, false);
+};
+
 const AssignmentsTable = ({
   assignments = List(),
   statuses = [],
   showGroup = true,
   userId = null,
+  bestSubmissions = {},
   intl: { locale }
 }) =>
   <Table hover>
@@ -68,6 +82,13 @@ const AssignmentsTable = ({
               defaultMessage="Group"
             />
           </th>}
+        {displayPoints(bestSubmissions, assignments) &&
+          <th>
+            <FormattedMessage
+              id="app.assignments.points"
+              defaultMessage="Points"
+            />
+          </th>}
         <th>
           <FormattedMessage
             id="app.assignments.deadline"
@@ -99,6 +120,7 @@ const AssignmentsTable = ({
             showGroup={showGroup}
             status={statuses[assignment.id]}
             locale={locale}
+            bestSubmission={bestSubmissions[assignment.id]}
           />
         )}
     </tbody>
@@ -109,6 +131,7 @@ AssignmentsTable.propTypes = {
   showGroup: PropTypes.bool,
   statuses: PropTypes.oneOfType([PropTypes.object, PropTypes.array]),
   userId: PropTypes.string,
+  bestSubmissions: PropTypes.object,
   intl: PropTypes.shape({ locale: PropTypes.string.isRequired }).isRequired
 };
 

src/components/Groups/StudentsView/StudentsView.js

@@ -14,6 +14,7 @@ const StudentsView = ({
   group,
   statuses = [],
   assignments,
+  bestSubmissions,
   isAdmin = false,
   intl: { locale }
 }) =>
@@ -47,6 +48,7 @@ const StudentsView = ({
             assignments={assignments}
             showGroup={false}
             statuses={statuses}
+            bestSubmissions={bestSubmissions}
           />
         </Box>
       </Col>
@@ -83,6 +85,7 @@ StudentsView.propTypes = {
   assignments: ImmutablePropTypes.list.isRequired,
   statuses: PropTypes.oneOfType([PropTypes.object, PropTypes.array]),
   isAdmin: PropTypes.bool,
+  bestSubmissions: PropTypes.object,
   intl: PropTypes.shape({ locale: PropTypes.string.isRequired }).isRequired
 };
 

src/pages/Group/Group.js

@@ -62,6 +62,8 @@ import { getStatusesForLoggedUser } from '../../redux/selectors/stats';
 import { getLocalizedName } from '../../helpers/getLocalizedData';
 import withLinks from '../../hoc/withLinks';
 import { isReady } from '../../redux/helpers/resourceManager/index';
+import { fetchBestSubmission } from '../../redux/modules/groupResults';
+import { getBestSubmissionsForLoggedInUser } from '../../redux/selectors/groupResults';
 
 class Group extends Component {
   static isAdminOrSupervisorOf = (group, userId) =>
@@ -89,6 +91,13 @@ class Group extends Component {
                 dispatch(fetchStudents(groupId)),
                 dispatch(fetchGroupsStats(groupId))
               ])
+            : Promise.resolve(),
+          group.students.indexOf(userId) >= 0
+            ? Promise.all(
+                group.assignments.all.map(assignmentId =>
+                  dispatch(fetchBestSubmission(userId, assignmentId))
+                )
+              )
             : Promise.resolve()
         ])
       ),
@@ -105,6 +114,7 @@ class Group extends Component {
 
     if (groupId !== newProps.params.groupId) {
       newProps.loadAsync(newProps.userId, newProps.isSuperAdmin);
+      return;
     }
 
     if (isReady(this.props.group) && isReady(newProps.group)) {
@@ -174,6 +184,7 @@ class Group extends Component {
       publicAssignments = List(),
       stats,
       statuses,
+      bestSubmissions,
       isStudent,
       isAdmin,
       isSuperAdmin,
@@ -215,13 +226,13 @@ class Group extends Component {
                   </Button>
                 </LinkContainer>
               </p>}
-
             {(isStudent || isSupervisor || isAdmin || isSuperAdmin) &&
               <StudentsView
                 group={data}
                 stats={stats}
                 statuses={statuses}
                 assignments={publicAssignments}
+                bestSubmissions={bestSubmissions}
                 isAdmin={isAdmin || isSuperAdmin}
               />}
 
@@ -287,6 +298,7 @@ Group.propTypes = {
   loadAsync: PropTypes.func,
   stats: PropTypes.object,
   statuses: PropTypes.oneOfType([PropTypes.object, PropTypes.array]),
+  bestSubmissions: PropTypes.object,
   assignExercise: PropTypes.func.isRequired,
   createGroupExercise: PropTypes.func.isRequired,
   push: PropTypes.func.isRequired,
@@ -311,6 +323,7 @@ const mapStateToProps = (state, { params: { groupId } }) => {
     allAssignments: groupsAllAssignmentsSelector(state, groupId),
     groupExercises: getExercisesForGroup(state, groupId),
     statuses: getStatusesForLoggedUser(state, groupId),
+    bestSubmissions: getBestSubmissionsForLoggedInUser(state),
     supervisors: supervisorsOfGroupSelector(state, groupId),
     students: studentsOfGroupSelector(state, groupId),
     isStudent: isStudentOf(userId, groupId)(state),

src/redux/modules/groupResults.js

@@ -52,21 +52,14 @@ const reducer = handleActions(
 
     [additionalActionTypes.BEST_SUBMISSION_FULFILLED]: (
       state,
-      { payload = {}, meta: { assignmentId } }
+      { payload = {}, meta: { assignmentId, userId } }
     ) =>
-      Object.keys(payload).reduce(
-        (state, userId) =>
-          state
-            .setIn(
-              ['resources', assignmentId, userId, 'data'],
-              fromJS(payload[userId])
-            )
-            .setIn(
-              ['resources', assignmentId, userId, 'state'],
-              resourceStatus.FULFILLED
-            ),
-        state
-      )
+      state
+        .setIn(['resources', assignmentId, userId, 'data'], fromJS(payload))
+        .setIn(
+          ['resources', assignmentId, userId, 'state'],
+          resourceStatus.FULFILLED
+        )
   }),
   initialState
 );

src/redux/selectors/groupResults.js

@@ -1,13 +1,14 @@
 import { createSelector } from 'reselect';
+import { loggedInUserIdSelector } from './auth';
 
 export const getGroupResults = state => state.groupResults;
 export const getBestSubmission = (userId, assignmentId) =>
   createSelector(
     getGroupResults,
     groupResults =>
       groupResults &&
-        groupResults.getIn(['resources', assignmentId]) !== null &&
-        groupResults.getIn(['resources', assignmentId, userId]) !== null
+      groupResults.getIn(['resources', assignmentId]) !== null &&
+      groupResults.getIn(['resources', assignmentId, userId]) !== null
         ? groupResults.getIn(['resources', assignmentId, userId])
         : null
   );
@@ -29,3 +30,16 @@ export const getBestSubmissionsAssoc = (assignments, users) =>
 
     return submissions;
   });
+
+export const getBestSubmissionsForLoggedInUser = createSelector(
+  [getGroupResults, loggedInUserIdSelector],
+  (groupResults, userId) => {
+    const submissions = {};
+    groupResults
+      .get('resources')
+      .forEach(
+        (value, assignmentId) => (submissions[assignmentId] = value.get(userId))
+      );
+    return submissions;
+  }
+);