Changeset 28709

Timestamp:

Jun 17, 2014, 1:11:44 PM (10 years ago)

Author:

plg

Message:

feature 3093: search form, ability to select the list of properties on which
the search terms applies.

feature 3094: minor redesign on search form.

Location:

trunk

Files:

• include/functions_search.inc.php (modified) (1 diff)
• language/en_UK/common.lang.php (modified) (1 diff)
• language/fr_FR/common.lang.php (modified) (1 diff)
• search.php (modified) (2 diffs)
• themes/default/template/search.tpl (modified) (5 diffs)
• themes/default/theme.css (modified) (3 diffs)

trunk/include/functions_search.inc.php

@@ -86 +86 @@
   if (isset($search['fields']['allwords']))
   {
-    $fields = array('file', 'name', 'comment', 'author');
+    $fields = array('file', 'name', 'comment');
+
+    if (isset($search['fields']['allwords']['fields']) and count($search['fields']['allwords']['fields']) > 0)
+    {
+      $fields = array_intersect($fields, $search['fields']['allwords']['fields']);
+    }
+    
     // in the OR mode, request bust be :
     // ((field1 LIKE '%word1%' OR field2 LIKE '%word1%')

trunk/language/en_UK/common.lang.php

@@ -420 +420 @@
 $lang['Theme'] = 'Theme';
 $lang['No results for'] = 'No results for';
+$lang['Apply on properties'] = 'Apply on properties';
+$lang['Photo title'] = 'Photo title';
+$lang['Photo description'] = 'Photo description';
 ?>

trunk/language/fr_FR/common.lang.php

@@ -420 +420 @@
 $lang['Theme'] = 'Thème';
 $lang['No results for'] = 'Aucun résultat pour';
+$lang['Apply on properties'] = 'Appliquer sur les propriétés';
+$lang['Photo title'] = 'Titre de la photo';
+$lang['Photo description'] = 'Description de la photo';

trunk/search.php

@@ -49 +49 @@
   {
     check_input_parameter('mode', $_POST, false, '/^(OR|AND)$/');
+
+    $fields = array_intersect($_POST['fields'], array('name', 'comment', 'file'));
     
     $drop_char_match = array(
@@ -70 +72 @@
         ),
       'mode' => $_POST['mode'],
+      'fields' => $fields,
       );
   }

trunk/themes/default/template/search.tpl

@@ -32 +32 @@
 <form class="filter" method="post" name="search" action="{$F_SEARCH_ACTION}">
 <fieldset>
-  <legend>{'Filter'|@translate}</legend>
-  <label>{'Search for words'|@translate}
-    <input type="text" name="search_allwords" size="35">
-  </label>
-  <ul>
-    <li><label>
-      <input type="radio" name="mode" value="AND" checked="checked">{'Search for all terms'|@translate}
-    </label></li>
-    <li><label>
-      <input type="radio" name="mode" value="OR">{'Search for any term'|@translate}
-    </label></li>
-  </ul>
-  <label>{'Search for Author'|@translate}
+  <legend>{'Search for words'|@translate}</legend>
+  <p>
+    <input type="text" name="search_allwords">
+    <br>
+    <label><input type="radio" name="mode" value="AND" checked="checked">{'Search for all terms'|@translate}</label>
+    <label><input type="radio" name="mode" value="OR">{'Search for any term'|@translate}</label>
+  </p>
+
+  <p>
+    <strong>{'Apply on properties'|translate}</strong><br>
+    <label><input type="checkbox" name="fields[]" value="name" checked="checked">{'Photo title'|translate}</label>
+    <label><input type="checkbox" name="fields[]" value="comment" checked="checked">{'Photo description'|translate}</label>
+    <label><input type="checkbox" name="fields[]" value="file" checked="checked">{'File name'|translate}</label>
+  </p>
+
+</fieldset>
+
+<fieldset>
+  <legend>{'Search for Author'|@translate}</legend>
+  <p>
     <select id="authors" placeholder="{'Type in a search term'|translate}" name="authors[]" multiple style="width:500px;">
 {foreach from=$AUTHORS item=author}
@@ -50 +57 @@
 {/foreach}
     </select>
-  </label>
+  </p>
 </fieldset>
 
@@ -56 +63 @@
 <fieldset>
   <legend>{'Search tags'|@translate}</legend>
-
+  <p>
   <select id="tags" placeholder="{'Type in a search term'|translate}" name="tags[]" multiple style="width:500px;">
 {foreach from=$TAGS item=tag}
@@ -62 +69 @@
 {/foreach}
   </select>
-
   <label><span><input type="radio" name="tag_mode" value="AND" checked="checked"> {'All tags'|@translate}</span></label>
   <label><span><input type="radio" name="tag_mode" value="OR"> {'Any tag'|@translate}</span></label>
+  </p>
 </fieldset>
 {/if}
@@ -121 +128 @@
 <fieldset>
   <legend>{'Search in albums'|@translate}</legend>
-  <label>{'Albums'|@translate}
+  <p>
     <select class="categoryList" name="cat[]" multiple="multiple" size="15">
       {html_options options=$category_options selected=$category_options_selected}
     </select>
-  </label>
-  <ul>
-    <li><label>{'Search in sub-albums'|@translate}</label></li>
-    <li><label>
-      <input type="radio" name="subcats-included" value="1" checked="checked">{'Yes'|@translate}
-    </label></li>
-    <li><label>
-      <input type="radio" name="subcats-included" value="0">{'No'|@translate}
-    </label></li>
-  </ul>
+    <br>
+    <label><input type="checkbox" name="subcats-included" value="1" checked="checked">{'Search in sub-albums'|@translate}</label>
+  </p>
 </fieldset>
 <p>

trunk/themes/default/theme.css

@@ -510 +510 @@
 }
 
-.filter UL,
-.filter LABEL {
+.filter UL {
         display: block;
         float: left;
@@ -523 +522 @@
 }
 
+.filter FIELDSET {
+  padding-bottom:0;
+}
+
 .filter LI LABEL {
         display: inline;
@@ -528 +531 @@
 }
 
+.filter P {
+  text-align:left;
+  margin:0 0 15px 0;
+  line-height:20px;
+}
+
+.filter input[name="search_allwords"] {
+  width:500px;
+}
+
+.filter P input[type="checkbox"], .filter P input[type="radio"] {
+  vertical-align:middle;
+  margin-top:-4px;
+}
+
+.filter input[type="submit"] {
+  margin-left:1em;
+}
+
+.filter P LABEL {
+  margin-right:1em;
+}
+
 /*IE6 needs override because of > selector*/
 .filter LABEL>INPUT,
 .filter LABEL>SELECT {
-        display: block;
+        /* display: block; */
         margin: 0.5em 0;
 }