Update:Archive/1.0/Developers: Difference between revisions

Line 222: Line 222:


=== Planning and Preparation ===
=== Planning and Preparation ===
* Eliminate areas that will not ever be implemented (until v2.0)
* <strike>Eliminate areas that will not ever be implemented (until v2.0)</strike>
* Determine which sections of UMO need the most attention
* <strike>Determine which sections of UMO need the most attention</strike>
* Create list of items we will review
* <strike>Create list of items we will review</strike>
* Clean code for review (PHP_beautifier)
* <strike>Clean code for review (PHP_beautifier)</strike> - did not make a difference, moved on.


=== Code Review ===
=== Code Review ===
* Document includes, dependencies
* <strike>Document includes, dependencies</strike>
* Review application based on priorities, listing needed changes
* <strike>Review application based on priorities, listing needed changes</strike>
** Imperative security changes
** <strike>Imperative security changes</strike>
** Efficiency issues that really need to be changed
** <strike>Efficiency issues that really need to be changed</strike>
** Efficiency or style issues that are cosmetic
** <strike>Efficiency or style issues that are cosmetic</strike>
* Wiki might be good for cataloguing these
* Test behavior in identical staging environment for all public use-cases. (underway)


=== Code Adjustments ===
=== Code Adjustments ===
* Patch branch to include local modifications in prod. (under review)
* Update code based on quantified 'needed changes'
* Update code based on quantified 'needed changes'
** Rewrite heavy-hit scripts (VersionCheck.php, PFS) (morgamic)
** Unfreeze branch and open it up for patching. (kveton)
** Deal out important patching to community based on wiki information. (morgamic,alanjstr)
* If possible, update code for readability and efficiency
* If possible, update code for readability and efficiency


3,035

edits