Polish qdp (o19s#1681)

* look up by the qdp id now supported

* complete placeholder message for qdp id

* show in hidden html the details.

Author: epugh

app/controllers/query_doc_pairs_controller.rb

@@ -12,8 +12,8 @@ def index
     query = @book.query_doc_pairs
 
     if params[:q].present?
-      query = query.where('query_text LIKE ? OR doc_id LIKE ? OR document_fields LIKE ?',
-                          "%#{params[:q]}%", "%#{params[:q]}%", "%#{params[:q]}%")
+      query = query.where('query_text LIKE ? OR query_doc_pairs.id LIKE ? OR doc_id LIKE ? OR document_fields LIKE ?',
+                          "%#{params[:q]}%", "%#{params[:q]}%", "%#{params[:q]}%", "%#{params[:q]}%")
     end
 
     # Eager load judgements count to avoid N+1 queries when showing the count

app/views/judgements/_form.html.erb

@@ -1,3 +1,12 @@
+<% content_for :debug_info do %>
+  <!--
+  BOOK_ID: <%= @book.id %>
+  QUERY_DOC_PAIR_ID: <%= query_doc_pair.id %>
+  JUDGEMENT_ID: <%= judgement.id %>
+  JUDGEMENT USER ID: <%= judgement&.user_id %>
+  CURRENT USER ID: <%= @current_user.id %>
+  -->
+<% end %>
 
 <script>
   function rate(rating, key_name) {
@@ -12,12 +21,12 @@
       keyboard_shortcuts_suffixes.forEach(suffix => {
           const spanElement = document.getElementById("span-for-" + suffix);
           const buttonElement = document.getElementById("button-for-" + suffix);
-          
+
           if (spanElement) {
               spanElement.style.color = "gray";
               spanElement.style.fontWeight = "normal";
           }
-          
+
           if (buttonElement) {
               buttonElement.classList.remove("btn-preselected");
           }
@@ -30,12 +39,12 @@
           if (suffix !== except_suffix) {
               const spanElement = document.getElementById("span-for-" + suffix);
               const buttonElement = document.getElementById("button-for-" + suffix);
-              
+
               if (spanElement) {
                   spanElement.style.color = "gray";
                   spanElement.style.fontWeight = "normal";
               }
-              
+
               if (buttonElement) {
                   buttonElement.classList.remove("btn-preselected");
               }
@@ -47,12 +56,12 @@
       resetStyles();
       const spanElement = document.getElementById("span-for-" + suffix);
       const buttonElement = document.getElementById("button-for-" + suffix);
-      
+
       if (spanElement) {
           spanElement.style.color = "black";
           spanElement.style.fontWeight = "bold";
       }
-      
+
       if (buttonElement) {
           buttonElement.classList.add("btn-preselected");
       }
@@ -91,7 +100,7 @@
       }
 
       const keyCode = e.which || e.keyCode;
-      
+
       if (keyCode == 97 || keyCode == 65) {
           keyPressed("a");
       } else if (keyCode == 115 || keyCode == 83) {
@@ -120,9 +129,9 @@
       if (isExplanationModalOpen() || isTypingInTextField()) {
         return;
       }
-      
+
       const keyCode = e.which || e.keyCode;
-      
+
       if (keyCode == 97 || keyCode == 65) {
           keyDown("a");
       } else if (keyCode == 115 || keyCode == 83) {
@@ -211,7 +220,7 @@
     </div>
 
 
-    <div class="float-end">        
+    <div class="float-end">
       <%= link_to 'I will Judge Later', book_query_doc_pair_judge_later_path(@book, query_doc_pair), class: 'btn btn-info m-1', role: 'button' %>
       <br/>
       <button type="button" class="btn btn-warning m-1" data-bs-toggle="modal" data-bs-target="#explanationModal">
@@ -222,12 +231,12 @@
     <p class="card-text">
       <%= render 'query_doc_pairs/document_fields', document_fields: @document_fields %>
     </p>
-    
+
     <div class="mt-3">
       <label for="judgement_explanation" class="form-label">Explanation (optional)</label>
       <%= form.text_area :explanation, class: 'form-control', rows: 2, placeholder: 'Why did you choose this rating?' %>
     </div>
-    
+
     <% if @book.show_rank? %>
     <p class="card-text">
       <b>Document Rank: </b><%= judgement.query_doc_pair.position %>
@@ -275,7 +284,7 @@
         </tr>
       </table>
     <% end %>
-    
+
 
   </div>
   <% if image_url %>

app/views/layouts/application.html.erb

@@ -1,21 +1,24 @@
 <!DOCTYPE html>
 <html>
+<% if content_for?(:debug_info) %>
+  <%= yield :debug_info %>
+<% end %>
 <head>
   <meta charset="utf-8"/>
   <meta name="viewport" content="width=device-width,initial-scale=1">
 	<meta name="apple-mobile-web-app-capable" content="yes">
 	<meta name="mobile-web-app-capable" content="yes">
   <%= action_cable_meta_tag %>
   <base href="<%= ENV['RAILS_RELATIVE_URL_ROOT'] %>/">
-  
+
   <%= favicon_link_tag asset_path "favicon.ico" %>
 
 
 <% if flash[:unfurl] %>
   <!-- We have unfurl -->
-  
+
   <title>Quepid: Case <%= flash[:unfurl]["case_name"] %></title>
-  
+
   <!-— facebook open graph tags -->
   <meta property="og:type" content="website" />
   <% if flash[:unfurl]["id"] %>
@@ -24,7 +27,7 @@
   <meta property="og:title" content="Case <%= flash[:unfurl]["case_name"] %>" />
   <meta property="og:description" content="Case <%= flash[:unfurl]["case_name"] %> Description here." />
   <meta property="og:image" content="<%= image_url "quepid-icon.png" %>" />
-  
+
   <!-— twitter card tags additive with the og: tags -->
   <meta name="twitter:card" content="summary">
   <meta name="twitter:site" content="@quepid_search">
@@ -38,17 +41,17 @@
     <meta name="twitter:data1" value="<%=Case.find_by_id(flash[:unfurl]["id"]).tries.latest.try_number %>" />
   <% end %>
 <% else %>
-  
+
   <title>Quepid -- Relevant Search Solved</title>
-  
+
   <!-— facebook open graph tags -->
   <meta property="og:type" content="website" />
   <meta name="twitter:site" content="@quepid_search">
   <meta property="og:url" content="<%= root_path %>" />
   <meta property="og:title" content="Quepid -- Relevant Search Solved" />
   <meta property="og:description" content="Tired of irrelevant search results? Use Quepid products & services to help improve the quality and relevancy of your search results." />
   <meta property="og:image" content="<%= image_url "quepid-icon.png" %>" />
-  
+
   <!-— twitter card tags additive with the og: tags -->
   <meta name="twitter:card" content="summary">
   <meta name="twitter:domain" value="<%= URI.parse(root_url).host %>" />
@@ -65,7 +68,7 @@
 
   <%= stylesheet_link_tag 'application', media: 'all', 'data-turbo-track': 'reload' %>
   <%= csrf_meta_tags %>
-  
+
   <%= javascript_importmap_tags 'application_modern' %>
   <%= debugbar_head  if defined? Debugbar %>
 
@@ -77,17 +80,17 @@
 
 <div class="container-fluid">
   <div class="row">
-    
+
     <% if @current_user  && !(controller_name == 'judgements' and ['new','edit'].any? params[:action]) %>
       <%= render 'layouts/sidebar' %>
     <% end %>
-    
+
     <!-- Begin page content -->
     <main class="col-md-10 ms-sm-auto col-lg-<%= @current_user ? '11' : '12' %> px-md-5">
       <%= yield %>
     </main>
   </div>
 </div>
-<%= debugbar_body if defined? Debugbar %> 
+<%= debugbar_body if defined? Debugbar %>
 </body>
 </html>

app/views/query_doc_pairs/index.html.erb

@@ -3,7 +3,7 @@
   <div class="btn-toolbar mb-2 mb-md-0">
     <div class="btn-group me-2">
       <%= link_to 'New Query Doc Pair', new_book_query_doc_pair_path, class: 'btn btn-sm btn-outline-secondary' %>
-      <%= link_to "Back to Book", @book, class: 'btn btn-sm btn-outline-secondary' %>    
+      <%= link_to "Back to Book", @book, class: 'btn btn-sm btn-outline-secondary' %>
     </div>
   </div>
 </div>
@@ -18,7 +18,7 @@
         <div class="input-group" style="max-width: 400px;">
           <%= text_field_tag :q, params[:q],
               class: 'form-control form-control-sm',
-              placeholder: 'Filter by query, doc id, or document field contents',
+              placeholder: 'Filter by id, query, doc id, or document field contents',
               id: 'q'
           %>
           <%= button_tag type: 'submit', class: 'btn btn-sm btn-outline-secondary', id: 'button-addon2' do %>
@@ -40,7 +40,7 @@
       </div>
     <% end %>
   </div>
-  
+
   <div class="card-body">
     <table class="table table-hover table-sm">
       <thead>
@@ -55,22 +55,22 @@
           <th>Document fields</th>
         </tr>
       </thead>
-  
-      <tbody>      
+
+      <tbody>
         <% @query_doc_pairs.each do |query_doc_pair| %>
           <tr>
             <td>
               <%= link_to query_doc_pair.id, book_query_doc_pair_path(@book, query_doc_pair) %>
             </td>
-      
+
             <td><%= query_doc_pair.query_text %></td>
             <td><%= query_doc_pair.position %></td>
             <td><%= query_doc_pair.doc_id %></td>
             <% if @include_judgement_count %>
               <td>
                 <%= link_to_if query_doc_pair.judgements_count>0, query_doc_pair.judgements_count, book_judgements_path(@book, q: "query_doc_pair_id:#{query_doc_pair.id}") %>
               </td>
-            <% end %>            
+            <% end %>
             <td>
               <% if query_doc_pair.document_fields.present? %>
                 <% document_fields_json = query_doc_pair.document_fields.to_json %>
@@ -92,7 +92,7 @@
     </table>
   </div>
   <div class="card-footer d-flex justify-content-center">
-    <%== @pagy.series_nav(:bootstrap) %> 
+    <%== @pagy.series_nav(:bootstrap) %>
   </div>
 
   <!-- Document Fields Modal -->