Merge pull request mozilla-services#639 from ossreleasefeed/display-u…

…ser-msg-on-empty-result-760637 providing user message when no results are returned fixes bug 760637
mr-const · Jun 6, 2012 · a3608a5 · a3608a5
2 parents 482bc49 + 8873320
commit a3608a5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/webapp-php/application/views/common/signature_urls.php b/webapp-php/application/views/common/signature_urls.php
@@ -1,4 +1,5 @@
 <div id="sigurls" class="ui-tabs-hide">
+    <?php if(count($urls) > 0) { ?>
     <table id="sigurls-tbl" class="tablesorter data-table">
         <thead>
             <tr>
@@ -18,4 +19,7 @@
         <?php } ?>
         </tbody>
     </table>
+    <?php } else { ?>
+        <p>No URLs found for signature <?php if (isset($display_signature)) { out::H($display_signature); } ?></p>
+    <?php } ?>
 </div>
diff --git a/webapp-php/application/views/report/do_list.php b/webapp-php/application/views/report/do_list.php
@@ -191,6 +191,7 @@
     if ($logged_in) {
         View::factory('common/signature_urls', array(
             'urls' => $urls,
+            'display_signature' => $display_signature
         ))->render(TRUE);
     }
 ?>