CSC517 Spring 2019/E1923 New Framework For Import/Export: Difference between revisions
No edit summary |
|||
Line 60: | Line 60: | ||
*'''sign_up_topic''' | *'''sign_up_topic''' | ||
*'''user''' | *'''user''' | ||
== Proposed Import Changes == | |||
Most import functionality is routed through the ImportFileController and import class methods on models that are being imported. We intend to remove other helpers and files, such as the ImportFileHelper, the ImportTopicHelper, and the QuestionnaireHelper to keep import routing consistent. We will also remove the QuestionnaireController and route that import through the ImportFileController. In addition, we will update ImportFileController methods (in particular, #hash_rows_with_headers and #import_from_hash) to make better use of polymorphism and eliminate large and unnecessary if/else blocks. | |||
We will also insert more specific object creation specifications. Consider if a user's import contains objects that do not exist in the database. We will provide an option for the user to specify whether or not new/dependent objects should be created and account for this boolean in relevant ::import functions for the models. | |||
To summarize our changes: | |||
* Route all import traffic through ImportFileController and ::import calls on models. | |||
* Refactor ImportFileController. | |||
* Insert object creation conditions into all relevant ::import functions and into the ImportFileController form. |
Revision as of 14:36, 7 April 2019
Project Introduction
Team
- Veeha Khanna, vkhanna
- Randy Paluszkiewicz, rpalusz
- Drew Marshburn, rdmarshb
- Andrew Miller, asmille5
Description of Project
What it does: The export/import feature is the most helpful feature for instructors to set up assignments. The instructors usually have a list of students, teams, etc from their learning management system. Being able to export/import these into expertiza saves a lot of time when setting up an assignment.
What should be done: Expertiza provides multiple export and import features for eg. export students, teams etc. Essentially what it does is it fetches some data from database and save it as a file in desired format. However, same functionality is implemented multiple times for exporting and importing different things. The aim of this project is to design a generic export/import feature. What you need to do is implement a framework which supports exporting and importing functionality based on input. There are substantial degrees of freedom in this project. You need to design the module which will take the database table names, column names from which data needs to be exported or the imported into. So that this module can be used for every export/import feature that expertiza provides.
Most imports and exports just import or export a single table. But sometimes exports involve “details.” For example, when questionnaires are imported or exported, the “advice” (guidelines on what characteristics of work merit each score) may be imported or exported along with the questionnaire. I suspect that all instances of import/export of multiple tables are called “details” in the current code. Provide a mechanism so that “details” can be imported or exported along with the main table involved.
Existing Import Functionality
Existing import functionality is primarily routed through the import_file_controller and an import class method for various models. Two models, SignUpTopic and User, rely on helper classes that extract attributes from a hash and create an ActiveRecord object. Additionally, Questionnaire relies on a helper that can import Question objects from a CSV and adjust the size of the associated QuestionAdvice. More detail about specific functions is provided below.
Controllers
- import_file_controller
- File processing methods:
- #get_delimiter - Sets proper delimiter for filetype
- #parse_line - Processes line (row) of the file
- #parse_to_grid - Turns file into 2D array
- #parse_to_hash - Turns file into hash where 'header' stores header row and 'body' stores all contents.
- #hash_rows_with_headers - Creates hash for each row of file. Keys are headers, values are row values.
- Import methods:
- #import_from_hash - Primary import functionality. Creates objects for hashed rows (from #hash_rows_with_headers).
- #import - Larger controller of import, sets error messages and displays.
- File processing methods:
- questionnaires_controller
- ::import - Allows import from CSV using QuestionnaireHelper
Helpers
- import_file_helper
- ::define_attributes - Sets and returns attributes for User object from hash.
- ::create_new_user - Makes a user object in the database.
- import_topics_helper
- ::define_attributes - Sets and returns attributes for a SignUpTopic from hash.
- ::create_new_sign_up_topic - Makes SignUpTopic objects in the database.
- questionnaire_helper
- ::get_questions_from_csv - Allows Question and QuestionAdvice import/creation with CSV file.
Models
All models have an ::import method called by the ImportFileController. In addition, SignUpTopic and User objects rely on their helpers, which are mentioned above.
- assignment_participant
- assignment_team
- course_participant
- course_team
- team
- metareview_response_map
- question
- review_response_map
- sign_up_sheet
- sign_up_topic
- user
Proposed Import Changes
Most import functionality is routed through the ImportFileController and import class methods on models that are being imported. We intend to remove other helpers and files, such as the ImportFileHelper, the ImportTopicHelper, and the QuestionnaireHelper to keep import routing consistent. We will also remove the QuestionnaireController and route that import through the ImportFileController. In addition, we will update ImportFileController methods (in particular, #hash_rows_with_headers and #import_from_hash) to make better use of polymorphism and eliminate large and unnecessary if/else blocks.
We will also insert more specific object creation specifications. Consider if a user's import contains objects that do not exist in the database. We will provide an option for the user to specify whether or not new/dependent objects should be created and account for this boolean in relevant ::import functions for the models.
To summarize our changes:
- Route all import traffic through ImportFileController and ::import calls on models.
- Refactor ImportFileController.
- Insert object creation conditions into all relevant ::import functions and into the ImportFileController form.