Tag List Report

The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.

Tag Class Total number of occurrences Tag strings used by tag class
@todo 6 @todo
TODO 34 TODO

Each tag is detailed below:

@todo

Number of occurrences found in the code: 6

org.apache.continuum.dao.DaoUtilsImpl Line
Move to a better place 154
Move to a better place 196
Move to a better place 270
org.apache.maven.continuum.store.AbstractContinuumStoreTestCase Line
push down to a Jdo specific test 1671
org.apache.maven.continuum.store.ContinuumStoreTest Line
I think this should have all the JDO stuff from the abstract test, and the abstract test should use a mock continuum store with the exception of the integration tests which should be running against a fully deployed plexus application instead 57
review for ambiguities and ensure it is all encapsulated in the store, otherwise the code may make the same mistake about not deleting things, etc 60

TODO

Number of occurrences found in the code: 34

org.apache.continuum.dao.BuildDefinitionDaoImpl Line
Move this method to a service class 278
org.apache.continuum.dao.BuildResultDaoImpl Line
Use projectDao 74
these are in the wrong spot - set them on success (though currently some depend on latest build being the one in progress) 104
org.apache.continuum.dao.ProjectGroupDaoImpl Line
why do we need to do this? if not - build results are not removed and a integrity constraint is violated. I assume its because of the fetch groups 78
org.apache.maven.continuum.store.AbstractContinuumStoreTestCase Line
simplify by deep copying the relationships in createTest... ? 345
better way? this assumes that some untested methods already work! 369
!!! -- definitely need to test the changeset stuff since it uses modello.refid 801
simplify by deep copying the relationships in createTest... ? 818
org.apache.maven.continuum.store.ContinuumStoreTest Line
test if it has any attachments 345
test if it has any attachments 424
test the project group's notifiers are physically deleted 669
test the project group's build definitions are physically deleted 670
test the build result was physically deleted 697
test the build result's SCM result was physically deleted 698
test the build result's SCM result's change sets and change files were physically deleted 699
reports, artifacts, data 740
test the developer was physically deleted 849
test the dependency was physically deleted 897
test the notifier was physically deleted 944
test the def was physically deleted 1006
test the schedule/profile was NOT physically deleted 1007
test the notifier was physically deleted 1059
test the def was physically deleted 1119
test the schedule/profile was NOT physically deleted 1120
test the project's checkout SCM result was physically deleted 1420
test the project's checkout SCM result's change sets and change files were physically deleted 1421
test the project's dependencies are physically deleted 1422
test the project's developers are physically deleted 1423
test the project's builds are physically deleted 1424
test the project's notifiers are physically deleted 1425
test the project's build definitions are physically deleted 1426
artifacts 1567
report 1568
long error data } 1569