fixed message manager bug with empty data: always create the <ul> even if there's no <li> within it

1 files changed, 3 insertions, 3 deletions

diff --git a/web/cobrands/fixmybarangay/message_manager_client.js b/web/cobrands/fixmybarangay/message_manager_client.js
index 6f76816ea..e9a5c8d6b 100644
--- a/web/cobrands/fixmybarangay/message_manager_client.js
+++ b/web/cobrands/fixmybarangay/message_manager_client.js
@@ -311,16 +311,16 @@ var message_manager = (function() {
         var messages = data.messages;
         _username = data.username;
         if (messages instanceof Array) {
+            var $ul = $('<ul class="mm-root mm-current"/>');
             if (messages.length === 0) {
-                $output.html('<p class="mm-empty">No messages available.</p>');
+                $output.append('<p class="mm-empty">No messages available.</p>');
             } else {
-                var $ul = $('<ul class="mm-root mm-current"/>');
                 for(i=0; i< messages.length; i++) {
                     litem = get_message_li(messages[i], 0);
                     $ul.append(litem);
                 }
-                $output.append($ul);
             }
+            $output.append($ul);
         } else {
             $output.html('<p>No messages (server did not send a list).</p>');
         }