Roundup Tracker - Issues

Message7076

Author rouilj
Recipients cmeerw, rouilj, schlatterbeck, tttech-klonner
Date 2021-02-11.23:08:45
Message-id <1613084926.41.0.49522541279.issue2551112@roundup.psfhosted.org>
In-reply-to
Hi Christof:

Ralf do you have feedback here?

Christof, I think you are saying that the problem is with inheriting 
RequestDispatcher and replacing the __init__ method right?

If that is true, then I think this patch can stand. The new
class: MyRequestDispatcher (in your example) with the simple __init__
will have a delay on the first request by not pre-compiling
templates by calling get_tracker in __init__ but I don't see that as 
show stopper.

You mentioned you wanted this configurable, what would be the
advantage of adding another config.ini option? If you were going
to write a new __init__ method anyway that can remove the get_tracker
dependency from __init__ anyway.

Also are you inheriting RequestDispatcher in your current trackers
so this change actually breaks something?

-- rouilj
History
Date User Action Args
2021-02-11 23:08:46rouiljsetmessageid: <1613084926.41.0.49522541279.issue2551112@roundup.psfhosted.org>
2021-02-11 23:08:46rouiljsetrecipients: + rouilj, schlatterbeck, cmeerw, tttech-klonner
2021-02-11 23:08:46rouiljlinkissue2551112 messages
2021-02-11 23:08:45rouiljcreate