diff options
| author | Manuel Urbano Santos <mu@member.fsf.org> | 2011-11-27 13:49:47 +0100 |
|---|---|---|
| committer | Manuel Urbano Santos <mu@member.fsf.org> | 2011-11-27 13:49:47 +0100 |
| commit | 3038ba87e4fdc5612f57affeedb643a614e0c9a2 (patch) | |
| tree | 81b55b7da2215905a63972f3501291d32269d3d0 /mediagoblin/tools/text.py | |
| parent | 30188321531e1b0d3c78166498702bbd8c7dc2bc (diff) | |
| download | mediagoblin-3038ba87e4fdc5612f57affeedb643a614e0c9a2.tar.lz mediagoblin-3038ba87e4fdc5612f57affeedb643a614e0c9a2.tar.xz mediagoblin-3038ba87e4fdc5612f57affeedb643a614e0c9a2.zip | |
* Bug #671: Tags list on Edit page is not seperated by spaces and hard to read : Make 'media_tags_as_string' function put a space after each comma.
* Feature #678: Drop custom delimiters in tags : I declare a constant in the begining of text.py file.
Diffstat (limited to 'mediagoblin/tools/text.py')
| -rw-r--r-- | mediagoblin/tools/text.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/mediagoblin/tools/text.py b/mediagoblin/tools/text.py index be1adb00..d576224d 100644 --- a/mediagoblin/tools/text.py +++ b/mediagoblin/tools/text.py @@ -43,6 +43,7 @@ HTML_CLEANER = Cleaner( host_whitelist=(), whitelist_tags=set([])) +TAGS_DELIMITER=','; def clean_html(html): # clean_html barfs on an empty string @@ -67,7 +68,7 @@ def convert_to_tag_list_of_dicts(tag_string): # Split the tag string into a list of tags for tag in stripped_tag_string.split( - mg_globals.app_config['tags_delimiter']): + TAGS_DELIMITER): # Ignore empty or duplicate tags if tag.strip() and tag.strip() not in [t['name'] for t in taglist]: @@ -85,7 +86,7 @@ def media_tags_as_string(media_entry_tags): """ media_tag_string = '' if media_entry_tags: - media_tag_string = mg_globals.app_config['tags_delimiter'].join( + media_tag_string = (TAGS_DELIMITER+u' ').join( [tag['name'] for tag in media_entry_tags]) return media_tag_string |