oarevalo / BugLogHQ

BugLogHQ is a tool to centralize the handling of automated bug reports from multiple applications. BugLogHQ provides a unified view of error messages sent from any number of applications, allowing the developer to search, graph, forward, and explore the bug reports submitted by the applications.
http://www.bugloghq.com
154 stars 67 forks source link

Purge Scheduler Bug #63

Closed cfarleyaz closed 11 years ago

cfarleyaz commented 11 years ago

We kept enabling the Purge option and it would never purge and could not figure out why. Ends up if you have purge enabled, but digest NOT enabled there's a typo in the "configureDigest" function (in BugLogListener.cfc).

If digest is not enabled, it actually removes the purge CRON job, we updated this line within the "configureDigest" function - line #444 in our copy:

<cfset scheduler.removeTask("bugLogPurgeHistory") />

TO:

<cfset scheduler.removeTask("bugLogSendDigest") />

So if digest is NOT enabled, it removes the digest CRON job - not the purge one.

Figured you would want to know-

atuttle commented 11 years ago

You've not formatted your code samples properly. Check this out for more info: http://github.github.com/github-flavored-markdown/

oarevalo commented 11 years ago

Thanks! can't see your fix though.

atuttle commented 11 years ago

Oscar, since you're the repo owner you can edit the original issue content. Click the edit button and you'll be able to see it all. Just wrap the code in back-ticks, like so:

code here

cfarleyaz commented 11 years ago

sorry first post to github, think I updated it correctly so you can see the code

oarevalo commented 11 years ago

Thanks again, I added your fix to the 1.8 branch. I'll merge shortly into master to bring the current version up to 1.8.2 (with a few other minor changes that have been waiting on the branch)