TODO

(in /home/.mouschi/ostraaten/dailybuild.epfwiki.net/tuesday/source) app/controllers/application.rb:

  * [ 26] [TODO] don't send error email when action error causes an error
  * [ 54] [TODO] to have caching of pages per user group add something like :params => params.merge( :utype=> 'admin' )
  * [ 70] [TODO] should be flash.now?

app/controllers/login_controller.rb:

  * [100] [TODO] security enhancement: require the old password
  * [101] [TODO] like to use flash.now here but functional tests will fail! Is a bug?

app/controllers/pages_controller.rb:

  * [ 23] [TODO] check that we really need this
  * [ 33] [TODO] add 'Compare with previous' link
  * [ 93] [TODO] this is not used?
  * [106] [TODO] these kind of statement are no longer necessary,
  * [149] [TODO] force post method
  * [200] [TODO] a better name would be source_version_id?
  * [213] [TODO] his will cause an error in new.rhtml. Implement test
  * [221] [TODO] improved security - move authorisation check to Checkout.undo?
  * [243] [TODO] Implement test
  * [287] [TODO] be able to quickly retrieve a list of pages based on content type in a site

app/controllers/portal_controller.rb:

  * [ 78] [FIXME] Bugzilla 231125

app/controllers/users_controller.rb:

  * [ 46] [TODO] caching of this page
  * [ 82] [TODO] Cannot destroy user with versions and comments,

app/helpers/application_helper.rb:

  * [ 38] [TODO] a better name for this method
  * [182] [TODO] below incorrect

app/models/baseline_process.rb:

  * [ 56] [TODO] obsolete?
  * [ 90] [TODO] do this for Wikis as well, including status

app/models/checkout.rb:

  * [ 67] [TODO] rename HEAD_PATTERN -> HEAD_REGEXP
  * [ 91] [TODO] workaround for 250148: No-wrap should be part of CSS file https://bugs.eclipse.org/bugs/show_bug.cgi?id=250148
  * [ 93] [TODO] set title equal to pageTitle?
  * [119] [TODO] step 4 does not seem to work anymore with current version of OpenUP (EPF)
  * [120] [TODO] move to version as part of checkout
  * [122] [TODO] is this right? document this

app/models/comment.rb:

  * [ 26] [TODO] remove these redundant properties

app/models/da_text.rb:

  * [ 24] [TODO] problably there is a better way to do this

app/models/notification.rb:

  * [ 24] [TODO] user class as third parameter?

app/models/notifier.rb:

  * [150] [TODO] replace with notification

app/models/page.rb:

  * [ 62] [TODO] use regular expressions the right way

app/models/site.rb:

  * [ 29] [TODO] validate :user not :user_id
  * [ 31] [TODO] Folde can contain . (dot) because it is created from a filename
  * [ 46] [TODO]

app/models/sweeper.rb:

  * [ 19] [TODO] more advanced expiration

app/models/update.rb:

  * [ 21] [TODO] validates_presence_of :user, :baseline_process, :wiki

app/models/user.rb:

  * [  7] [TODO] disabled this, because this causes some error and we are not using Bugzilla integration

app/models/version.rb:

  * [ 24] [TODO] not necessary?
  * [ 66] [TODO] this doesn't add value
  * [ 82] [TODO] use page.last_version
  * [116] [TODO] remove? This should not be necessary, see html
  * [117] [TODO] remove?
  * [127] [TODO] Workaround for Tidy adding title attributes which causes false changes
  * [196] [TODO] implement
  * [206] [TODO] implement

app/models/wiki.rb:

  * [113] [TODO] notify users that want to be notified (add about notify_baseline_updates column to users)
  * [114] [TODO] Change 68 - Update should continue with checkouts and should not overwrite not harvested changes
  * [196] [TODO] anomaly, the following line shouldn't be necessary but is, in some cases.
  * [242] [TODO] rename to snippets

app/models/wiki_page.rb:

  * [ 21] [TODO] database change -> rename page_id to wiki_page_id
  * [ 26] [TODO] not working, find out why
  * [ 34] [TODO] (but now now) anomaly works in the console but not in the application or in tests. Then is always returns nil. Replaced with method
  * [ 40] [TODO] anomaly. the above statement does not work. Comments is always nil. When I change the key
  * [ 48] [FIXME] Bugzilla 231125
  * [ 70] [TODO] not sure if this is used
  * [ 79] [TODO] new_using_template when there is already a checkout causes error without clear message
  * [111] [TODO] don't think this is working, but there is a workaround
  * [135] [TODO] this method should also do the html stuff
  * [154] [TODO] check that true works instead of 1
  * [185] [TODO] Bugzilla 231125
  * [235] [TODO] add check that if the page is a new page (tool = 'Wiki'), the user needs to be set

lib/generic.rb:

  * [103] [TODO] is Tidy ignoring this option
  * [116] [TODO] integrate new tidy dll when it is available. Note: this will improve generated diffs with xhtmldiff

test/functional/comments_controller_test.rb:

  * [ 61] [TODO] test update

test/functional/feedbacks_controller_test.rb:

  * [ 50] [TODO] cannot test this because the controller uses request.referer

test/functional/pages_controller_test.rb:

  * [185] [TODO] Bug 218832 - RTE

test/functional/portal_controller_test.rb:

  * [134] [TODO] more advanced archives test

test/functional/uploads_controller_test.rb:

  * [ 88] [TODO] test this, currently we cannot test this because use of request.referer causes errors: "The error occurred while evaluating nil.[]"

test/functional/users_controller_test.rb:

  * [235] [TODO] implement test

test/integration/creating_sites_and_wikis_test.rb:

  * [ 59] [TODO] http://blog.finalist.com/2007/02/12/test-driven-development-met-ruby-on-rails/
  * [ 79] [TODO] can't test this because we use flash.now, this only works without 'now'
  * [126] [TODO] test for sending of email
  * [209] [TODO] cleanup

test/integration/setup_and_sign_in_test.rb:

  * [143] [TODO] Rails bug?
  * [144] [TODO] Rails bug?
  * [163] [TODO] we can't use cookies[:epfwiki_id] anymore?

test/test_helper.rb:

  * [100] [TODO] implement functional test for confirm message

test/unit/checkout_test.rb:

  * [ 35] [TODO] 5. we cannot checkout a page in a site that has not been wikified yet
  * [ 36] [TODO] 6. we can supply a note with a checkout
  * [ 37] [TODO] 7. checkout removes
  * [ 47] [TODO] test for message, how "RuntimeError: Versions can only be created in Wiki sites"
  * [ 72] [TODO] mixed results with this, caused by page.checkout, that doesn't seem to work
  * [ 97] [TODO] 5. don't supply html which will checkin using the version file

test/unit/notifier_test.rb:

  * [ 31] [TODO] include all record types:

test/unit/user_test.rb:

  * [246] [TODO] fails
  * [287] [TODO] we can update attributes

test/unit/version_test.rb:

  * [180] [TODO] (but now now) note part of it is in the previous test