Daniel Diniz <ajaksu@gmail.com> [Fri, 03 Jul 2009 14:35:03 +0200] rev 2502
Surveys can now have required questions and comments can be turned on/off.
Storing comments is now working. However some issues arise when form errors occur, like missing context and the errors also trigger before a form has been submitted.
Reviewed by: Lennard de Rijk
Daniel Diniz <ajaksu@gmail.com> [Fri, 03 Jul 2009 14:19:23 +0200] rev 2501
Added Survey From Field validation.
Also added the COMMENT_PREFIX in Survey model to ensure that comments are stored in the same way everwhere.
Reviewed by: Lennard de Rijk
Daniel Diniz <ajaksu@gmail.com> [Fri, 03 Jul 2009 12:10:24 +0200] rev 2500
Style fixes and a forgotten parameter in Survey(Record) logic.
Reviewed by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Fri, 03 Jul 2009 12:05:23 +0200] rev 2499
Removed unnecessary variable in ProjectSurvey take method.
Lennard de Rijk <ljvderijk@gmail.com> [Fri, 03 Jul 2009 12:02:57 +0200] rev 2498
Overwrite _getSurveyRecordFor in ProjectSurvey view to take StudentProject into account.
Lennard de Rijk <ljvderijk@gmail.com> [Fri, 03 Jul 2009 12:00:53 +0200] rev 2497
Added _getSurveyRecordFor method to Survey View.
This method is used by the Take view to get the right SurveyRecord for the current request. Subclassed Views that want to fetch their records in another way can override this method.
Lennard de Rijk <ljvderijk@gmail.com> [Thu, 02 Jul 2009 18:35:57 +0200] rev 2496
Added default Survey list templates for ProjectSurvey and GradingProjectSurvey.
Daniel Diniz <ajaksu@gmail.com> [Thu, 02 Jul 2009 18:33:13 +0200] rev 2495
Enable the ProjectSurvey and GradingProjectSurvey views in the core.
Also add menu items to the program menu for easy access to these options.
Reviewed by: Lennard de Rijk
Lennard de Rijk <ljvderijk@gmail.com> [Thu, 02 Jul 2009 17:51:07 +0200] rev 2494
Implemented taking ProjectSurveys.
The user will first be served a list of all eligble StudentProjects for which the ProjectSurvey can be taken. After clicking on the list the key_name of the project is attached as a GET param and the code uses the normal take view with the hooks to let everything run smoothly.
An extra access check method is required which checks if the Project mentioned in the GET param is a valid project for the current user. This check will run together with the isSurveyTakable checker.
Lennard de Rijk <ljvderijk@gmail.com> [Thu, 02 Jul 2009 17:47:24 +0200] rev 2493
Edit SurveyRecord model to not make use of different Survey property for each subclass.
This does mean that a simple query needs to be constructed by hand whenever you want to query for every SurveyRecord for a specific Survey. However the naming is now the same over each SurveyRecord and it fits well into the rest of the design of Melange.