Todo List
MantisBT
- (though I'm not sure they need to be turned off anymore
- there just won't be anyone in those categories)
I guess it serves as an example and a placeholder for this config option
- yarick123: now it is implemented for notification email only
- yarick123: now it is implemented for notification email only
- review form security tokens for this page
- should page_top1 be before meta redirect?
- FIXME: Looks like "From", "to", and "Copy" need i18n. Possibly more in this file.
- (thraxisp) this may want a browser check ( MS IE >= 5.0, Mozilla >= 1.0, Safari >=1.2, ...)
CoreAPI
- Review use of $p_email within mantis
- when we rewrite authentication api for plugins, this should be merged with prepare_username and return some object
- when we rewrite authentication api for plugins, this should be merged with prepare_password and return some object
- Not managed FTP file upload
- include reporter name and handler name, the problem is that handler can be 0, in this case no corresponding name will be found. Use equivalent of (+) in Oracle.
- return values are unclear... should we error when access is denied and provide an api to check whether it will be?
- (thraxisp) this may want a browser check ( MS IE >= 5.0, Mozilla >= 1.0, Safari >=1.2, ...)
- (thraxisp) this may want a browser check ( MS IE >= 5.0, Mozilla >= 1.0, Safari >=1.2, ...)
- Use/Behaviour of this function should be reviewed before 1.2.0 final
- Use/Behaviour of this function should be reviewed before 1.2.0 final
- Use/Behaviour of this function should be reviewed before 1.2.0 final
- Check if there is a way to do that using ADODB rather than implementing it here.
- add param bool $p_gmt whether to use GMT or current timezone (default false)
- Use/Behaviour of this function should be reviewed before 1.2.0 final
- Use/Behaviour of this function should be reviewed before 1.2.0 final
- Use/Behaviour of this function should be reviewed before 1.2.0 final
- Use/Behaviour of this function should be reviewed before 1.2.0 final
- limit_email_domain called after we look for @ in domain name?
- I'm not sure this shouldn't return an array of user ids... more work for the caller but cleaner from an API point of view.
- yarick123: email_collect_recipients(...) will be completely rewritten to provide additional information such as language, user access,..
- yarick123:sort recipients list by language to reduce switches between different languages
- In case of synchronous email sending, we may get a race condition where two requests send the same email.
- function needs to be abstracted
- is a check for these necessary? if ( ( $t_filter_value === 'none' ) || ( $t_filter_value === '[none]' ) )
- Had to make all these parameters required because we can't use call-time pass by reference anymore. I really preferred not having to pass all the params in if you didn't want to, but I wanted to get rid of the errors for now. If we can think of a better way later (maybe return an object) that would be great.
- this function is to be modified by Victor to add CRC... for now it just passes the parameters through to setcookie()
- improve this formatting - to only be about 50% of the screen width so that it doesn't become hard to read.
- Implement caching by retrieving all needed information in one query.
- Implement logging to LDAP queries same way like DB queries.
- Javascript should be removed and added dynamically via external script
- relationship of this function to profile_get_row?
- add some currency formating in the future
- This probably belongs in the string_api.php
|
|