On 03/28/2015 10:35 PM, Thomas De Schampheleire wrote:
On Sat, Mar 28, 2015 at 10:34 PM, Thomas De Schampheleire
<patrickdeping...@gmail.com> wrote:
# HG changeset patch
# User Thomas De Schampheleire <thomas.de.schamphele...@gmail.com>
# Date 1427573281 -3600
# Sat Mar 28 21:08:01 2015 +0100
# Node ID 2f4d67b8acefc3e85ddb5c165bc4054b0ddab90d
# Parent 65c5e70a1d0c1861d7dc835f204d132342920da2
helpers: add automatic logging to Flash
Log all messages displayed through a flash to the user.
The log level equals the flash category if it is defined and not 'success',
otherwise falls back to 'debug'.
Subsequent patches can change the log level for individual flash calls to
make the log more useful.
diff --git a/kallithea/lib/helpers.py b/kallithea/lib/helpers.py
--- a/kallithea/lib/helpers.py
+++ b/kallithea/lib/helpers.py
@@ -423,6 +423,29 @@
class Flash(_Flash):
+ def __call__(self, message, category=None, ignore_duplicate=False,
log_category=None):
+ '''Show a message to the user
+
+ category: notice (default), warning, error, success
+ log_category: debug, notice, warning, error, success
+
+ If log_category is not set, it defaults to 'debug' unless category is
+ set and different from 'success'.
+ '''
+ log_function = {
+ 'debug': log.debug,
+ 'notice': log.info,
+ 'success': log.info,
+ 'warning': log.warning,
+ 'error': log.error,
+ }
+
+ if not log_category:
+ log_category = category if category and category != 'success' else
'debug'
+
+ log_function[log_category]('Flash: %s' % message)
+ super(Flash, self).__call__(message, category, ignore_duplicate)
+
def pop_messages(self):
"""Return all accumulated messages and delete them from the session.
We could consider mapping 'category=warning' on 'log_category=debug'
too, as most such flashes are recategorized as such in the next patch.
In most cases, a warning flash means a user mistake/disallowed action
but not a real problem.
Hmm ... I guess this boils down to 3 cases:
1: the user did something wrong - do it better next time and everything
is fine
2: help the admin debug the setup or why the user action was wrong
3: help the developer debug during development or in a bug report
I don't know ... but it would be nice to have some standards for how we
handle these situations ... and it seems like developer debug and admin
debug is something different ...
Also, I don't know if we like or dislike _ or camel/pascal casing as in
"log_category". Leaving it undefined would however cause confusion. We
should make up our mind and put it in contributins.rts . The good thing
about the Mercurial convention is that it at least has an opinion ...
/Mads
_______________________________________________
kallithea-general mailing list
kallithea-general@sfconservancy.org
http://lists.sfconservancy.org/mailman/listinfo/kallithea-general
