2018-06 Code Review
Overview
Date | 2018-07 |
---|---|
Status | FIXED |
Release | v1.2.0 |
TestType | manual |
TestTopic | Code Review |
Context | Modular Migration |
Tester | |
Standard |
Background
openIMIS needed to evolve from MS IMIS to a generic standard product that can be fully customized and scaled to the needs of a growing number of implementing organisations. We had to realize a quick analysis of the source code considering its performance and persistence in terms of developments foreseen in the technical roadmap. The idea is to maintain the MS Visual Basic core while at the same time starting changes towards modularity and to explore options for a successively adding new / replacing old modules on the basis of open source technologies.
Result Summary
The results of the code review allowed to plan the following migration of the software package into the new modular architecture and to identify the most pressing vulnerabilities and pain points for quick fixes.
Methodology
The analysis was done by the members of the Developers Committee during the months of June and July in 2018 at a very high level with short examples from real code on the basis of the current master version in the github repository.
The review covers all aspects from the suggestions for a way forward from the technical roadmap, and is divided between evaluation of cross-cutting aspects and an evaluation per functional area.
The following elements were identified as out of scope for the current review
Link individual code files to functional areas
Interdependencies between modules
C# API
Detailed Results
The links in the lists below point to a document that is in a protected space and will only work for members of the review committee. You will not be able to access it. Kindly use the pdf copy of the report instead, which is attached to this page:
Cross Cutting Aspects
Scoring:
Implemented
Partially implemented
Not implemented
Cross Cutting Aspect | Score |
2 | |
2 | |