From fef2775941cf2e0f6f35ae1ee838c004b2ddf913 Mon Sep 17 00:00:00 2001 From: Arun Prakash Jana Date: Wed, 1 Jun 2016 03:10:51 +0530 Subject: [PATCH] A more robust parse_args(). --- buku | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/buku b/buku index 89f7352..fb2a42f 100755 --- a/buku +++ b/buku @@ -856,7 +856,7 @@ def parse_tags(keywords=None): """Format and get tag string from tokens""" if keywords is None: - keywords = [] + return None tags = DELIMITER origTags = [] @@ -1459,7 +1459,7 @@ if __name__ == '__main__': # Parse tags into a comma-separated string tags = DELIMITER keywords = args.addurl - if tagManual is not None and not (tagManual[0] == DELIMITER and len(tagManual) == 1): + if tagManual is not None: # Add DELIMITER as url+tags may not end with comma keywords = args.addurl + [DELIMITER] + tagManual @@ -1484,10 +1484,7 @@ if __name__ == '__main__': new_url = '' # Parse tags into a comma-separated string - tags = None - if tagManual is not None and not (tagManual[0] == DELIMITER and len(tagManual) == 1): - tags = parse_tags(tagManual) - + tags = parse_tags(tagManual) bdb.update_bookmark(int(args.update[0]), new_url, titleManual, tags, description) # Delete record(s)