colink
|
Date: 3/24/2007 5:44 pm · Subject: Group expirations and notify time calculations · Rating: 1
In Operation/Group.pm, all of the time settings like expireOffset use the Interval form widget. That returns a time in seconds. Inside Group.pm, however, all the time settings are set to be in days. And, worse, inside Workflow/Activity/ExpireGroupings.pm, all times are considered days so they're multiplied by 86,400, which double converts them into seconds. The fix is to: - Update the documentation in Group.pm to denote that seconds, rather than days, are stored here.
- Remove all double conversion to seconds in ExpireGroupings.pm
- Check anyplace else that times are gathered via Interval.pm and make sure that this same bug isn't happening.
|
nogura
|
Date: 3/27/2007 12:58 am · Subject: Re: Group expirations and notify time calculations · Rating: -1
Fixed per your suggestions. I can't seem to verify that it's working. That may be because of my dev enviornment, or my newness. Please check it out at your convenience and let me know.
|
