diff options
author | Vitaly Takmazov | 2015-10-26 17:03:58 +0300 |
---|---|---|
committer | Vitaly Takmazov | 2015-10-26 17:03:58 +0300 |
commit | 9dfbc063653c729f7a74f1e02260073594fd66fa (patch) | |
tree | ac205fcd1b548919a380d4b490fb432e8c5e4328 /src/main/java/com/juick/http/www | |
parent | 40b5be875667c718f8732fd7597203c3324c4b2e (diff) |
allow empty tags
Diffstat (limited to 'src/main/java/com/juick/http/www')
-rw-r--r-- | src/main/java/com/juick/http/www/NewMessage.java | 32 |
1 files changed, 14 insertions, 18 deletions
diff --git a/src/main/java/com/juick/http/www/NewMessage.java b/src/main/java/com/juick/http/www/NewMessage.java index 34733511e..e2fe96294 100644 --- a/src/main/java/com/juick/http/www/NewMessage.java +++ b/src/main/java/com/juick/http/www/NewMessage.java @@ -141,27 +141,23 @@ public class NewMessage { body = body.replace("\r", ""); String tagsStr = request.getParameter("tags"); - if (tagsStr == null || tagsStr.isEmpty()) { - response.sendError(400); - return; - } - String tagsArr[] = tagsStr.split("[ \\,]"); - for (int i = 0; i < tagsArr.length; i++) { - if (tagsArr[i].startsWith("*")) { - tagsArr[i] = tagsArr[i].substring(1); + ArrayList<com.juick.Tag> tags = new ArrayList<Tag>(); + String tagsArr[] = new String[1]; + if (tagsStr != null && !tagsStr.isEmpty()) { + tagsArr = tagsStr.split("[ \\,]"); + for (int i = 0; i < tagsArr.length; i++) { + if (tagsArr[i].startsWith("*")) { + tagsArr[i] = tagsArr[i].substring(1); + } + if (tagsArr[i].length() > 64) { + tagsArr[i] = tagsArr[i].substring(0, 64); + } } - if (tagsArr[i].length() > 64) { - tagsArr[i] = tagsArr[i].substring(0, 64); + tags = TagQueries.getTags(sql, tagsArr, true); + while (tags.size() > 5) { + tags.remove(5); } } - ArrayList<com.juick.Tag> tags = TagQueries.getTags(sql, tagsArr, true); - if (tags.isEmpty()) { - response.sendError(400); - return; - } - while (tags.size() > 5) { - tags.remove(5); - } String attachmentFName = null; try { |