paper_trail_history 0.1.3 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8b3e54f1afaacc531556a2a10d0bfde605e5e8e9bc02c4571d11d5fd1349cd00
-  data.tar.gz: e0f5777f1c40623e7ffa1cedff478b3b850c47e1418a1c52c4658b071c1a59a0
+  metadata.gz: e8f45c5efd0324c99a10d738d2c8db6e6f06b809ded91d04ae2ea1b5a7d6a30a
+  data.tar.gz: 3e4280f2afc76462071723e6bf61e2bc28339e7159e69f2223a3b08cde8b36aa
 SHA512:
-  metadata.gz: 82be7c5bab02749ab38277d18a9b2d7062100eaed096e80a2d495c6f820f066faf0afe69742ff8022387998af446960bdd815b360d0676453cf4cd734b082ffc
-  data.tar.gz: b6870227f27cac3e3cfe18c61d5e3386453b2874f2840843119e20fa2a1c945f48b5ae980e77546b8261997eb91d18321ace95b2b8d83bee82481268feb59ac9
+  metadata.gz: 1013e6d19a201f4d8c16c6ddcbf7592d7342b3f67a8b3f24f6819f0a7f5f8861b7bda50d1f91dd0830886f898bdd358e4b35e78dc39a070e2b29a608faa713e6
+  data.tar.gz: e4c97d3b9b7795d5d427352b21b42bdf67f8143e526486a4a7fcf5d4b649cbdb0b0fab64852a02465b3f0feee314aca4fd08da5a867464fefb428b71ba4a7a1c

data/app/controllers/paper_trail_history/versions_controller.rb

@@ -14,19 +14,20 @@ module PaperTrailHistory
       result = VersionService.restore_version(@version.id)
 
       if result[:success]
-        redirect_back fallback_location: version_path(@version),
-                      notice: result[:message]
+        redirect_back_or_to(version_path(@version, model_name: @version.item_type), notice: result[:message])
       else
-        redirect_back fallback_location: version_path(@version),
-                      alert: t('paper_trail_history.errors.restore_failed', error: result[:error])
+        redirect_back_or_to(version_path(@version, model_name: @version.item_type),
+                            alert: t('paper_trail_history.errors.restore_failed', error: result[:error]))
       end
     end
 
     private
 
     def find_version
-      @version = PaperTrail::Version.find(params[:id])
-    rescue ActiveRecord::RecordNotFound
+      @version = VersionService.find_version(params[:id], params[:model_name])
+
+      return if @version
+
       redirect_to root_path, alert: t('paper_trail_history.errors.version_not_found')
     end
   end

data/app/helpers/paper_trail_history/application_helper.rb

@@ -3,5 +3,16 @@
 module PaperTrailHistory
   # Helper module providing utility methods for PaperTrailHistory views
   module ApplicationHelper
+    # Generate version path with model_name context when available
+    def version_link_path(version, options = {})
+      model_name = options[:model_name] || version.item_type
+      version_path(version.id, model_name: model_name)
+    end
+
+    # Generate restore version path with model_name context
+    def restore_version_link_path(version, options = {})
+      model_name = options[:model_name] || version.item_type
+      restore_version_path(version.id, model_name: model_name)
+    end
   end
 end

data/app/models/paper_trail_history/version_service.rb

@@ -19,6 +19,19 @@ module PaperTrailHistory
       apply_record_filters(versions, params).order(created_at: :desc)
     end
 
+    def self.find_version(version_id, model_name = nil)
+      if model_name.present?
+        # Direct query when model_name is known (fast)
+        trackable_model = TrackableModel.find(model_name)
+        return nil unless trackable_model
+
+        trackable_model.version_class.unscoped.find_by(id: version_id)
+      else
+        # Fall back to searching across all tables (backwards compatible)
+        find_version_across_tables(version_id)
+      end
+    end
+
     def self.restore_version(version_id)
       version = find_version_across_tables(version_id)
       return validate_version_for_restore(version) unless version_restorable?(version)

data/app/views/layouts/paper_trail_history/application.html.erb

@@ -8,9 +8,88 @@
 
   <%= yield :head %>
 
-  <%= stylesheet_link_tag "paper_trail_history/application", media: "all" %>
   <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.3.0/dist/css/bootstrap.min.css" rel="stylesheet">
   <link href="https://cdn.jsdelivr.net/npm/bootstrap-icons@1.10.0/font/bootstrap-icons.css" rel="stylesheet">
+
+  <style>
+    /* Git-style diff formatting */
+    .diff-container {
+      font-family: 'SFMono-Regular', 'Monaco', 'Inconsolata', 'Roboto Mono', monospace;
+      font-size: 0.9rem;
+    }
+
+    .diff-section {
+      border: 1px solid #e1e4e8;
+      border-radius: 6px;
+      overflow: hidden;
+    }
+
+    .diff-header {
+      background-color: #f6f8fa;
+      padding: 8px 12px;
+      border-bottom: 1px solid #e1e4e8;
+      font-weight: 600;
+      color: #586069;
+    }
+
+    .diff-content {
+      background-color: #fff;
+    }
+
+    .diff-line {
+      display: flex;
+      align-items: flex-start;
+      padding: 2px 0;
+      line-height: 1.45;
+      position: relative;
+    }
+
+    .diff-line code {
+      background: none;
+      border: none;
+      padding: 4px 8px;
+      margin: 0;
+      flex: 1;
+      font-size: 0.85rem;
+      white-space: pre-wrap;
+      word-break: break-all;
+    }
+
+    .diff-marker {
+      display: inline-block;
+      width: 20px;
+      text-align: center;
+      font-weight: bold;
+      flex-shrink: 0;
+      font-size: 0.9rem;
+      padding: 4px 0;
+      user-select: none;
+    }
+
+    .diff-removed {
+      background-color: #ffeef0;
+      border-left: 3px solid #f85149;
+    }
+
+    .diff-removed .diff-marker {
+      color: #f85149;
+      background-color: #ffdddf;
+    }
+
+    .diff-added {
+      background-color: #e6ffed;
+      border-left: 3px solid #2ea043;
+    }
+
+    .diff-added .diff-marker {
+      color: #2ea043;
+      background-color: #ccf2d4;
+    }
+
+    .diff-line:hover {
+      background-color: rgba(0, 0, 0, 0.05);
+    }
+  </style>
 </head>
 <body>
   <nav class="navbar navbar-expand-lg navbar-dark bg-dark">

data/app/views/paper_trail_history/shared/_versions_table.html.erb

@@ -16,8 +16,8 @@
       <% versions.each do |decorated_version| %>
         <tr>
           <td>
-            <%= link_to "##{decorated_version.version.id}", 
-                version_path(decorated_version.version.id), 
+            <%= link_to "##{decorated_version.version.id}",
+                version_link_path(decorated_version.version),
                 class: "text-decoration-none" %>
           </td>
           <td>
@@ -41,13 +41,13 @@
           <% end %>
           <td>
             <div class="btn-group btn-group-sm">
-              <%= link_to "View", version_path(decorated_version.version.id), class: "btn btn-outline-primary btn-sm" %>
+              <%= link_to "View", version_link_path(decorated_version.version), class: "btn btn-outline-primary btn-sm" %>
               <% if decorated_version.can_restore? %>
-                <%= button_to "Restore", 
-                    restore_version_path(decorated_version.version.id), 
+                <%= button_to "Restore",
+                    restore_version_link_path(decorated_version.version),
                     method: :patch,
-                    data: { 
-                      confirm: "Are you sure you want to restore this version?" 
+                    data: {
+                      confirm: "Are you sure you want to restore this version?"
                     },
                     class: "btn btn-outline-warning btn-sm" %>
               <% end %>

data/app/views/paper_trail_history/versions/show.html.erb

@@ -10,11 +10,11 @@
   <h1>Version Details</h1>
   <div>
     <% if @decorated_version.can_restore? %>
-      <%= button_to "Restore This Version", 
-          restore_version_path(@version), 
+      <%= button_to "Restore This Version",
+          restore_version_link_path(@version),
           method: :patch,
-          data: { 
-            confirm: "Are you sure you want to restore this version? This will overwrite the current data." 
+          data: {
+            confirm: "Are you sure you want to restore this version? This will overwrite the current data."
           },
           class: "btn btn-warning" %>
     <% end %>
@@ -68,11 +68,11 @@
             This will restore the record to the state it was in at this version.
             <strong>This action cannot be undone</strong>, but will create a new version entry.
           </p>
-          <%= button_to "Restore This Version", 
-              restore_version_path(@version), 
+          <%= button_to "Restore This Version",
+              restore_version_link_path(@version),
               method: :patch,
-              data: { 
-                confirm: "Are you sure you want to restore this version? This will overwrite the current data and cannot be undone." 
+              data: {
+                confirm: "Are you sure you want to restore this version? This will overwrite the current data and cannot be undone."
               },
               class: "btn btn-warning" %>
         </div>

data/lib/paper_trail_history/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PaperTrailHistory
-  VERSION = '0.1.3'
+  VERSION = '0.2.1'
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: paper_trail_history
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.2.1
 platform: ruby
 authors:
 - Benjamin Deutscher
 bindir: bin
 cert_chain: []
-date: 2025-08-13 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: paper_trail
@@ -29,14 +29,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: '8.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: '8.0'
 description: |
   PaperTrailHistory is a mountable Rails engine that provides a comprehensive web interface for viewing, searching,
   and managing audit trail versions created by the PaperTrail gem. Features include listing trackable models,
@@ -50,7 +50,6 @@ files:
 - MIT-LICENSE
 - README.md
 - Rakefile
-- app/assets/stylesheets/paper_trail_history/application.css
 - app/controllers/paper_trail_history/application_controller.rb
 - app/controllers/paper_trail_history/models_controller.rb
 - app/controllers/paper_trail_history/records_controller.rb
@@ -93,14 +92,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.1.0
+      version: 3.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.6
+rubygems_version: 3.6.9
 specification_version: 4
 summary: A Rails engine providing a web interface for managing PaperTrail versions.
 test_files: []

data/app/assets/stylesheets/paper_trail_history/application.css

@@ -1,93 +0,0 @@
-/*
- * This is a manifest file that'll be compiled into application.css, which will include all the files
- * listed below.
- *
- * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
- * or any plugin's vendor/assets/stylesheets directory can be referenced here using a relative path.
- *
- * You're free to add application-wide styles to this file and they'll appear at the bottom of the
- * compiled file so the styles you add here take precedence over styles defined in any other CSS/SCSS
- * files in this directory. Styles in this file should be added after the last require_* statement.
- * It is generally better to create a new file per style scope.
- *
- *= require_tree .
- *= require_self
- */
-
-/* Git-style diff formatting */
-.diff-container {
-  font-family: 'SFMono-Regular', 'Monaco', 'Inconsolata', 'Roboto Mono', monospace;
-  font-size: 0.9rem;
-}
-
-.diff-section {
-  border: 1px solid #e1e4e8;
-  border-radius: 6px;
-  overflow: hidden;
-}
-
-.diff-header {
-  background-color: #f6f8fa;
-  padding: 8px 12px;
-  border-bottom: 1px solid #e1e4e8;
-  font-weight: 600;
-  color: #586069;
-}
-
-.diff-content {
-  background-color: #fff;
-}
-
-.diff-line {
-  display: flex;
-  align-items: flex-start;
-  padding: 2px 0;
-  line-height: 1.45;
-  position: relative;
-}
-
-.diff-line code {
-  background: none;
-  border: none;
-  padding: 4px 8px;
-  margin: 0;
-  flex: 1;
-  font-size: 0.85rem;
-  white-space: pre-wrap;
-  word-break: break-all;
-}
-
-.diff-marker {
-  display: inline-block;
-  width: 20px;
-  text-align: center;
-  font-weight: bold;
-  flex-shrink: 0;
-  font-size: 0.9rem;
-  padding: 4px 0;
-  user-select: none;
-}
-
-.diff-removed {
-  background-color: #ffeef0;
-  border-left: 3px solid #f85149;
-}
-
-.diff-removed .diff-marker {
-  color: #f85149;
-  background-color: #ffdddf;
-}
-
-.diff-added {
-  background-color: #e6ffed;
-  border-left: 3px solid #2ea043;
-}
-
-.diff-added .diff-marker {
-  color: #2ea043;
-  background-color: #ccf2d4;
-}
-
-.diff-line:hover {
-  background-color: rgba(0, 0, 0, 0.05);
-}