Edgewall Software
Modify

Opened 13 years ago

Closed 11 years ago

#5052 closed enhancement (worksforme)

Attempt to load SilverCity once per session

Reported by: quarl@… Owned by: Christian Boos
Priority: low Milestone:
Component: wiki system Version:
Severity: minor Keywords: mimeview renderer macro
Cc: Branch:
Release Notes:
API Changes:

Description

Is it possible to only attempt to load SilverCity once per session?

I get a million traceback warnings in my trac log file because every render call attempts to load SilverCity, then falls back to enscript.

It would also be more efficient to only attempt to load it once.

Regards, Karl

Attachments (1)

loader.py.diff (1.3 KB ) - added by dharris <dharris+trac@…> 12 years ago.
Patch for loader.py to reduce errors on load failures

Download all attachments as: .zip

Change History (6)

comment:1 by quarl, 13 years ago

Another approach would be to change the log level to DEBUG.

comment:2 by Christian Boos, 13 years ago

Component: generalwiki
Keywords: mimeview renderer macro added
Milestone: 1.0
Owner: changed from Jonas Borgström to Christian Boos
Priority: normallow

Agreed, we could be a bit smarter here.

comment:3 by dharris <dharris+trac@…>, 12 years ago

Perhaps log level INFO would be more appropriate until the code is changed to check less frequently.

I'm attaching a simple patch I've applied to log these at INFO.

by dharris <dharris+trac@…>, 12 years ago

Attachment: loader.py.diff added

Patch for loader.py to reduce errors on load failures

in reply to:  3 comment:4 by Emmanuel Blot, 12 years ago

Replying to dharris <dharris+trac@gmail.com>:

Perhaps log level INFO would be more appropriate until the code is changed to check less frequently.

I think this is more a piece of information for developers or admins who wish to debug their installation, so DEBUG seems more appropriate here, especially if the same information keeps being printed with no added value.

comment:5 by Christian Boos, 11 years ago

Milestone: 1.0
Resolution: worksforme
Status: newclosed

This is no longer an issue.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.
to as closed The owner will be changed from Christian Boos to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.