posted 16 years ago
Finally got around to look at this code... my suggestion would be to have a constructor like:
AttachmentCommon( request, int id, boolean isPrivateMessage)
If privateMessage is true, then there should be a check against a new global config setting that verifies that private message attachments are allowed or not. This also means that the id parameter should be handled as the PM id rather than a Forum Id.
Also, it would be nice to verify that the Quota code can limit the maximum disk space that a user can use with stored PM attachments.
[originally posted on jforum.net by monroe]