feat: make valid org filter field a dropdown (#10524)

* make valid org filter field a dropdown * update html file * display filter form when no data returned * add option for case of not selection
openfoodfacts · Jul 9, 2024 · b38e62c · b38e62c
1 parent d4f05b9
commit b38e62c
Showing 1 changed file with 33 additions and 28 deletions.
diff --git a/templates/web/pages/dashboard/display_orgs_table.tt.html b/templates/web/pages/dashboard/display_orgs_table.tt.html
@@ -1,16 +1,21 @@
 <!-- start templates/[% template.name %] -->
 
-[% IF has_orgs %]
 <form method="GET" action="/cgi/display_org_table.pl">
     <label for="name">[% lang("name") %]:</label>
     <input type="text" id="name" name="name" value="[% name | html %]">
-    
+
     <label for="valid_org">[% lang("verified_status") %]:</label>
-    <input type="text" id="valid_org" name="valid_org" value="[% valid_org | html %]">
+    <select id="valid_org" name="valid_org">
+        <option value="" [% IF !valid_org %]selected[% END %]>-- Select status --</option>
+        <option value="unreviewed" [% IF valid_org == 'unreviewed' %]selected[% END %]>Unreviewed</option>
+        <option value="accepted" [% IF valid_org == 'accepted' %]selected[% END %]>Accepted</option>
+        <option value="rejected" [% IF valid_org == 'rejected' %]selected[% END %]>Rejected</option>
+    </select>
 
     <button type="submit">[% lang("search") %]</button>
 </form>
 
+[% IF has_orgs %]
 <table id="tagstable">
     <thead>
         <tr>
@@ -28,35 +33,35 @@
     <tbody>
         [% SET count = 1 %]
         [% FOREACH org IN orgs %]
-            <tr>
-                <td>[% count %].</td>
-                <td>[% org.name %]</td>
-                <td>[% org.creator %]</td>
-                <td>[% org.org_id %]</td>
-                <td>[% org.valid_org %]</td>
-                <td>[% display_date_tag(org.created_t) %]</td>
-                <td>
-                    [% IF org.last_logged_member_t %]
-                        [% display_date_tag(org.last_logged_member_t) %]
-                    [% ELSE %]
-                        &nbsp;
-                    [% END %]
-                </td>
-                <td>
-                    [% IF org.last_import_t %]
-                        [% display_date_tag(org.last_import_t) %]
-                    [% ELSE %]
-                        &nbsp;
-                    [% END %]
-                </td>
-                <td>[% org.last_import_type %]</td>
-            </tr>
-            [% SET count = count + 1 %]
+        <tr>
+            <td>[% count %].</td>
+            <td>[% org.name %]</td>
+            <td>[% org.creator %]</td>
+            <td>[% org.org_id %]</td>
+            <td>[% org.valid_org %]</td>
+            <td>[% display_date_tag(org.created_t) %]</td>
+            <td>
+                [% IF org.last_logged_member_t %]
+                    [% display_date_tag(org.last_logged_member_t) %]
+                [% ELSE %]
+                    &nbsp;
+                [% END %]
+            </td>
+            <td>
+                [% IF org.last_import_t %]
+                    [% display_date_tag(org.last_import_t) %]
+                [% ELSE %]
+                    &nbsp;
+                [% END %]
+            </td>
+            <td>[% org.last_import_type %]</td>
+        </tr>
+        [% SET count = count + 1 %]
         [% END %]
     </tbody>
 </table>
 [% ELSE %]
-<p style="font-size: 1.2rem;">No organizations found</p>
+<p style="font-size: 2rem; padding-top: 40px;">No organizations found.</p>
 [% END %]
 
 <!-- end templates/[% template.name %] -->