Todo List

ObjectRelationalMappingLayer

StoredObject::getGeneralData()

• add check for $key in fields.

StoredObject::getSelectByIDString()

• why not default $multi and $m
• why passing $table not using $this->tables[$tableName]?

Address::populateFromFormVars()

• Handle mutliple matches better?

StoredObject::remove()

• add error reporting when ON DELETE RESTRICT?

WebPages

back.php

• Remove the back button prohibition and then this page

done.php

• Revisit email now that passwords are assigned and most changes result in an email.
• Integrate the email/PDF options with the config file

edit-member.php

• Lock down more user pre-set fields, but allow team to set needed ones if not set

getCompNameFromURL()

• add a regexp to check for legal compunix names?

getTeamIDFromURL()

• add a regeexp to enfofce legal team IDs?

grant-permissions.php

• Lock down more user pre-set fields, but allow team to set needed ones if not set

new-member.php

• Needs better/stronger check for existing people/users

Database

CIB_DB::addPending()

• elevate keys to class status throughout

CIB_DB::availibleUserName()

• fix typo here and in calling location(s)

CIB_DB

• Further tease out SQL to the PhraseBook. The CIB_DB class should evolve into the functional API -- it needs more parameter checking and more consistency of return values to be useful. The places where a SQL statement was wrapped up for use here, but is only used by a single location in the code, might get "demoted" to pure PhraseBook status, etc.

CIB_DB::CIB_DB()

• Add a check after connecting to make sure its a central DB and not a per-comp DB

CIB_DB::denyPermissionToUser()

• Refactor to one of the Access Control Classes or new

CIB_DB::expandTeamNamesFromID()

• refactor keys

ResultsDB::getAffiliation()

• Look into a CIB_Person Mapper (light weight compared to the Ragu_Person)

ResultsDB::getAllEntriesFor()

• Refactor results array into a new class

CIB_DB::getApprovedOutsideComps()

• examine deprecated status and remove?

PS_DB::getCanLevels()

• handle multi-type competitions better

PS_DB::getCanStyles()

• handle multi-type competitions better

CIB_DB::getCaptainsEmails()

• rename function and callers

CIB_DB::getClosedComps()

• unify with getHostedComps()? allow offset/limit?

Team_DB::getCompsOpenForRegistration()

• unify with call in parent

CIB_DB::getConfigComps()

• Unify with Link/Label return type, elevate to Class?
• Consider replacing User $user with string $username?

DB::getDBpass()

• remove the password and look up from the config file?

CIB_DB::getDocumentList()

• look over, purge?

ResultsDB::getHomeTown()

• Look into a CIB_Person Mapper (light weight compared to the Ragu_Person)

CIB_DB::getHostedComps()

• conver $offset to pure integer
• conver $limit to pure integer
• Abstract away name of columns for ordering

ResultsDB::getIdentificationByID()

• remove duplicate entries from DanceAs that also appear in Teams, to show only historic values in DanceAs

CIB_DB::getPendingKey()

• Refactor keys to a Key Class (StoredObject?)

CIB_DB::getPendingKeys()

• Consider refactoring function to Access Control-LockSmith?
• change parameter name to username
• Refactor keys to a Key Class (StoredObject?)

CIB_DB::getPendingPermissions()

• elevate keys to class status throughout

ResultsDB::getPeopleByName()

• Look at reconcilling with the current Prime/Partner matching code, but remember these touch different schema databases.

ResultsDB::getPersonName()

• Look into a CIB_Person Mapper (light weight compared to the Ragu_Person)

ResultsDB::getResultsFor()

• Refactor results array into a new class

CIB_DB::getSyncRequests()

• Consider refactoring function to Access Control

CIB_DB::getTeamAdmin()

• Consider replacing User $user with string $username?

CIB_DB::getTeamMemberships()

• Consider replacing User $user with string $username?

CIB_DB::getTeamRequests()

• Consider refactoring function to Access Control

Team_DB::getUpcomingComps()

• unify with call in parent

CIB_DB::getUserPasswordByName()

• Remove if possible, relocated to User

CIB_DB::grantPermissionToUser()

• Refactor to one of the Access Control Classes or new

DB::helper_limitOffset()

• Change to public and rename

DB::helper_queryToArray()

• Change to public and rename

DB::helper_queryToArrayCategories()

• Change to public and rename

DB::helper_queryToSimpleArray()

• Change to public and rename

DB::helper_singleValueQuery()

• Change to public and rename

PS_DB

• Further tease out SQL to the PhraseBook.

PS_DB::PS_DB()

• Add a check after connecting to make sure its a central DB and not a per-comp DB

ResultsDB

• Further tease out SQL to the PhraseBook. The ResultsDBDB class should evolve into the functional API -- it needs more parameter checking and more consistency of return values to be useful. The places where a SQL statement was wrapped up for use here, but is only used by a single location in the code, might get "demoted" to pure PhraseBook status, etc.

ResultsDB::ResultsDB()

• Add a check after connecting to make sure its a central DB and not a per-comp DB

Team_DB

• Further tease out SQL to the PhraseBook.

Team_DB::Team_DB()

• Add a check after connecting to make sure its a central DB and not a per-comp DB

Tests

TestEventEntryListFormatter_tabbedOutput::TestInductiveLeap()

• Refactor after simpletest allows cloning/overriding of mocks

TestCompetitorListFormatter_tabbedOutput::TestInductiveLeap()

• Refactor after simpletest allows cloning/overriding of mocks

TestEventEntryListFormatter_textOutput::TestInductiveLeapNoWrap()

• Refactor after simpletest allows cloning/overriding of mocks
• Convert to inductive proof, but splitting the loop into a sub method to test its operation directly.

TestEventEntryListFormatter_textOutput::TestInductiveLeapWrap()

• Refactor after simpletest allows cloning/overriding of mocks
• Convert to inductive proof, but splitting the loop into a sub method to test its operation directly.

TestCompetitorAffilListFormatter_tabbedOutput::TestInductiveLeap_different()

• Refactor after simpletest allows cloning/overriding of mocks

TestCompetitorAffilListFormatter_tabbedOutput::TestInductiveLeap_same()

• Refactor after simpletest allows cloning/overriding of mocks

TestCompetitorAffilListFormatter_textFormattedOutput::TestInductiveStepDifferentAffil()

• Refactor after simpletest allows cloning/overriding of mocks

TestCompetitorListFormatter_textFormattedOutput::TestInductiveStepDifferentLeader()

• Refactor after simpletest allows cloning/overriding of mocks

TestCompetitorAffilListFormatter_textFormattedOutput::TestInductiveStepDifferentLeaderSameSchool()

• Refactor after simpletest allows cloning/overriding of mocks

TestCompetitorAffilListFormatter_textFormattedOutput::TestInductiveStepSameLeader()

• Refactor after simpletest allows cloning/overriding of mocks

TestCompetitorListFormatter_textFormattedOutput::TestInductiveStepSameLeader()

• Refactor after simpletest allows cloning/overriding of mocks

TestSlidingDoorsAdminLoginRequirement

• add tests of logged in users with insufficient permissions

CommandLine-Scripts

compmngr_export.php

• Compare against compmngr_export_v2.php and delete it not needed

compmngr_export_v2.php

• Compare against compmngr_export.php and delete it not needed

filterBlanks()

• read through the code and see why we need this then document it

newresults.php

• Parse judge list off bottom of new chester format list

rebuild_sd_stats.php

• Compare with current version running on production

s3-export.php

• A revised version of the stats system is in use on a hacked production system. The changes will be merged back to this tree shortly.

team-summary-lists.php

• remove email address requirement

• This is probably broken

DataFormatting

ListFormatter::_format()

• Look into splitting type out to a strategy?

Utilities

Registry::instance()

• when changed to PHP mark constructor private

PhraseBook::loadDictFile()

• test for file existence

PhraseBook

• Write a SQLDoc documentation generator

Documentation generated on Tue, 25 Apr 2006 13:12:41 -0400 by phpDocumentor 1.3.0RC3