~azzar1/unity/add-show-desktop-key

« back to all changes in this revision

Viewing changes to ivle/database.py

  • Committer: William Grant
  • Date: 2010-05-07 07:16:52 UTC
  • Revision ID: grantw@unimelb.edu.au-20100507071652-4z2zidt9le05ueix
Add unique indices on assessed(loginid, projectid) and assessed(groupid, projectid), where they should have been from the start.

Show diffs side-by-side

added added

removed removed

Lines of Context:
17
17
 
18
18
# Author: Matt Giuca, Will Grant
19
19
 
20
 
"""
21
 
Database Classes and Utilities for Storm ORM
 
20
"""Database utilities and content classes.
22
21
 
23
22
This module provides all of the classes which map to database tables.
24
23
It also provides miscellaneous utility functions for database interaction.
26
25
 
27
26
import hashlib
28
27
import datetime
 
28
import os
 
29
import urlparse
29
30
 
30
31
from storm.locals import create_database, Store, Int, Unicode, DateTime, \
31
32
                         Reference, ReferenceSet, Bool, Storm, Desc
 
33
from storm.expr import Select, Max
32
34
from storm.exceptions import NotOneError, IntegrityError
33
35
 
34
 
import ivle.conf
35
36
from ivle.worksheet.rst import rst
36
37
 
37
38
__all__ = ['get_store',
51
52
                % (self.__class__.__name__, k))
52
53
        setattr(self, k, v)
53
54
 
54
 
def get_conn_string():
55
 
    """
56
 
    Returns the Storm connection string, generated from the conf file.
 
55
def get_conn_string(config):
 
56
    """Create a Storm connection string to the IVLE database
 
57
 
 
58
    @param config: The IVLE configuration.
57
59
    """
58
60
 
59
61
    clusterstr = ''
60
 
    if ivle.conf.db_user:
61
 
        clusterstr += ivle.conf.db_user
62
 
        if ivle.conf.db_password:
63
 
            clusterstr += ':' + ivle.conf.db_password
 
62
    if config['database']['username']:
 
63
        clusterstr += config['database']['username']
 
64
        if config['database']['password']:
 
65
            clusterstr += ':' + config['database']['password']
64
66
        clusterstr += '@'
65
67
 
66
 
    host = ivle.conf.db_host or 'localhost'
67
 
    port = ivle.conf.db_port or 5432
 
68
    host = config['database']['host'] or 'localhost'
 
69
    port = config['database']['port'] or 5432
68
70
 
69
71
    clusterstr += '%s:%d' % (host, port)
70
72
 
71
 
    return "postgres://%s/%s" % (clusterstr, ivle.conf.db_dbname)
72
 
 
73
 
def get_store():
74
 
    """
75
 
    Open a database connection and transaction. Return a storm.store.Store
76
 
    instance connected to the configured IVLE database.
77
 
    """
78
 
    return Store(create_database(get_conn_string()))
 
73
    return "postgres://%s/%s" % (clusterstr, config['database']['name'])
 
74
 
 
75
def get_store(config):
 
76
    """Create a Storm store connected to the IVLE database.
 
77
 
 
78
    @param config: The IVLE configuration.
 
79
    """
 
80
    return Store(create_database(get_conn_string(config)))
79
81
 
80
82
# USERS #
81
83
 
82
84
class User(Storm):
83
 
    """
84
 
    Represents an IVLE user.
85
 
    """
 
85
    """An IVLE user account."""
86
86
    __storm_table__ = "login"
87
87
 
88
88
    id = Int(primary=True, name="loginid")
119
119
 
120
120
    @property
121
121
    def display_name(self):
 
122
        """Returns the "nice name" of the user or group."""
122
123
        return self.fullname
123
124
 
124
125
    @property
 
126
    def short_name(self):
 
127
        """Returns the database "identifier" name of the user or group."""
 
128
        return self.login
 
129
 
 
130
    @property
125
131
    def password_expired(self):
126
132
        fieldval = self.pass_exp
127
133
        return fieldval is not None and datetime.datetime.now() > fieldval
164
170
 
165
171
    # TODO: Invitations should be listed too?
166
172
    def get_groups(self, offering=None):
 
173
        """Get groups of which this user is a member.
 
174
 
 
175
        @param offering: An optional offering to restrict the search to.
 
176
        """
167
177
        preds = [
168
178
            ProjectGroupMembership.user_id == self.id,
169
179
            ProjectGroup.id == ProjectGroupMembership.project_group_id,
190
200
        return self._get_enrolments(False) 
191
201
 
192
202
    def get_projects(self, offering=None, active_only=True):
193
 
        '''Return Projects that the user can submit.
 
203
        """Find projects that the user can submit.
194
204
 
195
205
        This will include projects for offerings in which the user is
196
206
        enrolled, as long as the project is not in a project set which has
197
207
        groups (ie. if maximum number of group members is 0).
198
208
 
199
 
        Unless active_only is False, only projects for active offerings will
200
 
        be returned.
201
 
 
202
 
        If an offering is specified, returned projects will be limited to
203
 
        those for that offering.
204
 
        '''
 
209
        @param active_only: Whether to only search active offerings.
 
210
        @param offering: An optional offering to restrict the search to.
 
211
        """
205
212
        return Store.of(self).find(Project,
206
213
            Project.project_set_id == ProjectSet.id,
207
214
            ProjectSet.max_students_per_group == None,
210
217
            Semester.id == Offering.semester_id,
211
218
            (not active_only) or (Semester.state == u'current'),
212
219
            Enrolment.offering_id == Offering.id,
213
 
            Enrolment.user_id == self.id)
 
220
            Enrolment.user_id == self.id,
 
221
            Enrolment.active == True)
214
222
 
215
223
    @staticmethod
216
224
    def hash_password(password):
 
225
        """Hash a password with MD5."""
217
226
        return hashlib.md5(password).hexdigest()
218
227
 
219
228
    @classmethod
220
229
    def get_by_login(cls, store, login):
221
 
        """
222
 
        Get the User from the db associated with a given store and
223
 
        login.
224
 
        """
 
230
        """Find a user in a store by login name."""
225
231
        return store.find(cls, cls.login == unicode(login)).one()
226
232
 
227
 
    def get_permissions(self, user):
 
233
    def get_svn_url(self, config, req):
 
234
        """Get the subversion repository URL for this user or group."""
 
235
        login = req.user.login
 
236
        url = urlparse.urlsplit(config['urls']['svn_addr'])
 
237
        url = urlparse.urlunsplit(url[:1] + (login+'@'+url[1],) + url[2:])
 
238
        path = 'users/%s' % self.login
 
239
        return urlparse.urljoin(url, path)
 
240
 
 
241
    def get_permissions(self, user, config):
 
242
        """Determine privileges held by a user over this object.
 
243
 
 
244
        If the user requesting privileges is this user or an admin,
 
245
        they may do everything. Otherwise they may do nothing.
 
246
        """
228
247
        if user and user.admin or user is self:
229
 
            return set(['view', 'edit', 'submit_project'])
 
248
            return set(['view_public', 'view', 'edit', 'submit_project'])
230
249
        else:
231
 
            return set()
 
250
            return set(['view_public'])
232
251
 
233
252
# SUBJECTS AND ENROLMENTS #
234
253
 
235
254
class Subject(Storm):
 
255
    """A subject (or course) which is run in some semesters."""
 
256
 
236
257
    __storm_table__ = "subject"
237
258
 
238
259
    id = Int(primary=True, name="subjectid")
239
260
    code = Unicode(name="subj_code")
240
261
    name = Unicode(name="subj_name")
241
262
    short_name = Unicode(name="subj_short_name")
242
 
    url = Unicode()
243
263
 
244
264
    offerings = ReferenceSet(id, 'Offering.subject_id')
245
265
 
248
268
    def __repr__(self):
249
269
        return "<%s '%s'>" % (type(self).__name__, self.short_name)
250
270
 
251
 
    def get_permissions(self, user):
 
271
    def get_permissions(self, user, config):
 
272
        """Determine privileges held by a user over this object.
 
273
 
 
274
        If the user requesting privileges is an admin, they may edit.
 
275
        Otherwise they may only read.
 
276
        """
252
277
        perms = set()
253
278
        if user is not None:
254
279
            perms.add('view')
257
282
        return perms
258
283
 
259
284
    def active_offerings(self):
260
 
        """Return a sequence of currently active offerings for this subject
 
285
        """Find active offerings for this subject.
 
286
 
 
287
        Return a sequence of currently active offerings for this subject
261
288
        (offerings whose semester.state is "current"). There should be 0 or 1
262
289
        elements in this sequence, but it's possible there are more.
263
290
        """
265
292
                                   Semester.state == u'current')
266
293
 
267
294
    def offering_for_semester(self, year, semester):
268
 
        """Get the offering for the given year/semester, or None."""
 
295
        """Get the offering for the given year/semester, or None.
 
296
 
 
297
        @param year: A string representation of the year.
 
298
        @param semester: A string representation of the semester.
 
299
        """
269
300
        return self.offerings.find(Offering.semester_id == Semester.id,
270
301
                               Semester.year == unicode(year),
271
302
                               Semester.semester == unicode(semester)).one()
272
303
 
273
304
class Semester(Storm):
 
305
    """A semester in which subjects can be run."""
 
306
 
274
307
    __storm_table__ = "semester"
275
308
 
276
309
    id = Int(primary=True, name="semesterid")
290
323
        return "<%s %s/%s>" % (type(self).__name__, self.year, self.semester)
291
324
 
292
325
class Offering(Storm):
 
326
    """An offering of a subject in a particular semester."""
 
327
 
293
328
    __storm_table__ = "offering"
294
329
 
295
330
    id = Int(primary=True, name="offeringid")
297
332
    subject = Reference(subject_id, Subject.id)
298
333
    semester_id = Int(name="semesterid")
299
334
    semester = Reference(semester_id, Semester.id)
 
335
    description = Unicode()
 
336
    url = Unicode()
 
337
    show_worksheet_marks = Bool()
 
338
    worksheet_cutoff = DateTime()
300
339
    groups_student_permissions = Unicode()
301
340
 
302
341
    enrolments = ReferenceSet(id, 'Enrolment.offering_id')
305
344
                           'Enrolment.user_id',
306
345
                           'User.id')
307
346
    project_sets = ReferenceSet(id, 'ProjectSet.offering_id')
 
347
    projects = ReferenceSet(id,
 
348
                            'ProjectSet.offering_id',
 
349
                            'ProjectSet.id',
 
350
                            'Project.project_set_id')
308
351
 
309
352
    worksheets = ReferenceSet(id, 
310
353
        'Worksheet.offering_id', 
318
361
                                  self.semester)
319
362
 
320
363
    def enrol(self, user, role=u'student'):
321
 
        '''Enrol a user in this offering.'''
 
364
        """Enrol a user in this offering.
 
365
 
 
366
        Enrolments handle both the staff and student cases. The role controls
 
367
        the privileges granted by this enrolment.
 
368
        """
322
369
        enrolment = Store.of(self).find(Enrolment,
323
370
                               Enrolment.user_id == user.id,
324
371
                               Enrolment.offering_id == self.id).one()
337
384
                               Enrolment.offering_id == self.id).one()
338
385
        Store.of(enrolment).remove(enrolment)
339
386
 
340
 
    def get_permissions(self, user):
 
387
    def get_permissions(self, user, config):
341
388
        perms = set()
342
389
        if user is not None:
343
390
            enrolment = self.get_enrolment(user)
344
391
            if enrolment or user.admin:
345
392
                perms.add('view')
346
 
            if (enrolment and enrolment.role in (u'tutor', u'lecturer')) \
347
 
               or user.admin:
348
 
                perms.add('edit')
 
393
            if enrolment and enrolment.role == u'tutor':
 
394
                perms.add('view_project_submissions')
 
395
                # Site-specific policy on the role of tutors
 
396
                if config['policy']['tutors_can_enrol_students']:
 
397
                    perms.add('enrol')
 
398
                    perms.add('enrol_student')
 
399
                if config['policy']['tutors_can_edit_worksheets']:
 
400
                    perms.add('edit_worksheets')
 
401
                if config['policy']['tutors_can_admin_groups']:
 
402
                    perms.add('admin_groups')
 
403
            if (enrolment and enrolment.role in (u'lecturer')) or user.admin:
 
404
                perms.add('view_project_submissions')
 
405
                perms.add('admin_groups')
 
406
                perms.add('edit_worksheets')
 
407
                perms.add('view_worksheet_marks')
 
408
                perms.add('edit')           # Can edit projects & details
 
409
                perms.add('enrol')          # Can see enrolment screen at all
 
410
                perms.add('enrol_student')  # Can enrol students
 
411
                perms.add('enrol_tutor')    # Can enrol tutors
 
412
            if user.admin:
 
413
                perms.add('enrol_lecturer') # Can enrol lecturers
349
414
        return perms
350
415
 
351
416
    def get_enrolment(self, user):
 
417
        """Find the user's enrolment in this offering."""
352
418
        try:
353
419
            enrolment = self.enrolments.find(user=user).one()
354
420
        except NotOneError:
356
422
 
357
423
        return enrolment
358
424
 
 
425
    def get_members_by_role(self, role):
 
426
        return Store.of(self).find(User,
 
427
                Enrolment.user_id == User.id,
 
428
                Enrolment.offering_id == self.id,
 
429
                Enrolment.role == role
 
430
                ).order_by(User.login)
 
431
 
 
432
    @property
 
433
    def students(self):
 
434
        return self.get_members_by_role(u'student')
 
435
 
 
436
    def get_open_projects_for_user(self, user):
 
437
        """Find all projects currently open to submissions by a user."""
 
438
        # XXX: Respect extensions.
 
439
        return self.projects.find(Project.deadline > datetime.datetime.now())
 
440
 
 
441
    def has_worksheet_cutoff_passed(self, user):
 
442
        """Check whether the worksheet cutoff has passed.
 
443
        A user is required, in case we support extensions.
 
444
        """
 
445
        if self.worksheet_cutoff is None:
 
446
            return False
 
447
        else:
 
448
            return self.worksheet_cutoff < datetime.datetime.now()
 
449
 
 
450
    def clone_worksheets(self, source):
 
451
        """Clone all worksheets from the specified source to this offering."""
 
452
        import ivle.worksheet.utils
 
453
        for worksheet in source.worksheets:
 
454
            newws = Worksheet()
 
455
            newws.seq_no = worksheet.seq_no
 
456
            newws.identifier = worksheet.identifier
 
457
            newws.name = worksheet.name
 
458
            newws.assessable = worksheet.assessable
 
459
            newws.published = worksheet.published
 
460
            newws.data = worksheet.data
 
461
            newws.format = worksheet.format
 
462
            newws.offering = self
 
463
            Store.of(self).add(newws)
 
464
            ivle.worksheet.utils.update_exerciselist(newws)
 
465
 
 
466
 
359
467
class Enrolment(Storm):
 
468
    """An enrolment of a user in an offering.
 
469
 
 
470
    This represents the roles of both staff and students.
 
471
    """
 
472
 
360
473
    __storm_table__ = "enrolment"
361
474
    __storm_primary__ = "user_id", "offering_id"
362
475
 
382
495
        return "<%s %r in %r>" % (type(self).__name__, self.user,
383
496
                                  self.offering)
384
497
 
 
498
    def get_permissions(self, user, config):
 
499
        # A user can edit any enrolment that they could have created.
 
500
        perms = set()
 
501
        if ('enrol_' + str(self.role)) in self.offering.get_permissions(
 
502
            user, config):
 
503
            perms.add('edit')
 
504
        return perms
 
505
 
 
506
    def delete(self):
 
507
        """Delete this enrolment."""
 
508
        Store.of(self).remove(self)
 
509
 
 
510
 
385
511
# PROJECTS #
386
512
 
387
513
class ProjectSet(Storm):
 
514
    """A set of projects that share common groups.
 
515
 
 
516
    Each student project group is attached to a project set. The group is
 
517
    valid for all projects in the group's set.
 
518
    """
 
519
 
388
520
    __storm_table__ = "project_set"
389
521
 
390
522
    id = Int(name="projectsetid", primary=True)
401
533
        return "<%s %d in %r>" % (type(self).__name__, self.id,
402
534
                                  self.offering)
403
535
 
 
536
    def get_permissions(self, user, config):
 
537
        return self.offering.get_permissions(user, config)
 
538
 
 
539
    def get_groups_for_user(self, user):
 
540
        """List all groups in this offering of which the user is a member."""
 
541
        assert self.is_group
 
542
        return Store.of(self).find(
 
543
            ProjectGroup,
 
544
            ProjectGroupMembership.user_id == user.id,
 
545
            ProjectGroupMembership.project_group_id == ProjectGroup.id,
 
546
            ProjectGroup.project_set_id == self.id)
 
547
 
 
548
    def get_submission_principal(self, user):
 
549
        """Get the principal on behalf of which the user can submit.
 
550
 
 
551
        If this is a solo project set, the given user is returned. If
 
552
        the user is a member of exactly one group, all the group is
 
553
        returned. Otherwise, None is returned.
 
554
        """
 
555
        if self.is_group:
 
556
            groups = self.get_groups_for_user(user)
 
557
            if groups.count() == 1:
 
558
                return groups.one()
 
559
            else:
 
560
                return None
 
561
        else:
 
562
            return user
 
563
 
 
564
    @property
 
565
    def is_group(self):
 
566
        return self.max_students_per_group is not None
 
567
 
 
568
    @property
 
569
    def assigned(self):
 
570
        """Get the entities (groups or users) assigned to submit this project.
 
571
 
 
572
        This will be a Storm ResultSet.
 
573
        """
 
574
        #If its a solo project, return everyone in offering
 
575
        if self.is_group:
 
576
            return self.project_groups
 
577
        else:
 
578
            return self.offering.students
 
579
 
 
580
class DeadlinePassed(Exception):
 
581
    """An exception indicating that a project cannot be submitted because the
 
582
    deadline has passed."""
 
583
    def __init__(self):
 
584
        pass
 
585
    def __str__(self):
 
586
        return "The project deadline has passed"
 
587
 
404
588
class Project(Storm):
 
589
    """A student project for which submissions can be made."""
 
590
 
405
591
    __storm_table__ = "project"
406
592
 
407
593
    id = Int(name="projectid", primary=True)
425
611
        return "<%s '%s' in %r>" % (type(self).__name__, self.short_name,
426
612
                                  self.project_set.offering)
427
613
 
428
 
    def can_submit(self, principal):
 
614
    def can_submit(self, principal, user):
429
615
        return (self in principal.get_projects() and
430
 
                self.deadline > datetime.datetime.now())
 
616
                not self.has_deadline_passed(user))
431
617
 
432
618
    def submit(self, principal, path, revision, who):
433
619
        """Submit a Subversion path and revision to a project.
434
620
 
435
 
        'principal' is the owner of the Subversion repository, and the
436
 
        entity on behalf of whom the submission is being made. 'path' is
437
 
        a path within that repository, and 'revision' specifies which
438
 
        revision of that path. 'who' is the person making the submission.
 
621
        @param principal: The owner of the Subversion repository, and the
 
622
                          entity on behalf of whom the submission is being made
 
623
        @param path: A path within that repository to submit.
 
624
        @param revision: The revision of that path to submit.
 
625
        @param who: The user who is actually making the submission.
439
626
        """
440
627
 
441
 
        if not self.can_submit(principal):
442
 
            raise Exception('cannot submit')
 
628
        if not self.can_submit(principal, who):
 
629
            raise DeadlinePassed()
443
630
 
444
631
        a = Assessed.get(Store.of(self), principal, self)
445
632
        ps = ProjectSubmission()
446
 
        ps.path = path
 
633
        # Raise SubmissionError if the path is illegal
 
634
        ps.path = ProjectSubmission.test_and_normalise_path(path)
447
635
        ps.revision = revision
448
636
        ps.date_submitted = datetime.datetime.now()
449
637
        ps.assessed = a
451
639
 
452
640
        return ps
453
641
 
 
642
    def get_permissions(self, user, config):
 
643
        return self.project_set.offering.get_permissions(user, config)
 
644
 
 
645
    @property
 
646
    def latest_submissions(self):
 
647
        """Return the latest submission for each Assessed."""
 
648
        return Store.of(self).find(ProjectSubmission,
 
649
            Assessed.project_id == self.id,
 
650
            ProjectSubmission.assessed_id == Assessed.id,
 
651
            ProjectSubmission.date_submitted == Select(
 
652
                    Max(ProjectSubmission.date_submitted),
 
653
                    ProjectSubmission.assessed_id == Assessed.id,
 
654
                    tables=ProjectSubmission
 
655
            )
 
656
        )
 
657
 
 
658
    def has_deadline_passed(self, user):
 
659
        """Check whether the deadline has passed."""
 
660
        # XXX: Need to respect extensions.
 
661
        return self.deadline < datetime.datetime.now()
 
662
 
 
663
    def get_submissions_for_principal(self, principal):
 
664
        """Fetch a ResultSet of all submissions by a particular principal."""
 
665
        assessed = Assessed.get(Store.of(self), principal, self)
 
666
        if assessed is None:
 
667
            return
 
668
        return assessed.submissions
 
669
 
 
670
    @property
 
671
    def can_delete(self):
 
672
        """Can only delete if there are no submissions."""
 
673
        return self.submissions.count() == 0
 
674
 
 
675
    def delete(self):
 
676
        """Delete the project. Fails if can_delete is False."""
 
677
        if not self.can_delete:
 
678
            raise IntegrityError()
 
679
        for assessed in self.assesseds:
 
680
            assessed.delete()
 
681
        Store.of(self).remove(self)
454
682
 
455
683
class ProjectGroup(Storm):
 
684
    """A group of students working together on a project."""
 
685
 
456
686
    __storm_table__ = "project_group"
457
687
 
458
688
    id = Int(name="groupid", primary=True)
477
707
 
478
708
    @property
479
709
    def display_name(self):
480
 
        return '%s (%s)' % (self.nick, self.name)
 
710
        """Returns the "nice name" of the user or group."""
 
711
        return self.nick
 
712
 
 
713
    @property
 
714
    def short_name(self):
 
715
        """Returns the database "identifier" name of the user or group."""
 
716
        return self.name
481
717
 
482
718
    def get_projects(self, offering=None, active_only=True):
483
 
        '''Return Projects that the group can submit.
 
719
        '''Find projects that the group can submit.
484
720
 
485
721
        This will include projects in the project set which owns this group,
486
722
        unless the project set disallows groups (in which case none will be
487
723
        returned).
488
724
 
489
 
        Unless active_only is False, projects will only be returned if the
490
 
        group's offering is active.
491
 
 
492
 
        If an offering is specified, projects will only be returned if it
493
 
        matches the group's.
 
725
        @param active_only: Whether to only search active offerings.
 
726
        @param offering: An optional offering to restrict the search to.
494
727
        '''
495
728
        return Store.of(self).find(Project,
496
729
            Project.project_set_id == ProjectSet.id,
501
734
            Semester.id == Offering.semester_id,
502
735
            (not active_only) or (Semester.state == u'current'))
503
736
 
 
737
    def get_svn_url(self, config, req):
 
738
        """Get the subversion repository URL for this user or group."""
 
739
        login = req.user.login
 
740
        url = urlparse.urlsplit(config['urls']['svn_addr'])
 
741
        url = urlparse.urlunsplit(url[:1] + (login+'@'+url[1],) + url[2:])
 
742
        path = 'groups/%s_%s_%s_%s' % (
 
743
                self.project_set.offering.subject.short_name,
 
744
                self.project_set.offering.semester.year,
 
745
                self.project_set.offering.semester.semester,
 
746
                self.name
 
747
                )
 
748
        return urlparse.urljoin(url, path)
504
749
 
505
 
    def get_permissions(self, user):
 
750
    def get_permissions(self, user, config):
506
751
        if user.admin or user in self.members:
507
752
            return set(['submit_project'])
508
753
        else:
509
754
            return set()
510
755
 
511
756
class ProjectGroupMembership(Storm):
 
757
    """A student's membership in a project group."""
 
758
 
512
759
    __storm_table__ = "group_member"
513
760
    __storm_primary__ = "user_id", "project_group_id"
514
761
 
524
771
                                  self.project_group)
525
772
 
526
773
class Assessed(Storm):
 
774
    """A composite of a user or group combined with a project.
 
775
 
 
776
    Each project submission and extension refers to an Assessed. It is the
 
777
    sole specifier of the repository and project.
 
778
    """
 
779
 
527
780
    __storm_table__ = "assessed"
528
781
 
529
782
    id = Int(name="assessedid", primary=True)
536
789
    project = Reference(project_id, Project.id)
537
790
 
538
791
    extensions = ReferenceSet(id, 'ProjectExtension.assessed_id')
539
 
    submissions = ReferenceSet(id, 'ProjectSubmission.assessed_id')
 
792
    submissions = ReferenceSet(
 
793
        id, 'ProjectSubmission.assessed_id', order_by='date_submitted')
540
794
 
541
795
    def __repr__(self):
542
796
        return "<%s %r in %r>" % (type(self).__name__,
543
797
            self.user or self.project_group, self.project)
544
798
 
 
799
    @property
 
800
    def is_group(self):
 
801
        """True if the Assessed is a group, False if it is a user."""
 
802
        return self.project_group is not None
 
803
 
 
804
    @property
 
805
    def principal(self):
 
806
        return self.project_group or self.user
 
807
 
 
808
    @property
 
809
    def checkout_location(self):
 
810
        """Returns the location of the Subversion workspace for this piece of
 
811
        assessment, relative to each group member's home directory."""
 
812
        subjectname = self.project.project_set.offering.subject.short_name
 
813
        if self.is_group:
 
814
            checkout_dir_name = self.principal.short_name
 
815
        else:
 
816
            checkout_dir_name = "mywork"
 
817
        return subjectname + "/" + checkout_dir_name
 
818
 
545
819
    @classmethod
546
820
    def get(cls, store, principal, project):
 
821
        """Find or create an Assessed for the given user or group and project.
 
822
 
 
823
        @param principal: The user or group.
 
824
        @param project: The project.
 
825
        """
547
826
        t = type(principal)
548
827
        if t not in (User, ProjectGroup):
549
828
            raise AssertionError('principal must be User or ProjectGroup')
551
830
        a = store.find(cls,
552
831
            (t is User) or (cls.project_group_id == principal.id),
553
832
            (t is ProjectGroup) or (cls.user_id == principal.id),
554
 
            Project.id == project.id).one()
 
833
            cls.project_id == project.id).one()
555
834
 
556
835
        if a is None:
557
836
            a = cls()
564
843
 
565
844
        return a
566
845
 
 
846
    def delete(self):
 
847
        """Delete the assessed. Fails if there are any submissions. Deletes
 
848
        extensions."""
 
849
        if self.submissions.count() > 0:
 
850
            raise IntegrityError()
 
851
        for extension in self.extensions:
 
852
            extension.delete()
 
853
        Store.of(self).remove(self)
567
854
 
568
855
class ProjectExtension(Storm):
 
856
    """An extension granted to a user or group on a particular project.
 
857
 
 
858
    The user or group and project are specified by the Assessed.
 
859
    """
 
860
 
569
861
    __storm_table__ = "project_extension"
570
862
 
571
863
    id = Int(name="extensionid", primary=True)
576
868
    approver = Reference(approver_id, User.id)
577
869
    notes = Unicode()
578
870
 
 
871
    def delete(self):
 
872
        """Delete the extension."""
 
873
        Store.of(self).remove(self)
 
874
 
 
875
class SubmissionError(Exception):
 
876
    """Denotes a validation error during submission."""
 
877
    pass
 
878
 
579
879
class ProjectSubmission(Storm):
 
880
    """A submission from a user or group repository to a particular project.
 
881
 
 
882
    The content of a submission is a single path and revision inside a
 
883
    repository. The repository is that owned by the submission's user and
 
884
    group, while the path and revision are explicit.
 
885
 
 
886
    The user or group and project are specified by the Assessed.
 
887
    """
 
888
 
580
889
    __storm_table__ = "project_submission"
581
890
 
582
891
    id = Int(name="submissionid", primary=True)
588
897
    submitter = Reference(submitter_id, User.id)
589
898
    date_submitted = DateTime()
590
899
 
 
900
    def get_verify_url(self, user):
 
901
        """Get the URL for verifying this submission, within the account of
 
902
        the given user."""
 
903
        # If this is a solo project, then self.path will be prefixed with the
 
904
        # subject name. Remove the first path segment.
 
905
        submitpath = self.path[1:] if self.path[:1] == '/' else self.path
 
906
        if not self.assessed.is_group:
 
907
            if '/' in submitpath:
 
908
                submitpath = submitpath.split('/', 1)[1]
 
909
            else:
 
910
                submitpath = ''
 
911
        return "/files/%s/%s/%s?r=%d" % (user.login,
 
912
            self.assessed.checkout_location, submitpath, self.revision)
 
913
 
 
914
    @staticmethod
 
915
    def test_and_normalise_path(path):
 
916
        """Test that path is valid, and normalise it. This prevents possible
 
917
        injections using malicious paths.
 
918
        Returns the updated path, if successful.
 
919
        Raises SubmissionError if invalid.
 
920
        """
 
921
        # Ensure the path is absolute to prevent being tacked onto working
 
922
        # directories.
 
923
        # Prevent '\n' because it will break all sorts of things.
 
924
        # Prevent '[' and ']' because they can be used to inject into the
 
925
        # svn.conf.
 
926
        # Normalise to avoid resulting in ".." path segments.
 
927
        if not os.path.isabs(path):
 
928
            raise SubmissionError("Path is not absolute")
 
929
        if any(c in path for c in "\n[]"):
 
930
            raise SubmissionError("Path must not contain '\\n', '[' or ']'")
 
931
        return os.path.normpath(path)
591
932
 
592
933
# WORKSHEETS AND EXERCISES #
593
934
 
594
935
class Exercise(Storm):
 
936
    """An exercise for students to complete in a worksheet.
 
937
 
 
938
    An exercise may be present in any number of worksheets.
 
939
    """
 
940
 
595
941
    __storm_table__ = "exercise"
596
942
    id = Unicode(primary=True, name="identifier")
597
943
    name = Unicode()
598
944
    description = Unicode()
 
945
    _description_xhtml_cache = Unicode(name='description_xhtml_cache')
599
946
    partial = Unicode()
600
947
    solution = Unicode()
601
948
    include = Unicode()
609
956
        'WorksheetExercise.worksheet_id',
610
957
        'Worksheet.id'
611
958
    )
612
 
    
 
959
 
613
960
    test_suites = ReferenceSet(id, 
614
961
        'TestSuite.exercise_id',
615
962
        order_by='seq_no')
619
966
    def __repr__(self):
620
967
        return "<%s %s>" % (type(self).__name__, self.name)
621
968
 
622
 
    def get_permissions(self, user):
 
969
    def get_permissions(self, user, config):
 
970
        return self.global_permissions(user, config)
 
971
 
 
972
    @staticmethod
 
973
    def global_permissions(user, config):
 
974
        """Gets the set of permissions this user has over *all* exercises.
 
975
        This is used to determine who may view the exercises list, and create
 
976
        new exercises."""
623
977
        perms = set()
624
978
        roles = set()
625
979
        if user is not None:
626
980
            if user.admin:
627
981
                perms.add('edit')
628
982
                perms.add('view')
629
 
            elif 'lecturer' in set((e.role for e in user.active_enrolments)):
630
 
                perms.add('edit')
631
 
                perms.add('view')
632
 
            
 
983
            elif u'lecturer' in set((e.role for e in user.active_enrolments)):
 
984
                perms.add('edit')
 
985
                perms.add('view')
 
986
            elif (config['policy']['tutors_can_edit_worksheets']
 
987
            and u'tutor' in set((e.role for e in user.active_enrolments))):
 
988
                # Site-specific policy on the role of tutors
 
989
                perms.add('edit')
 
990
                perms.add('view')
 
991
 
633
992
        return perms
634
 
    
635
 
    def get_description(self):
636
 
        return rst(self.description)
 
993
 
 
994
    def _cache_description_xhtml(self, invalidate=False):
 
995
        # Don't regenerate an existing cache unless forced.
 
996
        if self._description_xhtml_cache is not None and not invalidate:
 
997
            return
 
998
 
 
999
        if self.description:
 
1000
            self._description_xhtml_cache = rst(self.description)
 
1001
        else:
 
1002
            self._description_xhtml_cache = None
 
1003
 
 
1004
    @property
 
1005
    def description_xhtml(self):
 
1006
        """The XHTML exercise description, converted from reStructuredText."""
 
1007
        self._cache_description_xhtml()
 
1008
        return self._description_xhtml_cache
 
1009
 
 
1010
    def set_description(self, description):
 
1011
        self.description = description
 
1012
        self._cache_description_xhtml(invalidate=True)
637
1013
 
638
1014
    def delete(self):
639
1015
        """Deletes the exercise, providing it has no associated worksheets."""
644
1020
        Store.of(self).remove(self)
645
1021
 
646
1022
class Worksheet(Storm):
 
1023
    """A worksheet with exercises for students to complete.
 
1024
 
 
1025
    Worksheets are owned by offerings.
 
1026
    """
 
1027
 
647
1028
    __storm_table__ = "worksheet"
648
1029
 
649
1030
    id = Int(primary=True, name="worksheetid")
651
1032
    identifier = Unicode()
652
1033
    name = Unicode()
653
1034
    assessable = Bool()
 
1035
    published = Bool()
654
1036
    data = Unicode()
 
1037
    _data_xhtml_cache = Unicode(name='data_xhtml_cache')
655
1038
    seq_no = Int()
656
1039
    format = Unicode()
657
1040
 
675
1058
        return "<%s %s>" % (type(self).__name__, self.name)
676
1059
 
677
1060
    def remove_all_exercises(self):
678
 
        """
679
 
        Remove all exercises from this worksheet.
 
1061
        """Remove all exercises from this worksheet.
 
1062
 
680
1063
        This does not delete the exercises themselves. It just removes them
681
1064
        from the worksheet.
682
1065
        """
686
1069
                raise IntegrityError()
687
1070
        store.find(WorksheetExercise,
688
1071
            WorksheetExercise.worksheet == self).remove()
689
 
            
690
 
    def get_permissions(self, user):
691
 
        return self.offering.get_permissions(user)
692
 
    
693
 
    def get_xml(self):
694
 
        """Returns the xml of this worksheet, converts from rst if required."""
695
 
        if self.format == u'rst':
696
 
            ws_xml = rst(self.data)
697
 
            return ws_xml
 
1072
 
 
1073
    def get_permissions(self, user, config):
 
1074
        offering_perms = self.offering.get_permissions(user, config)
 
1075
 
 
1076
        perms = set()
 
1077
 
 
1078
        # Anybody who can view an offering can view a published
 
1079
        # worksheet.
 
1080
        if 'view' in offering_perms and self.published:
 
1081
            perms.add('view')
 
1082
 
 
1083
        # Any worksheet editors can both view and edit.
 
1084
        if 'edit_worksheets' in offering_perms:
 
1085
            perms.add('view')
 
1086
            perms.add('edit')
 
1087
 
 
1088
        return perms
 
1089
 
 
1090
    def _cache_data_xhtml(self, invalidate=False):
 
1091
        # Don't regenerate an existing cache unless forced.
 
1092
        if self._data_xhtml_cache is not None and not invalidate:
 
1093
            return
 
1094
 
 
1095
        if self.format == u'rst':
 
1096
            self._data_xhtml_cache = rst(self.data)
 
1097
        else:
 
1098
            self._data_xhtml_cache = None
 
1099
 
 
1100
    @property
 
1101
    def data_xhtml(self):
 
1102
        """The XHTML of this worksheet, converted from rST if required."""
 
1103
        # Update the rST -> XHTML cache, if required.
 
1104
        self._cache_data_xhtml()
 
1105
 
 
1106
        if self.format == u'rst':
 
1107
            return self._data_xhtml_cache
698
1108
        else:
699
1109
            return self.data
700
 
    
 
1110
 
 
1111
    def set_data(self, data):
 
1112
        self.data = data
 
1113
        self._cache_data_xhtml(invalidate=True)
 
1114
 
701
1115
    def delete(self):
702
1116
        """Deletes the worksheet, provided it has no attempts on any exercises.
703
 
        
 
1117
 
704
1118
        Returns True if delete succeeded, or False if this worksheet has
705
1119
        attempts attached."""
706
1120
        for ws_ex in self.all_worksheet_exercises:
707
1121
            if ws_ex.saves.count() > 0 or ws_ex.attempts.count() > 0:
708
1122
                raise IntegrityError()
709
 
        
 
1123
 
710
1124
        self.remove_all_exercises()
711
1125
        Store.of(self).remove(self)
712
 
        
 
1126
 
713
1127
class WorksheetExercise(Storm):
 
1128
    """A link between a worksheet and one of its exercises.
 
1129
 
 
1130
    These may be marked optional, in which case the exercise does not count
 
1131
    for marking purposes. The sequence number is used to order the worksheet
 
1132
    ToC.
 
1133
    """
 
1134
 
714
1135
    __storm_table__ = "worksheet_exercise"
715
 
    
 
1136
 
716
1137
    id = Int(primary=True, name="ws_ex_id")
717
1138
 
718
1139
    worksheet_id = Int(name="worksheetid")
722
1143
    optional = Bool()
723
1144
    active = Bool()
724
1145
    seq_no = Int()
725
 
    
 
1146
 
726
1147
    saves = ReferenceSet(id, "ExerciseSave.ws_ex_id")
727
1148
    attempts = ReferenceSet(id, "ExerciseAttempt.ws_ex_id")
728
1149
 
732
1153
        return "<%s %s in %s>" % (type(self).__name__, self.exercise.name,
733
1154
                                  self.worksheet.identifier)
734
1155
 
735
 
    def get_permissions(self, user):
736
 
        return self.worksheet.get_permissions(user)
737
 
    
 
1156
    def get_permissions(self, user, config):
 
1157
        return self.worksheet.get_permissions(user, config)
 
1158
 
738
1159
 
739
1160
class ExerciseSave(Storm):
740
 
    """
741
 
    Represents a potential solution to an exercise that a user has submitted
742
 
    to the server for storage.
743
 
    A basic ExerciseSave is just the current saved text for this exercise for
744
 
    this user (doesn't count towards their attempts).
745
 
    ExerciseSave may be extended with additional semantics (such as
746
 
    ExerciseAttempt).
747
 
    """
 
1161
    """A potential exercise solution submitted by a user for storage.
 
1162
 
 
1163
    This is not an actual tested attempt at an exercise, it's just a save of
 
1164
    the editing session.
 
1165
    """
 
1166
 
748
1167
    __storm_table__ = "exercise_save"
749
1168
    __storm_primary__ = "ws_ex_id", "user_id"
750
1169
 
760
1179
 
761
1180
    def __repr__(self):
762
1181
        return "<%s %s by %s at %s>" % (type(self).__name__,
763
 
            self.exercise.name, self.user.login, self.date.strftime("%c"))
 
1182
            self.worksheet_exercise.exercise.name, self.user.login,
 
1183
            self.date.strftime("%c"))
764
1184
 
765
1185
class ExerciseAttempt(ExerciseSave):
766
 
    """
767
 
    An ExerciseAttempt is a special case of an ExerciseSave. Like an
768
 
    ExerciseSave, it constitutes exercise solution data that the user has
769
 
    submitted to the server for storage.
770
 
    In addition, it contains additional information about the submission.
771
 
    complete - True if this submission was successful, rendering this exercise
772
 
        complete for this user.
773
 
    active - True if this submission is "active" (usually true). Submissions
774
 
        may be de-activated by privileged users for special reasons, and then
775
 
        they won't count (either as a penalty or success), but will still be
776
 
        stored.
777
 
    """
 
1186
    """An attempt at solving an exercise.
 
1187
 
 
1188
    This is a special case of ExerciseSave, used when the user submits a
 
1189
    candidate solution. Like an ExerciseSave, it constitutes exercise solution
 
1190
    data.
 
1191
 
 
1192
    In addition, it contains information about the result of the submission:
 
1193
 
 
1194
     - complete - True if this submission was successful, rendering this
 
1195
                  exercise complete for this user in this worksheet.
 
1196
     - active   - True if this submission is "active" (usually true).
 
1197
                  Submissions may be de-activated by privileged users for
 
1198
                  special reasons, and then they won't count (either as a
 
1199
                  penalty or success), but will still be stored.
 
1200
    """
 
1201
 
778
1202
    __storm_table__ = "exercise_attempt"
779
1203
    __storm_primary__ = "ws_ex_id", "user_id", "date"
780
1204
 
783
1207
    text = Unicode(name="attempt")
784
1208
    complete = Bool()
785
1209
    active = Bool()
786
 
    
787
 
    def get_permissions(self, user):
 
1210
 
 
1211
    def get_permissions(self, user, config):
788
1212
        return set(['view']) if user is self.user else set()
789
 
  
 
1213
 
790
1214
class TestSuite(Storm):
791
 
    """A Testsuite acts as a container for the test cases of an exercise."""
 
1215
    """A container to group an exercise's test cases.
 
1216
 
 
1217
    The test suite contains some information on how to test. The function to
 
1218
    test, variables to set and stdin data are stored here.
 
1219
    """
 
1220
 
792
1221
    __storm_table__ = "test_suite"
793
1222
    __storm_primary__ = "exercise_id", "suiteid"
794
 
    
 
1223
 
795
1224
    suiteid = Int()
796
1225
    exercise_id = Unicode(name="exerciseid")
797
1226
    description = Unicode()
801
1230
    exercise = Reference(exercise_id, Exercise.id)
802
1231
    test_cases = ReferenceSet(suiteid, 'TestCase.suiteid', order_by="seq_no")
803
1232
    variables = ReferenceSet(suiteid, 'TestSuiteVar.suiteid', order_by='arg_no')
804
 
    
 
1233
 
805
1234
    def delete(self):
806
1235
        """Delete this suite, without asking questions."""
807
 
        for vaariable in self.variables:
 
1236
        for variable in self.variables:
808
1237
            variable.delete()
809
1238
        for test_case in self.test_cases:
810
1239
            test_case.delete()
811
1240
        Store.of(self).remove(self)
812
1241
 
813
1242
class TestCase(Storm):
814
 
    """A TestCase is a member of a TestSuite.
815
 
    
816
 
    It contains the data necessary to check if an exercise is correct"""
 
1243
    """A container for actual tests (see TestCasePart), inside a test suite.
 
1244
 
 
1245
    It is the lowest level shown to students on their pass/fail status."""
 
1246
 
817
1247
    __storm_table__ = "test_case"
818
1248
    __storm_primary__ = "testid", "suiteid"
819
 
    
 
1249
 
820
1250
    testid = Int()
821
1251
    suiteid = Int()
822
1252
    suite = Reference(suiteid, "TestSuite.suiteid")
823
1253
    passmsg = Unicode()
824
1254
    failmsg = Unicode()
825
 
    test_default = Unicode()
 
1255
    test_default = Unicode() # Currently unused - only used for file matching.
826
1256
    seq_no = Int()
827
 
    
 
1257
 
828
1258
    parts = ReferenceSet(testid, "TestCasePart.testid")
829
 
    
 
1259
 
830
1260
    __init__ = _kwarg_init
831
 
    
 
1261
 
832
1262
    def delete(self):
833
1263
        for part in self.parts:
834
1264
            part.delete()
835
1265
        Store.of(self).remove(self)
836
1266
 
837
1267
class TestSuiteVar(Storm):
838
 
    """A container for the arguments of a Test Suite"""
 
1268
    """A variable used by an exercise test suite.
 
1269
 
 
1270
    This may represent a function argument or a normal variable.
 
1271
    """
 
1272
 
839
1273
    __storm_table__ = "suite_variable"
840
1274
    __storm_primary__ = "varid"
841
 
    
 
1275
 
842
1276
    varid = Int()
843
1277
    suiteid = Int()
844
1278
    var_name = Unicode()
845
1279
    var_value = Unicode()
846
1280
    var_type = Unicode()
847
1281
    arg_no = Int()
848
 
    
 
1282
 
849
1283
    suite = Reference(suiteid, "TestSuite.suiteid")
850
 
    
 
1284
 
851
1285
    __init__ = _kwarg_init
852
 
    
 
1286
 
853
1287
    def delete(self):
854
1288
        Store.of(self).remove(self)
855
 
    
 
1289
 
856
1290
class TestCasePart(Storm):
857
 
    """A container for the test elements of a Test Case"""
 
1291
    """An actual piece of code to test an exercise solution."""
 
1292
 
858
1293
    __storm_table__ = "test_case_part"
859
1294
    __storm_primary__ = "partid"
860
 
    
 
1295
 
861
1296
    partid = Int()
862
1297
    testid = Int()
863
 
    
 
1298
 
864
1299
    part_type = Unicode()
865
1300
    test_type = Unicode()
866
1301
    data = Unicode()
867
1302
    filename = Unicode()
868
 
    
 
1303
 
869
1304
    test = Reference(testid, "TestCase.testid")
870
 
    
 
1305
 
871
1306
    __init__ = _kwarg_init
872
 
    
 
1307
 
873
1308
    def delete(self):
874
1309
        Store.of(self).remove(self)