RC for prenotification

Author: lmrodriguezr

app/controllers/registers_controller.rb

@@ -204,14 +204,24 @@ def endorse
 
   # GET /registers/r:abc/prenotify
   def prenotify
+    @errored ||= {}
     @genomes = @register.names.map(&:type_genome).compact.select(&:pending?)
     redirect_to(notify_register_path(@register)) if @genomes.empty?
   end
 
   # POST /registers/r:abc/prenotify
   def prenotify_commit
+    genomes = params.require(:genome)
+    @errored = {}
+    genomes.each do |k, v|
+      genome = Genome.find(k)
+      unless genome.update_accession(v['accession'], v['database'])
+        @errored[k.to_i] = 'Cannot update accession'
+      end
+    end
+
     prenotify
-    render(:prenotify)
+    render(:prenotify) unless @genomes.empty?
   end
 
   # GET /registers/r:abc/notify

app/views/registers/prenotify.html.erb

@@ -22,6 +22,9 @@
         <th scope=col>Temporary designation</th>
         <th scope=col>Database</th>
         <th scope=col>Accession</th>
+        <% unless @errored.present? %>
+          <th scope=col>Errors</th>
+        <% end %>
       </tr>
     </thead>
     <tbody>
@@ -38,6 +41,9 @@
           <td><%= text_field_tag(
             "genome[#{genome.id}][accession]", nil, class: 'form-control'
           ) %></td>
+          <% if @errored[genome.id] %>
+            <td class="text-danger"><%= @errored[genome.id] %></td>
+          <% end %>
         </tr>
       <% end %>
     </tbody>
@@ -49,7 +55,3 @@
   <br/><br/>
 <% end %>
 
-<pre>
-<%= params.inspect %>
-</pre>
-