fixes #1139, fix other history page inconsistencies

app/views/artist_versions/index.html.erb

@@ -14,7 +14,9 @@
             <th>Updated by</th>
             <th>Active</th>
             <th>URLs</th>
-            <th></th>
+            <% if CurrentUser.is_member? %>
+              <th></th>
+            <% end %>
           </tr>
         </thead>
         <tbody>
@@ -34,11 +36,11 @@
                   <%= artist_version_urls_diff(artist_version) %>
                 </ul>
               </td>
-              <td>
-                <% if CurrentUser.is_member? %>
+              <% if CurrentUser.is_member? %>
+                <td>
                   <%= link_to "Revert", revert_artist_path(artist_version.artist_id, :version_id => artist_version.id), :method => :put, :confirm => "Are you sure you want to revert this artist?" %>
-                <% end %>
-              </td>
+                </td>
+              <% end %>
             </tr>
           <% end %>
         </tbody>

app/views/note_versions/index.html.erb

@@ -8,13 +8,15 @@
           <th></th>
           <th width="5%">Post</th>
           <th width="5%">Note</th>
-          <th width="50%">Body</th>
+          <th>Body</th>
           <% if CurrentUser.is_janitor? %>
             <th width="10%">IP Address</th>
           <% end %>
           <th width="10%">Edited By</th>
           <th width="10%">Date</th>
-          <th width="10%">Options</th>
+          <% if CurrentUser.is_member? %>
+            <th width="5%"></th>
+          <% end %>
         </tr>
       </thead>
       <tbody>
@@ -31,11 +33,11 @@
             <% end %>
             <td><%= link_to_user note_version.updater %></td>
             <td><%= compact_time note_version.updated_at %></td>
-            <td>
-              <% if CurrentUser.is_member? %>
+            <% if CurrentUser.is_member? %>
+              <td>
                 <%= link_to "Revert", revert_note_path(note_version.note_id, :version_id => note_version.id), :remote => true, :method => :put, :confirm => "Do you really want to revert to this version?" %>
-              <% end %>
-            </td>
+              </td>
+            <% end %>
           </tr>
         <% end %>
       </tbody>

app/views/pool_versions/index.html.erb

@@ -9,9 +9,13 @@
           <th>Post Count</th>
           <th>Changes</th>
           <th>Updater</th>
-          <th>IP Address</th>
+          <% if CurrentUser.is_janitor? %>
+            <th>IP Address</th>
+          <% end %>
           <th>Date</th>
-          <th></th>
+          <% if CurrentUser.is_member? %>
+            <th></th>
+          <% end %>
         </tr>
       </thead>
       <tbody>
@@ -21,13 +25,17 @@
             <td><%= link_to pool_version.post_id_array.size, pool_versions_path(:search => {:pool_id => pool_version.pool_id}) %></td>
             <td><%= pool_version_diff(pool_version) %></td>
             <td><%= link_to_user pool_version.updater%></td>
-            <td>
-              <% if CurrentUser.is_janitor? %>
+            <% if CurrentUser.is_janitor? %>
+              <td>
                 <%= pool_version.updater_ip_addr %>
-              <% end %>
-            </td>
+              </td>
+            <% end %>
             <td><%= compact_time pool_version.updated_at %></td>
-            <td><%= link_to "Revert", revert_pool_path(pool_version.pool_id, :version => pool_version.id) %></td>
+            <% if CurrentUser.is_member? %>
+              <td>
+                <%= link_to "Revert", revert_pool_path(pool_version.pool_id, :version => pool_version.id) %>
+              </td>
+            <% end %>
           </tr>
         <% end %>
       </tbody>

app/views/post_versions/_listing.html.erb

@@ -10,8 +10,10 @@
         <% if CurrentUser.is_admin? %>
           <th width="10%">IP Address</th>
         <% end %>
-        <th width="45%">Tags</th>
-        <th width="5%"></th>
+        <th>Tags</th>
+        <% if CurrentUser.is_member? %>
+          <th width="5%"></th>
+        <% end %>
       </tr>
     </thead>
     <tbody>
@@ -32,9 +34,11 @@
             </td>
           <% end %>
           <td><%= post_version_diff(post_version) %></td>
-          <td>
-            <%= link_to "Revert", revert_post_path(post_version.post_id, :version_id => post_version.id), :method => :put, :remote => true %>
-          </td>
+          <% if CurrentUser.is_member? %>
+            <td>
+              <%= link_to "Revert", revert_post_path(post_version.post_id, :version_id => post_version.id), :method => :put, :remote => true %>
+            </td>
+          <% end %>
         </tr>
       <% end %>
     </tbody>

app/views/wiki_page_versions/index.html.erb

@@ -11,7 +11,7 @@
             <tr>
               <th width="2%"></th>
               <th width="2%"></th>
-              <th width="55%">Title</th>
+              <th>Title</th>
               <th width="5%"></th>
               <% if CurrentUser.is_admin? %>
                 <th width="10%">IP Address</th>