-
Notifications
You must be signed in to change notification settings - Fork 19
Home
Thomas Mapesa edited this page Jul 21, 2014
·
24 revisions
- Feature List (Current)
- Standards
- General Programming
- Database
- PHP Classes
- The views
- HTML and CSS
- Javascript
- Github
- Tests
- Database Model
- Authentication and Authorization
- Login
- Logout
- User Management
- Create new user
- View list of existing users
- View individual user details
- Update user details
- Delete user
- Password management still missing
- Patient Records
- Create new patient
- View list of existing patients
- View individual patient details (personal information)
- Update patient details
- Delete patient
- Lab Sections - Create, Read, Update and Delete
- Create new lab section
- View list of existing lab sections
- View individual lab section details
- Update lab section details
- Delete lab section
- Test Types
- Create new test type
- View list of existing test types
- View individual test type details
- Update test type details
- Delete test type
- Measures
- Create new measure
- View list of existing measures
- View individual measure details
- Update measure details
- Delete measure
- Specimen Types - Create, Read, Update and Delete
- Create new specimen type
- View list of existing specimen types
- View individual specimen type details
- Update specimen type details
- Delete specimen type
- All indices should zero based. e.g. if a element has sequential numeric values these would be from the series 0,1,2,3,...
- Naming
- table names, column names: all lowercase, use underscores to separate words e.g specimen_id, test_type_id
- names with abbreviations may be treated differently e.g. targetTAT makes more sense
- better long names that makes sense
- Use of appropriate field type e.g. ‘SMALLINT’ instead of ‘STRING’ for age
- If a primary key is necessary on a table it should be the first column and should be named 'id'. It should be unsigned.
- When this id is referenced in another table (as a foreign key) it should be called 'tablename_id'
- Table names are in the plural e.g. specimens instead of specimen
- Create a migration file for every new table or change to a table. If necessary add the seed data using a separate file. See the Migrations & Seeding section of the online Laravel guide.
Back to Top ##PHP Classes This applies to Controllers, Models, migration and seeding classes, ...
- Class names always begins with uppercase e.g. CreateTestTypeMeasureTable
- When combining words every word begins with uppercase. Do not use underscores e.g. SpecimenType as opposed to Specimen_type
- Method or Function names begin in lowercase and words join in uppercase e.g. getSpecimenTypes()
- Provide descriptive class and function comments before their definitions
- Variable names - begin in lowercase. Multiple words should begin in upper case e.g. specimenType
- Better long names that makes sense e.g. specimenType instead of speTyp
- Soft delete(s) preferable to hard delete(s)
Back to Top ##Laravel views
- Using blade templating engine
- All html elements in lowercase
- Using bootstrap for css
Back to Top ##HTML and CSS
- All CSS should be placed in an external stylesheet file (public/css/)
- CSS class names should use hyphens when joining words e.g. ‘user-image’
- Indent for readability in order of precedence e.g.
form-group{}
.form-group.actions-row{}
.form-group > label{}
.form-group .form-control{}
.form-group .form-pane{}
.form-group .form-pane label{}
- Never use !important;
- CSS files should have categorised sections e.g. for user files, for general layout
- Use contextual elements such as <strong> and <em> instead of <b> or <i>
- Page layout should be done from the CSS file. Do not use for spacing. If done properly there should be no need to use <br> either.
- Comments: Include the section name e.g.
/**
* forms
*/
- Using jquery
- All javascript code should be in external files (public/js/)
- Create an issue for each change you want to make.
- Create a branch from the master for the same change. The branch name should be descriptive
- Once fixed, create a pull request asking a specific person to review the feature.
- Reviewed and approved features should be merged back to the master and the branch deleted.
- Every feature should have an accompanying test
- A feature should pass all tests before it is merged into the master branch