paper_trail_history 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9d4cf46205ebd10b78ce9d159df2a384e2b00071a80efad39bf2e79bbcb84b05
-  data.tar.gz: 7b1155089a6b7328d76d63d137fb527e0e68219da77da09f1358d824c6f2bfe5
+  metadata.gz: e8f45c5efd0324c99a10d738d2c8db6e6f06b809ded91d04ae2ea1b5a7d6a30a
+  data.tar.gz: 3e4280f2afc76462071723e6bf61e2bc28339e7159e69f2223a3b08cde8b36aa
 SHA512:
-  metadata.gz: 92abd0ba47e73f3980186c8f52819b58b7cdf33becc042d49703233a070c3a43a312f45d2d69ecd9bc10e67bb974675c93749f05eb528dd1806f410763e36ff4
-  data.tar.gz: f9a7cd1637b73274d866138706ad49ff1f1b1222575f4aca7dff2f8114584c9b30bcfb20f71ed731fb0af1234cb6bebef631f5eb2d23584a0bcd571dbee8cbec
+  metadata.gz: 1013e6d19a201f4d8c16c6ddcbf7592d7342b3f67a8b3f24f6819f0a7f5f8861b7bda50d1f91dd0830886f898bdd358e4b35e78dc39a070e2b29a608faa713e6
+  data.tar.gz: e4c97d3b9b7795d5d427352b21b42bdf67f8143e526486a4a7fcf5d4b649cbdb0b0fab64852a02465b3f0feee314aca4fd08da5a867464fefb428b71ba4a7a1c

data/app/controllers/paper_trail_history/versions_controller.rb

@@ -14,9 +14,9 @@ module PaperTrailHistory
       result = VersionService.restore_version(@version.id)
 
       if result[:success]
-        redirect_back_or_to(version_path(@version), notice: result[:message])
+        redirect_back_or_to(version_path(@version, model_name: @version.item_type), notice: result[:message])
       else
-        redirect_back_or_to(version_path(@version),
+        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
@@ -24,8 +24,10 @@ module PaperTrailHistory
     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/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.2.0'
+  VERSION = '0.2.1'
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: paper_trail_history
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Benjamin Deutscher
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-12-04 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: paper_trail
@@ -86,7 +85,6 @@ metadata:
   source_code_uri: https://github.com/its-bede/paper_trail_history
   changelog_uri: https://github.com/its-bede/paper_trail_history/blob/main/CHANGELOG.md
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -101,8 +99,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.23
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: A Rails engine providing a web interface for managing PaperTrail versions.
 test_files: []